[rfa] partial fix for PR gdb/1245

Andrew Cagney ac131313@redhat.com
Sun Jun 29 21:54:00 GMT 2003


>  	case ')':
>> +	  if (permissive)
>> +	    {
>> +	      return index;
>> +	    }
> 
> 
> Please drop the useless pair of braces.

BTW, there are advantages and disasvantages to constantly adding and 
deleting extra brances.  If they are included, follow-on diffs such as:

	if (permissive)
	  {
+	    process_index (&index);
	    return index;
	  }

read much easier :-)

It'ss a gnu coding standard loose loose situtation :-)

Andrew




More information about the Gdb-patches mailing list