# New Ticket Created by Klaas-Jan Stol # Please include the string: [perl #51894] # in the subject line of all future correspondence about this issue. # <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=51894 >
hi,
attached a patch that does some aligning of "=" tokens and limits the
scope of some variables.
I don't commit this myself, because I want to check whether this
practice of localizing vars. is ok, as it introduces "else" clauses.
For instance, instead of writing:
STRING *s;
if ( bla bla )
return -1;
s = xyz
more stuff()
return s
I changed this into:
if (bla bla)
return -1;
else {
STRING *s = xyz
more stuff()
return s;
}
Not sure whether which is better. My personal preference is the patch,
obviously.
Comments welcome, so I know how to do this in the future.
thanks,
kjs
io.patch
Description: Binary data
