[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: CVS update: MODIFIED: libsvn_fs ...

From: <kfogel_at_collab.net>
Date: 2001-07-24 23:20:04 CEST

Branko =?ISO-8859-2?Q?=C8ibej?= <brane@xbc.nu> writes:
> > (SVN_FS__REPOS_CONF_PRE_COMMIT_HOOKS,
> > SVN_FS__REPOS_CONF_POST_COMMIT_HOOKS): Replace
> > SVN_FS__REPOS_CONF_PRECOMMIT_HOOKS and
> > SVN_FS__REPOS_CONF_POSTCOMMIT_HOOKS respectively; an underscore was
> > added to each.
>
> <pedantic>
> This is actually wrong: "pre" is not a word, it's a prefix, therefore
> the correct spelling is "precommit", not "pre commit".
> </pedantic>
>
> Maybe I should load the SVN sources into Word and run a style check on
> them, shall I? :-)

I agree with you. But in practice, the filenames

   precommit.conf
   postcommit.conf

were a little too hard to read, and too much alike, for comfort. When
I added the hyphen...

   pre-commit.conf
   post-commit.conf

...I found them *much* easier to take in at a glance, and to
distinguish from one another.

Of course, now that there were hyphens in the actual filenames, I
realized it would be consistent to put corresponding underscores in
the C identifiers, as is customary when dealing with C's preference
for interpreting hyphen as the subtraction operator, because not a
whitespace-delimited, excuse me, whitespace_delimited language.
Sigh.

_K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:33 2006

This is an archived mail posted to the Subversion Dev mailing list.