On 01/11/2011 07:17 AM, Prabhu Gnana Sundar wrote:
> Hi Kamesh,
>
>
> On Tuesday 11 January 2011 05:23 PM, Kamesh Jayachandran wrote:
>> Hi Prabhu,
>>
>>> SVN_ERR(svn_config_get_config(&config, opt_state->config_dir, pool));
>>> SVN_ERR(svn_repos_create(&repos, opt_state->repository_path,
>>> NULL, NULL,
>>> - config, fs_config, pool));
>>> + NULL, fs_config, pool));
>>
>>
>>
>> If you do not need 'config' as well you need *not* populate it.
>
> Oh yeah ... now I just did the change by not populating the 'config' and
> also undeclared it.
> Thanks for the comments... :)
>
> I have attached the recent patch and the log message with this mail.
I suppose this patch is okay, but I really wish that the documentation for
svn_repos_create() was more clear. I mean, if the function doesn't use
'config', why does it accept 'config'? It tells API consumers that 'config'
can be passed as NULL, but doesn't say why anyone would choose or choose not
to do that. Weird.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2011-01-11 16:04:06 CET