You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2015/01/21 13:33:59 UTC

svn commit: r1653502 - /subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c

Author: philip
Date: Wed Jan 21 12:33:59 2015
New Revision: 1653502

URL: http://svn.apache.org/r1653502
Log:
* subversion/tests/libsvn_subr/mergeinfo-test.c
  (verify_mergeinfo_parse): Rewrite conditional to avoid a GCC 5
   warning "logical not is only applied to the left hand side of
   comparison".  It also makes more sense to compare the svn_revnum_t
   directly to another integer.

Modified:
    subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c

Modified: subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c?rev=1653502&r1=1653501&r2=1653502&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c (original)
+++ subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c Wed Jan 21 12:33:59 2015
@@ -104,7 +104,7 @@ verify_mergeinfo_parse(const char *input
 
       /* Were we expecting any more ranges? */
       if (j < MAX_NBR_RANGES - 1
-          && !expected_ranges[j].end == 0)
+          && expected_ranges[j].end != 0)
         return svn_error_createf(SVN_ERR_TEST_FAILED, NULL,
                                  "svn_mergeinfo_parse (%s) failed to "
                                  "produce the expected number of ranges",