Re: [DEMO][PATCH v2 6/5] compat: add a qsort_s() implementation based on GNU's qsort_r(1)
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Jan 26, 2017, 13:49 UTC
- Message-ID
- <alpine.DEB.2.20.1701261446540.3469@virtualbox>
- In-Reply-To
- <20170125185153.obqwxiniyz2omxsi@sigill.intra.peff.net>
Hi,
On Wed, 25 Jan 2017, Jeff King wrote:
Show 6 quoted lines
> 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