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

Re: [PATCH] global-ignores line too long in config

From: David Glasser <glasser_at_davidglasser.net>
Date: 2007-10-02 00:30:01 CEST

On 10/1/07, Daniel Rall <dlr@collab.net> wrote:
> On Sat, 29 Sep 2007, David Glasser wrote:
>
> > On 9/28/07, Daniel L. Rall <dlr@finemaltcoding.com> wrote:
> > > On Fri, 28 Sep 2007, David Glasser wrote:
> > > ...
> > > > Index: subversion/include/svn_config.h
> > > > ===================================================================
> > > > --- subversion/include/svn_config.h (revision 26831)
> > > > +++ subversion/include/svn_config.h (working copy)
> > > > @@ -121,9 +121,17 @@ typedef struct svn_config_t svn_config_t
> > > > /*** Configuration Default Values ***/
> > > >
> > > > /* '*' matches leading dots, e.g. '*.rej' matches '.foo.rej'. */
> > > > +/* We want this to be printed on two lines in the generated config file,
> > > > + * but we don't want the # character to end up in the variable.
> > > > + */
> > > > +#define SVN_CONFIG__DEFAULT_GLOBAL_IGNORES_LINE_1 \
> > > > + "*.o *.lo *.la *.al .libs *.so *.so.[0-9]* *.a *.pyc *.pyo"
> > > > +#define SVN_CONFIG__DEFAULT_GLOBAL_IGNORES_LINE_2 \
> > > > + "*.rej *~ #*# .#* .*.swp .DS_Store"
> > > > +
> > > > #define SVN_CONFIG_DEFAULT_GLOBAL_IGNORES \
> > > > - "*.o *.lo *.la *.al .libs *.so *.so.[0-9]* *.a *.pyc *.pyo" \
> > > > - " *.rej *~ #*# .#* .*.swp .DS_Store"
> > > > + SVN_CONFIG__DEFAULT_GLOBAL_IGNORES_LINE_1 " " \
> > > > + SVN_CONFIG__DEFAULT_GLOBAL_IGNORES_LINE_2
> > >
> > > I'd rather not see more double-underscore symbols -- and these
> > > #defines in particular -- in a public header file.
> > > You could put them into a new include/private/ header instead.
> > >
> > > The patch looks reasonable to me, other than that. (The ugliness
> > > is intrinsic, unfortunately.)
> >
> > I guess. So should the private header be included right there inside
> > svn_config.h? That also leads to the value not being visible in the
> > header, which is a shame.
>
> Bah. Perhaps it's not worth it. :-\

OK. In that case, I'm going to go with ugliness in the
developer-visible file (ie, the patch I posted) rather than ugliness
in the user-visible config file (ie, the status quo).

--dave

-- 
David Glasser | glasser_at_davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Oct 2 00:30:10 2007

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