Gabriela Gibson wrote on Sat, Mar 23, 2013 at 15:27:31 +0000: > [[[ > Align variables for easier reading. > > *subversion/include/config.h(): Align variables. > ]]] >
-0. The indentation is intention and reflects the nesting (e.g., --config-option=servers:global:http-proxy-username=danielsh). It's actually easier to read the way it is now. A recent log message by cmpilato actually called out that fact explicitly, didn't you notice that? Thanks, Daniel > Index: subversion/include/svn_config.h > =================================================================== > --- subversion/include/svn_config.h (revision 1460153) > +++ subversion/include/svn_config.h (working copy) > @@ -62,9 +62,9 @@ typedef struct svn_config_t svn_config_t; > * client configuration files. > * @{ > */ > -#define SVN_CONFIG_CATEGORY_SERVERS "servers" > -#define SVN_CONFIG_SECTION_GROUPS "groups" > -#define SVN_CONFIG_SECTION_GLOBAL "global" > +#define SVN_CONFIG_CATEGORY_SERVERS "servers" > +#define SVN_CONFIG_SECTION_GROUPS "groups" > +#define SVN_CONFIG_SECTION_GLOBAL "global" > #define SVN_CONFIG_OPTION_HTTP_PROXY_HOST "http-proxy-host" > #define SVN_CONFIG_OPTION_HTTP_PROXY_PORT "http-proxy-port" > #define SVN_CONFIG_OPTION_HTTP_PROXY_USERNAME "http-proxy-username"