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"
Received on 2013-03-23 16:58:41 CET