Daniel Shahaf wrote:
> Stefan Sperling wrote on Sun, 4 Jan 2009 at 14:39 +0100:
>> On Sun, Jan 04, 2009 at 09:07:01AM +0200, Daniel Shahaf wrote:
>>>>> @@ -180,6 +190,8 @@ path_rev_packed(svn_fs_t *fs, svn_revnum
>>>>> fs_fs_data_t *ffd = fs->fsap_data;
>>>>> assert(ffd->max_files_per_dir);
>>>>> + assert(is_packed_rev(fs, rev));
>>>> We're not using assert() anymore, but SVN_ASSERT(), assuming the method
>>>> returns an svn_error_t *.
>>>>
>>> It returns const char *.
>> You could probably use SVN_ERR_ASSERT_NO_RETURN() in this case.
>> It was made for functions that do not return svn_error_t*.
>>
>> But assert()s do not get compiled into release builds anyway.
>> It's aborts() which are really bad when used outside of the
>> SVN_ERR_ASSERT* macros.
>>
>
> If people think assert()s are bad, we can change those functions so
> they return 'svn_error_t *' and use SVN_ERR_ASSERT().
That would be the best.
Blair
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1003531
Received on 2009-01-04 20:15:44 CET