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

Re: [PATCH] Code tidying for subversion/include/config.h

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sat, 23 Mar 2013 17:58:03 +0200

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

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.