You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Ivan Zhakov <iv...@visualsvn.com> on 2009/05/12 13:05:18 UTC

Re: svn commit: r37701 - in trunk/subversion: include libsvn_subr

On Tue, May 12, 2009 at 4:52 PM, Greg Stein <gs...@gmail.com> wrote:
> Author: gstein
> Date: Tue May 12 05:52:45 2009
> New Revision: 37701
>
> Log:
> Add a MISSING_OK parameter to svn_io_remove_file(). A lot of our calls
> actually are okay with the file not being present. We have many places
> that catch this error, and ignore it.
>
Good thing. While I suggest use name "ignore_enoent" for consistency
with other function that already have the same semantic.

-- 
Ivan Zhakov
VisualSVN Team

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2212849

Re: svn commit: r37701 - in trunk/subversion: include libsvn_subr

Posted by Greg Stein <gs...@gmail.com>.
On Tue, May 12, 2009 at 15:05, Ivan Zhakov <iv...@visualsvn.com> wrote:
> On Tue, May 12, 2009 at 4:52 PM, Greg Stein <gs...@gmail.com> wrote:
>> Author: gstein
>> Date: Tue May 12 05:52:45 2009
>> New Revision: 37701
>>
>> Log:
>> Add a MISSING_OK parameter to svn_io_remove_file(). A lot of our calls
>> actually are okay with the file not being present. We have many places
>> that catch this error, and ignore it.
>
> Good thing. While I suggest use name "ignore_enoent" for consistency
> with other function that already have the same semantic.

Ah! I didn't see those. Thanks... will fix.

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2213468