aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Neto <dneto@google.com>2017-05-08 14:53:54 -0400
committerDavid Neto <dneto@google.com>2017-05-08 14:53:54 -0400
commitf0df72c2a7c04801a36dd9ae4923f14706ab6018 (patch)
treef13b72d23446a886aef3bd89000d9e1a0d00efdd
parent61b08ed9c17639b30420eca13ad05f91bf7f1eed (diff)
downloadeffcee-f0df72c2a7c04801a36dd9ae4923f14706ab6018.tar.gz
Avoid unused var warning
-rw-r--r--effcee/check.cc11
1 files changed, 8 insertions, 3 deletions
diff --git a/effcee/check.cc b/effcee/check.cc
index 28d9227..2e5591a 100644
--- a/effcee/check.cc
+++ b/effcee/check.cc
@@ -166,8 +166,10 @@ Check::Parts PartsForPattern(StringPiece pattern) {
if (regex_exists && (!var_exists || regex_start < var_start)) {
const auto consumed =
RE2::Consume(&pattern, "(.*?){{(.*?)}}", &fixed, &regex);
- assert(consumed &&
- "Did not make forward progress for regex in check rule");
+ if (!consumed) {
+ assert(consumed &&
+ "Did not make forward progress for regex in check rule");
+ }
if (!fixed.empty()) {
parts.emplace_back(make_unique<Check::Part>(Type::Fixed, fixed));
}
@@ -177,7 +179,10 @@ Check::Parts PartsForPattern(StringPiece pattern) {
} else if (var_exists && (!regex_exists || var_start < regex_start)) {
const auto consumed =
RE2::Consume(&pattern, "(.*?)\\[\\[(.*?)\\]\\]", &fixed, &var);
- assert(consumed && "Did not make forward progress for var in check rule");
+ if (!consumed) {
+ assert(consumed &&
+ "Did not make forward progress for var in check rule");
+ }
if (!fixed.empty()) {
parts.emplace_back(make_unique<Check::Part>(Type::Fixed, fixed));
}