Bug#696782: RFS: sequitur-g2p/0.0.r1668-1 [ITP] -- Grapheme to Phoneme conversion tool
- To: 696782@bugs.debian.org
- Subject: Bug#696782: RFS: sequitur-g2p/0.0.r1668-1 [ITP] -- Grapheme to Phoneme conversion tool
- From: Giulio Paci <giuliopaci@gmail.com>
- Date: Wed, 01 May 2013 11:34:27 +0200
- Message-id: <[🔎] 5180E1A3.6010209@gmail.com>
- Reply-to: Giulio Paci <giuliopaci@gmail.com>, 696782@bugs.debian.org
- In-reply-to: <20130430190043.GA4557@jwilk.net>
- References: <5114159A.5010308@gmail.com> <20130216121025.GA4373@jwilk.net> <20130320225350.GA4440@jwilk.net> <515F8A72.1010203@gmail.com> <20130407200029.GA5935@jwilk.net> <5161DE70.9040303@gmail.com> <20130412223135.GA5946@jwilk.net> <516A9154.1040503@gmail.com> <20130424175155.GA1300@jwilk.net> <517F20D1.5010500@gmail.com> <20130430190043.GA4557@jwilk.net>
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: