[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: removing flatten-list (alias for new flatten-tree function)
From: |
Alex Branham |
Subject: |
Re: removing flatten-list (alias for new flatten-tree function) |
Date: |
Mon, 17 Dec 2018 13:00:31 -0600 |
User-agent: |
mu4e 1.1.0; emacs 27.0.50 |
On Mon 17 Dec 2018 at 12:40, Paul Eggert <address@hidden> wrote:
> Thanks for writing the new flatten-tree function and cleaning up
> existing duplicate functions. I tweaked its performance a bit and
> cleaned up some more duplicates.
Thanks!
> While doing that I found that I was confused by the existence of the
> just-added alias 'flatten-list', and I propose removing it as per the
> attached patch. I think people who want to find and use the new
> 'flatten-tree' function will typically be better served if it has just
> one name that fits in with existing Emacs Lisp functions like
> 'copy-tree', as opposed to also having a name that is a misnomer.
The reason I included `flatten-list' was that the existing
implementations (both inside and outside of Emacs) were nearly all
called foo-flatten-list or similar. I thought it would perhaps be easier
for people to find `flatten-list' than `flatten-tree'. I don't have a
really strong opinion about this, though.
Alex