[m5-dev] changeset in m5: When nesting if statements, use braces to avoid...

[Nathan Binkert]

changeset 875cb7d09831 in /z/repo/m5
details: http://repo.m5sim.org/m5?cmd=changeset;node=875cb7d09831
description:
        When nesting if statements, use braces to avoid ambiguous else clauses.

diffstat:

0 files changed

diffs (40 lines):

diff -r eb5664be6075 -r 875cb7d09831 src/arch/sparc/tlb_map.hh
--- a/src/arch/sparc/tlb_map.hh Fri Sep 26 08:18:56 2008 -0700
+++ b/src/arch/sparc/tlb_map.hh Fri Sep 26 08:18:57 2008 -0700
@@ -52,7 +52,7 @@
 
         i = tree.upper_bound(r);
 
-        if (i == tree.begin())
+        if (i == tree.begin()) {
             if (r.real == i->first.real &&
                 r.partitionId == i->first.partitionId &&
                 i->first.va < r.va + r.size &&
@@ -62,6 +62,7 @@
             else
                 // Nothing could match, so return end()
                 return tree.end();
+        }
 
         i--;
 
diff -r eb5664be6075 -r 875cb7d09831 src/mem/bridge.cc
--- a/src/mem/bridge.cc Fri Sep 26 08:18:56 2008 -0700
+++ b/src/mem/bridge.cc Fri Sep 26 08:18:57 2008 -0700
@@ -130,7 +130,7 @@
         return true;
     }
 
-    if (pkt->needsResponse())
+    if (pkt->needsResponse()) {
         if (respQueueFull()) {
             DPRINTF(BusBridge, "Local queue full, no space for response, 
nacking\n");
             DPRINTF(BusBridge, "queue size: %d outreq: %d outstanding resp: 
%d\n",
@@ -141,6 +141,7 @@
             DPRINTF(BusBridge, "Request Needs response, reserving space\n");
             ++outstandingResponses;
         }
+    }
 
     otherPort->queueForSendTiming(pkt);
 
_______________________________________________
m5-dev mailing list
m5-dev@m5sim.org
http://m5sim.org/mailman/listinfo/m5-dev