You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@apr.apache.org by wr...@apache.org on 2007/09/29 21:56:47 UTC

svn commit: r580632 - /apr/apr/trunk/threadproc/unix/proc.c

Author: wrowe
Date: Sat Sep 29 12:56:47 2007
New Revision: 580632

URL: http://svn.apache.org/viewvc?rev=580632&view=rev
Log:
Thanks for catching the unbalanced parens, jerenkrantz.

These can be reduced further and still remain legible.

Modified:
    apr/apr/trunk/threadproc/unix/proc.c

Modified: apr/apr/trunk/threadproc/unix/proc.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/threadproc/unix/proc.c?rev=580632&r1=580631&r2=580632&view=diff
==============================================================================
--- apr/apr/trunk/threadproc/unix/proc.c (original)
+++ apr/apr/trunk/threadproc/unix/proc.c Sat Sep 29 12:56:47 2007
@@ -574,15 +574,15 @@
     }
 
     /* Parent process */
-    if ((attr->child_in && (attr->child_in->filedes == -1))) {
+    if (attr->child_in && (attr->child_in->filedes == -1)) {
         apr_file_close(attr->child_in);
     }
 
-    if ((attr->child_out) && (attr->child_out->filedes == -1)) {
+    if (attr->child_out && (attr->child_out->filedes == -1)) {
         apr_file_close(attr->child_out);
     }
 
-    if ((attr->child_err) && (attr->child_err->filedes == -1)) {
+    if (attr->child_err && (attr->child_err->filedes == -1)) {
         apr_file_close(attr->child_err);
     }