|
 |
25845f |
commit 78dd658a025fb16d004677284b4db0d0c0191845
|
|
|
25845f |
Author: Siddhesh Poyarekar <siddhesh@redhat.com>
|
|
|
25845f |
Date: Fri Aug 1 14:24:41 2014 +0530
|
|
|
25845f |
|
|
|
25845f |
Check if DEBUG is defined in regex_internal.c
|
|
|
25845f |
|
|
|
25845f |
The DEBUG macro is checked for its value in one place and if it is
|
|
|
25845f |
defined in another. Make this consistent across the two cases and use
|
|
|
25845f |
the same style that we did in mktime.c, which is to check if the macro
|
|
|
25845f |
is defined and it is set.
|
|
|
25845f |
|
|
|
25845f |
diff --git a/posix/regex_internal.c b/posix/regex_internal.c
|
|
|
25845f |
index 9be8a532e6d34150..1765035f06e906e5 100644
|
|
|
25845f |
--- a/posix/regex_internal.c
|
|
|
25845f |
+++ b/posix/regex_internal.c
|
|
|
25845f |
@@ -679,7 +679,7 @@ re_string_reconstruct (re_string_t *pstr, int idx, int eflags)
|
|
|
25845f |
pstr->valid_len - offset);
|
|
|
25845f |
pstr->valid_len -= offset;
|
|
|
25845f |
pstr->valid_raw_len -= offset;
|
|
|
25845f |
-#if DEBUG
|
|
|
25845f |
+#if defined DEBUG && DEBUG
|
|
|
25845f |
assert (pstr->valid_len > 0);
|
|
|
25845f |
#endif
|
|
|
25845f |
}
|
|
|
25845f |
@@ -936,7 +936,7 @@ re_string_context_at (const re_string_t *input, int idx, int eflags)
|
|
|
25845f |
int wc_idx = idx;
|
|
|
25845f |
while(input->wcs[wc_idx] == WEOF)
|
|
|
25845f |
{
|
|
|
25845f |
-#ifdef DEBUG
|
|
|
25845f |
+#if defined DEBUG && DEBUG
|
|
|
25845f |
/* It must not happen. */
|
|
|
25845f |
assert (wc_idx >= 0);
|
|
|
25845f |
#endif
|