From: Simon Ruderich Date: Sat, 21 Oct 2017 00:19:16 GMT Subject: Re: [PATCH 2/3] t5615: avoid re-using descriptor 4 Message-ID: <20171021001916.bohcqjuxcufxvpam@ruderich.org> In-Reply-To: <20171020224608.3ataitvdq5r33iih@sigill.intra.peff.net> On Fri, Oct 20, 2017 at 06:46:08PM -0400, Jeff King wrote: >> I agree. Maybe just stick with the original patch? > > OK. Why don't we live with that for now, then. The only advantage of the > "999" trickery is that it's less likely to come up again. If it doesn't, > then we're happy. If it does, then we can always switch then. I think switching the 4 to 9 (which you already brought up in this thread) is a good idea. It makes accidental conflicts less likely (it's rare to use so many file descriptors) and is easy to implement. Regards Simon -- + privacy is necessary + using gnupg http://gnupg.org + public key id: 0x92FEFDB7E44C32F9