# New Ticket Created by chromatic # Please include the string: [perl #42960] # in the subject line of all future correspondence about this issue. # <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=42960 >
On Tuesday 15 May 2007 11:22:25 jerry gay wrote: > just below this code in that file, there's an #if 0 block i'm not > responsible for. there vars were used only in that block. i wrapped > them in #if 0 simply to remove the warning, while keeping the intent > of the code the same. > > i agree that #if 0 is bad practice in general. i didn't have tuits at > the time to perform a code review on this block--i just corrected an > oversight by the original coder. i've forwarded this to rt, so > somebody can clean up this code properly. That's very reasonable. I think we've found our FOTW. -- c
