[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[PATCH 07/10] enclose some logical expressions in parens for clarity



From: Andrei Zavada <johnhommer@gmail.com>

---
 MUMmer3.20/src/tigr/repeat-match.cc               | 2 +-
 mugsy-seqan/projects/library/apps/mugsy/mugsy.cpp | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/MUMmer3.20/src/tigr/repeat-match.cc b/MUMmer3.20/src/tigr/repeat-match.cc
index a4bf103..f7aa881 100644
--- a/MUMmer3.20/src/tigr/repeat-match.cc
+++ b/MUMmer3.20/src/tigr/repeat-match.cc
@@ -847,7 +847,7 @@ void  List_Matches
         {
          if  (Data [i - 1] == Data [j - 1]
                   || Data [i + n] == Data [j + n]
-                  || i > String_Separator && j > String_Separator)
+                  || (i > String_Separator && j > String_Separator))
              continue;
          Reversed = FALSE;
          if  (j > String_Separator)
diff --git a/mugsy-seqan/projects/library/apps/mugsy/mugsy.cpp b/mugsy-seqan/projects/library/apps/mugsy/mugsy.cpp
index 177468e..da8f331 100644
--- a/mugsy-seqan/projects/library/apps/mugsy/mugsy.cpp
+++ b/mugsy-seqan/projects/library/apps/mugsy/mugsy.cpp
@@ -1187,7 +1187,7 @@ void doReadBlockFile(const std::string & filename,
 	      //Check if segment is reported as part of the LCB
 	      //The LCB identification step may not report all sequences that are part of the anchor
 	      if(sequenceOrientMap.find(sname) != sequenceOrientMap.end()){
-		if(!checkbounds || fragmentBegin(g,*vit)>=sequenceCoordsMap[sname].first && fragmentBegin(g,*vit)<=sequenceCoordsMap[sname].second){
+		if(!checkbounds || (fragmentBegin(g,*vit)>=sequenceCoordsMap[sname].first && fragmentBegin(g,*vit)<=sequenceCoordsMap[sname].second)){  // -Wuse-spacebar, ffs!
 		  vertexOrientMap[*vit] = sequenceOrientMap[sname];
 		  //add node to block
 		  currlcb.push_back(*vit);
-- 
2.3.5


Reply to: