From: Henrik Nordstrom <henrik@henriknordstrom.net>
Date: Sun, 4 Mar 2012 18:50:13 +0000 (+0100)
Subject: Break out assignment from if clause to make code more readable (and gcc happier)
X-Git-Tag: BumpSslServerFirst.take06~2^2~9
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=baa31c42f4b1200a30d58db539f82b6d7de73667;p=thirdparty%2Fsquid.git

Break out assignment from if clause to make code more readable (and gcc happier)
---

diff --git a/src/adaptation/ecap/XactionRep.cc b/src/adaptation/ecap/XactionRep.cc
index 3c6c875260..df0276223f 100644
--- a/src/adaptation/ecap/XactionRep.cc
+++ b/src/adaptation/ecap/XactionRep.cc
@@ -199,8 +199,8 @@ Adaptation::Ecap::XactionRep::visitEachMetaHeader(libecap::NamedValueVisitor &vi
 
     typedef Adaptation::Config::MetaHeaders::iterator ACAMLI;
     for (ACAMLI i = Adaptation::Config::metaHeaders.begin(); i != Adaptation::Config::metaHeaders.end(); ++i) {
-        const char *v;
-        if (v = (*i)->match(request, reply)) {
+        const char *v = (*i)->match(request, reply);
+        if (v) {
             const libecap::Name name((*i)->name.termedBuf());
             const libecap::Area value = libecap::Area::FromTempString(v);
             visitor.visit(name, value);