[exim-cvs] Quieten noisy compiler

Top Page
Delete this message
Reply to this message
Author: Exim Git Commits Mailing List
Date:  
To: exim-cvs
Subject: [exim-cvs] Quieten noisy compiler
Gitweb: http://git.exim.org/exim.git/commitdiff/8fa0a4d42bcf892375dafa2d8e4cc7f532a41145
Commit:     8fa0a4d42bcf892375dafa2d8e4cc7f532a41145
Parent:     8bc732e8b03ebb4309f4b42626917148d176db49
Author:     Jeremy Harris <jgh146exb@???>
AuthorDate: Sun Oct 12 18:18:51 2014 +0100
Committer:  Jeremy Harris <jgh146exb@???>
CommitDate: Sun Oct 12 18:18:51 2014 +0100


    Quieten noisy compiler


    As usual, gcc whining that perfectly valid C coding is
    "ambiguous".  Wrongly.
---
 src/src/deliver.c |   15 +++++++--------
 1 files changed, 7 insertions(+), 8 deletions(-)


diff --git a/src/src/deliver.c b/src/src/deliver.c
index 2ee69d3..881c977 100644
--- a/src/src/deliver.c
+++ b/src/src/deliver.c
@@ -2988,16 +2988,15 @@ while (!done)
      to get all available data from pipe. unfinished has to be true 
      as well. */
   if (remaining < required)
+    {
     if (unfinished)
       continue;
-    else
-      {
-      msg = string_sprintf("failed to read pipe from transport process "
-        "%d for transport %s: required size=%d > remaining size=%d and unfinished=false", 
-        pid, addr->transport->driver_name, required, remaining);
-      done = TRUE;
-      break;
-      }
+    msg = string_sprintf("failed to read pipe from transport process "
+      "%d for transport %s: required size=%d > remaining size=%d and unfinished=false", 
+      pid, addr->transport->driver_name, required, remaining);
+    done = TRUE;
+    break;
+    }


/* step behind the header */
ptr += PIPE_HEADER_SIZE;