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

Re: [PATCH] pass NULL instead of an unused parameter

From: Prabhu Gnana Sundar <prabhugs_at_collab.net>
Date: Tue, 11 Jan 2011 17:47:12 +0530

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.

Thanks and regards
Prabhu

Received on 2011-01-11 13:18:00 CET

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