[et-mgmt-tools] [PATCH 02 of 11] Cleanup on failure

Cole Robinson crobinso at redhat.com
Tue Jul 8 15:02:01 UTC 2008


John Levon wrote:
> On Tue, Jul 08, 2008 at 10:44:42AM -0400, Cole Robinson wrote:
> 
>>> +    for dirpath, _, files in os.walk(path):
>> You'll want to use a different variable other than '_' since this will
>> overwrite the gettext function in the local scope. Doesn't cause issues
>> in the current code, but could with future changes.
> 
> Hmm, OK. _ is pretty standard for "don't care", what would you prefer?

No preference: 'ignore', 'unused', anything reasonable and consistent.

- Cole




More information about the et-mgmt-tools mailing list