From: Johannes Schindelin Date: Thu, 26 Jan 2017 13:49:10 GMT Subject: Re: [DEMO][PATCH v2 6/5] compat: add a qsort_s() implementation based on GNU's qsort_r(1) Message-ID: In-Reply-To: <20170125185153.obqwxiniyz2omxsi@sigill.intra.peff.net> Hi, On Wed, 25 Jan 2017, Jeff King wrote: > On Wed, Jan 25, 2017 at 07:43:01PM +0100, René Scharfe wrote: > > > If we find such cases then we'd better fix them for all platforms, e.g. by > > importing timsort, no? > > Yes, as long as they are strict improvements. I think in many cases, we may be better off by replacing the use of a string-list for lookups by hashmaps for the same purpose. Ciao, Dscho