Il 30/04/2013 21:00, Jakub Wilk ha scritto:
> * Giulio Paci <[email protected]>, 2013-04-30, 03:39:
>>> In mGramCounts.py and LanguageModel.py I see this code:
>>>              self.fname = tempfile.mkstemp('counts')[1]
>>> This leaks a filedescriptor... (This doesn't affect Debian AFAICT, so I'm 
>>> mentioning it here only if case upstream awakens.)
>> I just fixed, just in case...
> 
> Okay, but let's not make the indentation worse that it already was. :)

Updated 1009 patch (it is now using tab+space like in the remaining of the 
file).
I have not seen the same issue in other patches.


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to