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"

Reply via email to