This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Improve readability in gcc.c (execute)


Hi,

I'm not at all understanding what happens in that code, but unless I'm really tired (and in that case, I'm sorry for interfering), your patch is not a noop:

  	    if (WTERMSIG (status) == SIGPIPE
  		&& (signal_count || greatest_status >= MIN_FATAL_STATUS))
! 	      {
! 		signal_count++;
! 		ret_code = -1;
! 	      }
  	    else
  #endif


In case the #if corresponding to the #endif we see here is not true, then "signal_count++; ret_code = -1;" will never be executed, while it was in the original code.


FX


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]