This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
handy.h: Rmv duplicated assert in inRANGE()
authorKarl Williamson <khw@cpan.org>
Sat, 21 Sep 2019 18:23:49 +0000 (12:23 -0600)
committerKarl Williamson <khw@cpan.org>
Thu, 26 Sep 2019 04:30:43 +0000 (22:30 -0600)
This assertion is done in the macro that is called to do the real work.

handy.h

diff --git a/handy.h b/handy.h
index f011882..e584efd 100644 (file)
--- a/handy.h
+++ b/handy.h
@@ -1326,7 +1326,7 @@ or casts
 /* Returns true if c is in the range l..u, where 'l' is non-negative
  * Written this way so that after optimization, only one conditional test is
  * needed. */
-#define inRANGE(c, l, u) (__ASSERT_((l) >= 0) __ASSERT_((u) >= (l))            \
+#define inRANGE(c, l, u) (__ASSERT_((u) >= (l))                                \
    (  (sizeof(c) == sizeof(U8))  ? withinCOUNT(((U8)  (c)), (l), ((u) - (l)))  \
     : (sizeof(c) == sizeof(U16)) ? withinCOUNT(((U16) (c)), (l), ((u) - (l)))  \
     : (sizeof(c) == sizeof(U32)) ? withinCOUNT(((U32) (c)), (l), ((u) - (l)))  \