[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#696782: RFS: sequitur-g2p/0.0.r1668-1 [ITP] -- Grapheme to Phoneme conversion tool



Il 30/04/2013 21:00, Jakub Wilk ha scritto:
> * Giulio Paci <giuliopaci@gmail.com>, 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.


Reply to: