fix lgtm alerts

Each of the removed tests was redundant.

diff --git a/mutt/logging.c b/mutt/logging.c
index 74760a2a38e6eb578ddd007be6fb938b71837a6b..f8c4ecb4181206f05d9bccf118b052f46508ddd8 100644 (file)
--- a/mutt/logging.c
+++ b/mutt/logging.c
@@ -494,8 +494,7 @@ int log_disp_terminal(time_t stamp, const char *file, int line,
   if (colour > 0)
     ret += fprintf(fp, "\033[0m");
 
-  if (level < 1)
-    ret += fprintf(fp, "\n");
+  ret += fprintf(fp, "\n");
 
   return ret;
 }

diff --git a/muttlib.c b/muttlib.c
index a5838a4913a25bc70e84f97d1f7903800eecbd2e..ce5370eb1ab1bb5201c17ae834ead1025fac8599 100644 (file)
--- a/muttlib.c
+++ b/muttlib.c
@@ -1173,7 +1173,7 @@ void mutt_expando_format(char *buf, size_t buflen, size_t col, int cols, const c
               col += pw;
             }
           }
-          else if (soft && pad < 0)
+          else if (soft)
           {
             int offset = ((flags & MUTT_FORMAT_ARROWCURSOR) && ArrowCursor) ? 3 : 0;
             int avail_cols = (cols > offset) ? (cols - offset) : 0;

diff --git a/ncrypt/pgp.c b/ncrypt/pgp.c
index 8ec8f0c732d3661a752902adb72d5b5902854873..a5ddce897f188ab52db6a60916a41cc08f6969b2 100644 (file)
--- a/ncrypt/pgp.c
+++ b/ncrypt/pgp.c
@@ -375,8 +375,7 @@ static int pgp_check_decryption_okay(FILE *fpin)
       if (!inside_decrypt)
       {
         mutt_debug(2, "\tPLAINTEXT encountered outside of DECRYPTION.\n");
-        if (rv > -2)
-          rv = -2;
+        rv = -2;
         break;
       }
     }
@@ -391,8 +390,7 @@ static int pgp_check_decryption_okay(FILE *fpin)
       /* Don't break out because we still have to check for
        * PLAINTEXT outside of the decryption boundaries. */
       mutt_debug(2, "\tDECRYPTION_OKAY encountered.\n");
-      if (rv > -2)
-        rv = 0;
+      rv = 0;
     }
   }
   FREE(&line);

diff --git a/ncrypt/pgppacket.c b/ncrypt/pgppacket.c
index 9ad9b7088dad385aed5b66d768e83155f108c75e..0bcf724324098d1c924bedced1c42f6de4fd678c 100644 (file)
--- a/ncrypt/pgppacket.c
+++ b/ncrypt/pgppacket.c
@@ -129,7 +129,7 @@ unsigned char *pgp_read_packet(FILE *fp, size_t *len)
         material = b;
         partial = false;
       }
-      else if (192 <= b && b <= 223)
+      else if (b <= 223)
       {
         material = (b - 192) * 256;
         if (fread(&b, 1, 1, fp) < 1)