Re: [Patch] new feature: sort labels

2002-10-16 Thread Angus Leeming
On Wednesday 16 October 2002 12:16 pm, Andre Poenitz wrote: > On Wed, Oct 16, 2002 at 12:15:42PM +0100, Angus Leeming wrote: > > You mean getLabelList -> fillLabelList? It's a nice idea. > > Not sure I like "fill" though. You're appending to it. > > Ok, so 'appendToLabelList'. > > I don't care abo

Re: [Patch] new feature: sort labels

2002-10-16 Thread Andre Poenitz
On Wed, Oct 16, 2002 at 12:15:42PM +0100, Angus Leeming wrote: > You mean getLabelList -> fillLabelList? It's a nice idea. Not > sure I like "fill" though. You're appending to it. Ok, so 'appendToLabelList'. I don't care about the name, I just do not want to see the dummy returns if nothing is

Re: [Patch] new feature: sort labels

2002-10-16 Thread Angus Leeming
On Wednesday 16 October 2002 12:09 pm, Andre Poenitz wrote: > [The second "problem" remains, though...] You mean getLabelList -> fillLabelList? It's a nice idea. Not sure I like "fill" though. You're appending to it. A

Re: [Patch] new feature: sort labels

2002-10-16 Thread Andre Poenitz
On Wed, Oct 16, 2002 at 12:07:33PM +0100, Angus Leeming wrote: > André, where is this destined to be used? If it's destined for > the frontends, then the xforms frontend already has a "sort" > button in the Reference dialog that does just this. This has the > advantage of displaying in sorted o

[Patch] new feature: sort labels - with patch this time

2002-10-16 Thread Andre Poenitz
Index: buffer.C === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/buffer.C,v retrieving revision 1.387 diff -u -p -r1.387 buffer.C --- buffer.C25 Sep 2002 12:19:13 - 1.387 +++ buffer.C16 Oct 2002 08:22:14 - @@

Re: [Patch] new feature: sort labels

2002-10-16 Thread Angus Leeming
On Wednesday 16 October 2002 9:29 am, Andre Poenitz wrote: > The patch attached sorts lables alphanumerically. This is nice > uif you have a hundred of them and stick to the 'sec:*', > 'eq:*' naming convention. > > It's a new feature though. > > Ok to apply anyway? André, where is this destined t

[Patch] new feature: sort labels

2002-10-16 Thread Andre Poenitz
The patch attached sorts lables alphanumerically. This is nice uif you have a hundred of them and stick to the 'sec:*', 'eq:*' naming convention. It's a new feature though. Ok to apply anyway? --- Related question: Would it be ok to change the functions vector getLabelList() const into s