You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Branko Čibej <br...@xbc.nu> on 2003/02/12 01:36:20 UTC
Re: svn commit: rev 4847 - in trunk/subversion: include libsvn_client
clients/cmdline libsvn_ra_svn libsvn_ra_dav
rooneg@tigris.org wrote:
>Author: rooneg
>Date: 2003-02-11 17:51:27 -0600 (Tue, 11 Feb 2003)
>New Revision: 4847
>
>Modified:
> trunk/subversion/clients/cmdline/main.c
> trunk/subversion/include/svn_config.h
> trunk/subversion/libsvn_client/auth.c
> trunk/subversion/libsvn_client/client.h
> trunk/subversion/libsvn_ra_dav/fetch.c
> trunk/subversion/libsvn_ra_dav/ra_dav.h
> trunk/subversion/libsvn_ra_dav/session.c
> trunk/subversion/libsvn_ra_dav/util.c
> trunk/subversion/libsvn_ra_svn/client.c
>Log:
>Fix a few nits that Greg 'Eagle Eyes' Stein found in revision 4803.
>
...
>Modified: trunk/subversion/include/svn_config.h
>==============================================================================
>--- trunk/subversion/include/svn_config.h (original)
>+++ trunk/subversion/include/svn_config.h Tue Feb 11 17:51:32 2003
>@@ -47,6 +47,11 @@
> /** Opaque structure describing a set of configuration options. */
> typedef struct svn_config_t svn_config_t;
>
>+/** Config category corresponding to the ~/.subversion/config file. */
>+#define SVN_CONFIG_CATEGORY_CONFIG "config"
>+
>+/** Config category corresponding to the ~/.subversion/servers file. */
>+#define SVN_CONFIG_CATEGORY_SERVERS "servers"
>
> /** Read configuration information from all available sources and merge it
> * into one @c svn_config_t object.
>
>
What's wrong with just moving SVN_CONFIG__USR_CONFIG_FILE and
SVN_CONFIG__USR_SERVERS_FILE here from config_impl.h? Then the
categories and file names could just be the same thing. I'd even use the
same constants for the registry key names, regardless of (inconsistent?)
capitalization.
--
Brane Čibej <br...@xbc.nu> http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: rev 4847 - in trunk/subversion: include libsvn_client
clients/cmdline libsvn_ra_svn libsvn_ra_dav
Posted by Branko Čibej <br...@xbc.nu>.
Garrett Rooney wrote:
> On Tuesday, February 11, 2003, at 08:36 PM, Branko Čibej wrote:
>
>> What's wrong with just moving SVN_CONFIG__USR_CONFIG_FILE and
>> SVN_CONFIG__USR_SERVERS_FILE here from config_impl.h? Then the
>> categories and file names could just be the same thing. I'd even use the
>> same constants for the registry key names, regardless of (inconsistent?)
>> capitalization.
>
>
> there's nothing especially wrong with it other than me not thinking to
> do it ;-)
>
> if nobody objects, i'll axe the __USR_*_FILE defines in config_impl.h
> and use the new public defines. i'll leave the windows registry stuff
> to someone who can test it though.
Fine.
--
Brane Čibej <br...@xbc.nu> http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: rev 4847 - in trunk/subversion: include libsvn_client clients/cmdline libsvn_ra_svn libsvn_ra_dav
Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On Tuesday, February 11, 2003, at 08:36 PM, Branko Čibej wrote:
> What's wrong with just moving SVN_CONFIG__USR_CONFIG_FILE and
> SVN_CONFIG__USR_SERVERS_FILE here from config_impl.h? Then the
> categories and file names could just be the same thing. I'd even use
> the
> same constants for the registry key names, regardless of
> (inconsistent?)
> capitalization.
there's nothing especially wrong with it other than me not thinking to
do it ;-)
if nobody objects, i'll axe the __USR_*_FILE defines in config_impl.h
and use the new public defines. i'll leave the windows registry stuff
to someone who can test it though.
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org