Re: gnus crashes on threads deeper than 333 articles

[Reiner Steib <reinersteib+gmane@imap.cc>]

On Tue, Dec 05 2006, Richard Stallman wrote:

>     > What about providing an option and keep the current recursive as
>     > default?  See the patch below (`gnus-sort-threads-loop' is your
>     > function).
>
>     I think this is a good approach.
>
> Why do we want to keep the recursive version?
> Why add an option?
>
> We know that the loop works, so let's just use that.

Well, the recursion is well tested (part of Gnus at least since 1997)
with only _two_ reported problems with it over all those years
(i.e. the situation when the recursion fails is *very rare*).  The new
function has been tested in practice only by two persons.  I wouldn't
feel comfortable to change such an important function so shortly
before the Emacs release without providing a simple way back in case
of problem with the loop implementation.

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/