On Mon, May 20, 2013 at 5:47 PM, Philip Martin
<philip.mar...@wandisco.com> wrote:
> Ivan Zhakov <i...@visualsvn.com> writes:
>
>> On Wed, Mar 27, 2013 at 10:39 PM,  <cmpil...@apache.org> wrote:
>>> Author: cmpilato
>>> Date: Wed Mar 27 18:39:56 2013
>>> New Revision: 1461775
>>>
>>> URL: http://svn.apache.org/r1461775
>>> Log:
>>> * subversion/include/svn_error.h
>>>   (SVN_ERR_ASSERT): Make this just a wrapper around SVN_ERR_ASSERT_E.
>>>
>> It seems to be incorrect change, because expr gets evaluated before
>> expanding SVN_ERR_ASSERT_E macro and we get just expression value
>> instead of expression itself. At least on Windows with VC10.
>
> Could you give a bit more information.  With GCC
>
>     SVN_ERR_ASSERT(1+1==3);
>
> produces
>
> ../src/subversion/svn/status-cmd.c:275: (apr_err=SVN_ERR_ASSERTION_FAIL)
> svn: E235000: In file '../src/subversion/svn/status-cmd.c' line 275: 
> assertion failed (1+1==3)
> Aborted
>
> Are you saying that VC10 produces something different?
>
Yes, with VC10 I get "assertion failed (3)"


-- 
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com

Reply via email to