[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