GNU bug report logs - #18360
Modularize recent qsort_r additions

Previous Next

Package: emacs;

Reported by: Paul Eggert <eggert <at> cs.ucla.edu>

Date: Fri, 29 Aug 2014 20:59:01 UTC

Severity: wishlist

Tags: patch

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


Message #20 received at 18360 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: 18360 <at> debbugs.gnu.org
Subject: Re: bug#18360: Modularize recent qsort_r additions
Date: Sat, 30 Aug 2014 19:43:41 +0300
> Date: Sat, 30 Aug 2014 09:39:32 -0700
> From: Paul Eggert <eggert <at> cs.ucla.edu>
> CC: 18360 <at> debbugs.gnu.org
> 
> Eli Zaretskii wrote:
> > That is a different issue, and should not get in our way here.  For
> > the purposes of this bug, can we agree that qsort_r is not needed, and
> > the changes that use it should be reverted to use qsort instead?
> 
> Yes and no.  The previous version did not use qsort, so reverting the 
> changes that use qsort_r will the drop the use of qsort (and qsort_r) 
> entirely.
> 
> One could further *modify* the changes to stop using qsort_r, and to use 
> qsort only.  But what would be the point?  qsort (and qsort_r) are 
> mistakes here; we cannot use them.  Dropping the use of qsort_r would be 
> like wiping the lipstick off a pig.

Then let's use a different sort routine, not qsort.




This bug report was last modified 10 years and 353 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.