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

Re: svn commit: r31291 - trunk/subversion/bindings/javahl/native

From: Philip Martin <philip_at_codematters.co.uk>
Date: Thu, 29 May 2008 14:34:28 +0100

Arfrever Frehtes Taifersar Arahesis <arfrever.fta_at_gmail.com> writes:

> 2008-05-19 20:24:13 Philip Martin napisaƂ(a):
>> arfrever_at_tigris.org writes:
>>
>> > + svn_error_clear(svn_dso_load(&dso, libname));
>> > + if (dso)
>> > + {
>> > + if (! apr_dso_sym(&provider_symbol, dso, funcname))
>> > + {
>> > + svn_auth_simple_provider_func_t func;
>> > + func = (svn_auth_simple_provider_func_t) provider_symbol;
>> > + func(provider, pool);
>> > + ret = TRUE;
>> > + }
>> > + }
>> > + return ret;
>> > +}
>> > +#endif
>>
>> I've seen this code before...
>>
>> Why is it acceptable to ignore errors from svn_dso_load?
>>
>> Why is it acceptable to use the dso variable if svn_dso_load returns
>> an error?
>
> Fixed in r31367.

The code still drops the error. Why is this code different from the
RA loader code? Why don't you simply return the error?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-29 15:34:49 CEST

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.