Arfrever Frehtes Taifersar Arahesis wrote on Tue, 30 Dec 2008 at 19:04 +0100:
> 2008-12-30 00:25:50 Daniel Shahaf napisał(a):
> > > /* Update the options in the config */
> > > + for (i = 0; i < opt_state.config_options->nelts; i++)
> > > + {
> > > + config_option_t *config_option = APR_ARRAY_IDX(opt_state.config_options,
> > > + i, config_option_t *);
> > > + if (strcmp(config_option->file, "config") == 0)
> > > + {
> > > + svn_config_set(cfg_config, config_option->section,
> > > + config_option->option, config_option->value);
> > > + }
> > > + else if(strcmp(config_option->file, "servers") == 0)
> > > + {
> > > + svn_config_set(cfg_servers, config_option->section,
> > > + config_option->option, config_option->value);
> > > + }
> >
> > else
> > {
> > /* print a warning */
> > }
> >
> > ?
> >
> > > + }
> > > +
>
> I implemented the majority of these suggestions in r34975.
>
>
Thanks. Looks good. I see you didn't implement the last one (quoted
here), any particular reason why?
Thanks,
Daniel
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=996140
Received on 2008-12-30 20:12:23 CET