You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2012/02/15 03:02:40 UTC
svn commit: r1244312 -
/subversion/trunk/subversion/tests/cmdline/svntest/main.py
Author: danielsh
Date: Wed Feb 15 02:02:40 2012
New Revision: 1244312
URL: http://svn.apache.org/viewvc?rev=1244312&view=rev
Log:
* subversion/tests/cmdline/svntest/main.py: Whitespace changes only.
Modified:
subversion/trunk/subversion/tests/cmdline/svntest/main.py
Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1244312&r1=1244311&r2=1244312&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Wed Feb 15 02:02:40 2012
@@ -302,13 +302,11 @@ def get_start_commit_hook_path(repo_dir)
return os.path.join(repo_dir, "hooks", "start-commit")
-
def get_pre_commit_hook_path(repo_dir):
"Return the path of the pre-commit-hook conf file in REPO_DIR."
return os.path.join(repo_dir, "hooks", "pre-commit")
-
def get_post_commit_hook_path(repo_dir):
"Return the path of the post-commit-hook conf file in REPO_DIR."
Re: svn commit: r1244312 -
/subversion/trunk/subversion/tests/cmdline/svntest/main.py
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
It was normalization: it makes the separation consistent with the
functions immediately above/below it in the file.
Greg Stein wrote on Wed, Feb 15, 2012 at 01:19:33 -0500:
> Since single blank lines are commonplace throughout our code/functions, I
> do like to use double-blank to separate functions. Visually, it works well
> to separate functions (or other semantic entities that typically contain
> single blanks).
>
> Maybe in this case, the file was using single-blank between funcs, so this
> commit is just a normalization. Tho I might have doubled all the others
> instead :-)
>
> Cheers,
> -g
> On Feb 14, 2012 9:03 PM, <da...@apache.org> wrote:
>
> > Author: danielsh
> > Date: Wed Feb 15 02:02:40 2012
> > New Revision: 1244312
> >
> > URL: http://svn.apache.org/viewvc?rev=1244312&view=rev
> > Log:
> > * subversion/tests/cmdline/svntest/main.py: Whitespace changes only.
> >
> > Modified:
> > subversion/trunk/subversion/tests/cmdline/svntest/main.py
> >
> > Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
> > URL:
> > http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1244312&r1=1244311&r2=1244312&view=diff
> >
> > ==============================================================================
> > --- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
> > +++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Wed Feb 15
> > 02:02:40 2012
> > @@ -302,13 +302,11 @@ def get_start_commit_hook_path(repo_dir)
> >
> > return os.path.join(repo_dir, "hooks", "start-commit")
> >
> > -
> > def get_pre_commit_hook_path(repo_dir):
> > "Return the path of the pre-commit-hook conf file in REPO_DIR."
> >
> > return os.path.join(repo_dir, "hooks", "pre-commit")
> >
> > -
> > def get_post_commit_hook_path(repo_dir):
> > "Return the path of the post-commit-hook conf file in REPO_DIR."
> >
> >
> >
> >
Re: svn commit: r1244312 - /subversion/trunk/subversion/tests/cmdline/svntest/main.py
Posted by Greg Stein <gs...@gmail.com>.
Since single blank lines are commonplace throughout our code/functions, I
do like to use double-blank to separate functions. Visually, it works well
to separate functions (or other semantic entities that typically contain
single blanks).
Maybe in this case, the file was using single-blank between funcs, so this
commit is just a normalization. Tho I might have doubled all the others
instead :-)
Cheers,
-g
On Feb 14, 2012 9:03 PM, <da...@apache.org> wrote:
> Author: danielsh
> Date: Wed Feb 15 02:02:40 2012
> New Revision: 1244312
>
> URL: http://svn.apache.org/viewvc?rev=1244312&view=rev
> Log:
> * subversion/tests/cmdline/svntest/main.py: Whitespace changes only.
>
> Modified:
> subversion/trunk/subversion/tests/cmdline/svntest/main.py
>
> Modified: subversion/trunk/subversion/tests/cmdline/svntest/main.py
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svntest/main.py?rev=1244312&r1=1244311&r2=1244312&view=diff
>
> ==============================================================================
> --- subversion/trunk/subversion/tests/cmdline/svntest/main.py (original)
> +++ subversion/trunk/subversion/tests/cmdline/svntest/main.py Wed Feb 15
> 02:02:40 2012
> @@ -302,13 +302,11 @@ def get_start_commit_hook_path(repo_dir)
>
> return os.path.join(repo_dir, "hooks", "start-commit")
>
> -
> def get_pre_commit_hook_path(repo_dir):
> "Return the path of the pre-commit-hook conf file in REPO_DIR."
>
> return os.path.join(repo_dir, "hooks", "pre-commit")
>
> -
> def get_post_commit_hook_path(repo_dir):
> "Return the path of the post-commit-hook conf file in REPO_DIR."
>
>
>
>