Re: BranchList class for comment

2003-07-15 Thread Martin Vermeer
On Wed, Jul 16, 2003 at 02:02:54AM +0200, Lars Gullik Bjønnes spake thusly: > > Martin Vermeer <[EMAIL PROTECTED]> writes: > > | Attached a screenshot from the branch list editor I am building, with > | a separate branches tab. Lars, was this about what you had in mind? > | > | Not yet working,

Re: Patch [Re: BranchList class for comment]

2003-07-15 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 10:12:26AM +0300, Martin Vermeer wrote: > I *had* a vector there... but the remove_if seemed to require a list > (is that correct? gcc 2.95.2.) Grr. Yes. list::remove_if uses a list, the 'global' one not. Sorry for confusing you. > > Likewise the global helper 'class match

Re: Patch [Re: BranchList class for comment]

2003-07-15 Thread Martin Vermeer
On Wed, Jul 16, 2003 at 08:15:53AM +0200, Andre Poenitz spake thusly: > On Wed, Jul 16, 2003 at 02:04:11AM +0200, Lars Gullik Bjxnnes wrote: > > Martin Vermeer <[EMAIL PROTECTED]> writes: > > > > | As I will be off to the country again tomorrow, someone else may check > > | this in if it can go

Re: Patch [Re: BranchList class for comment]

2003-07-15 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 02:04:11AM +0200, Lars Gullik Bjønnes wrote: > Martin Vermeer <[EMAIL PROTECTED]> writes: > > | As I will be off to the country again tomorrow, someone else may check > | this in if it can go like this. Otherwise until next week :-) > > I'd like next week then. Just so th

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Wed, Jul 16, 2003 at 01:58:41AM +0200, Lars Gullik Bjønnes wrote: > I am actually less concerned if it is your work that has created this, > IMHO it would be worse if it was the rowlist, parlist or the signals > work. It looks like my stuff afterall. > But of course if you can find out _why_ (

Re: BranchList class for comment

2003-07-15 Thread Juergen Spitzmueller
Lars Gullik Bjønnes wrote: > | Not yet working, but getting there :-) > > yes, getting there... > > I wonder ... could this be one of those dialogs that could benefit > from a > > --- --- > | | | | > | | add-> | | > | | <-remove|

Re: small clean-up proposal

2003-07-15 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Lars, you have posted the following warning in | controllers/ControlMath.C: | | string const find_xpm(string const & name) | { | #warning Use a static table for this (Lgb) | // And get O(log n) lookup (Lgb) | | string xpm_name = subst(

Re: BranchList class for comment

2003-07-15 Thread John Levon
On Wed, Jul 16, 2003 at 02:02:54AM +0200, Lars Gullik Bj?nnes wrote: > I wonder ... could this be one of those dialogs that could benefit > from a > > --- --- > | | | | > | | add-> | | > | | <-remove| | > | |

Re: Boost warnings

2003-07-15 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Compiling 1.3.x I get these warnings when compiling the | boost regex library. (gcc 3.3) and if you look further you will find some in the math_parser.C as well... | Lars, are the boost developers interested in such things? | If so could you pass the

Re: Patch [Re: BranchList class for comment]

2003-07-15 Thread Lars Gullik Bjønnes
Martin Vermeer <[EMAIL PROTECTED]> writes: | As I will be off to the country again tomorrow, someone else may check | this in if it can go like this. Otherwise until next week :-) I'd like next week then. Just so that we can decide a bit on further refinements and requirements before we add the

Re: BranchList class for comment

2003-07-15 Thread Lars Gullik Bjønnes
Martin Vermeer <[EMAIL PROTECTED]> writes: | Attached a screenshot from the branch list editor I am building, with | a separate branches tab. Lars, was this about what you had in mind? | | Not yet working, but getting there :-) yes, getting there... I wonder ... could this be one of those dial

Re: [patch] more drawing

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Tue, Jul 15, 2003 at 05:12:43PM +0200, Lars Gullik Bjønnes wrote: | > | but to be honest, I think this is a | > | waste of time. | > | > That depends on where the error comes from. | | Between 5 and 6 days ago. diff attached. | Mostly Alfredos signa

Patch [Re: BranchList class for comment]

2003-07-15 Thread Martin Vermeer
On Tue, Jul 15, 2003 at 08:05:15PM +0300, Martin Vermeer spake thusly: > > Attached a screenshot from the branch list editor I am building, with > a separate branches tab. Lars, was this about what you had in mind? > > Not yet working, but getting there :-) > > Martin Here is the 'branch' code

Re: [patch] more drawing

2003-07-15 Thread Alfredo Braunstein
Andre Poenitz wrote: > I just noticed that this kind of cursor movement has already been > present yesterday, so the current regression is performance only. I've added a bug to bugzilla yersterday, mostly the same but with LFUN_LEFTSEL inside table cells. LFUN_RIGHTSEL works ok. Doesn't this mean

Re: [PATCH] fix bug 1156

2003-07-15 Thread Juergen Spitzmueller
John Levon wrote: > > OK to apply? > > Looks good to me Applied. Juergen.

Re: why cut->paste text from other documents = blue underlined?

2003-07-15 Thread Garst R. Reese
John Levon wrote: > Annoying the user is never a good solution. > > I really think we have it right as is. > The only thing I would even think of changing is making the color change a document item rather than a global preference, but default the same as is. Garst

Re: BranchList class for comment

2003-07-15 Thread Martin Vermeer
Attached a screenshot from the branch list editor I am building, with a separate branches tab. Lars, was this about what you had in mind? Not yet working, but getting there :-) Martin <> pgp0.pgp Description: PGP signature

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 05:47:08PM +0200, Andre' Poenitz wrote: > On Tue, Jul 15, 2003 at 05:12:43PM +0200, Lars Gullik Bjønnes wrote: > > | but to be honest, I think this is a > > | waste of time. > > > > That depends on where the error comes from. > > Between 5 and 6 days ago. diff attached. >

small clean-up proposal

2003-07-15 Thread Angus Leeming
Lars, you have posted the following warning in controllers/ControlMath.C: string const find_xpm(string const & name) { #warning Use a static table for this (Lgb) // And get O(log n) lookup (Lgb) string xpm_name = subst(name, ' ', '_'); if (xpm_name == "(") xpm_name = "lpa

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 05:12:43PM +0200, Lars Gullik Bjønnes wrote: > | but to be honest, I think this is a > | waste of time. > > That depends on where the error comes from. Between 5 and 6 days ago. diff attached. Mostly Alfredos signals and my multiple LyXTexts per InsetText stuff. Maybe tim

Re: [patch] more drawing

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 03:01:23PM +0200, Andre Poenitz wrote: > Looks like it's time for a decision. Ignore that or try to fix tabular > "first"? As expected there are lots of mysterious regressions currently. So you may as well carry on with the extended work of removing update() IMHO. regards

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 04:26:12PM +0100, John Levon wrote: > On Tue, Jul 15, 2003 at 01:44:53PM +0200, Andre Poenitz wrote: > > > I think so. > > There's a tex2lyx component assigned to you now Ok. Andre' -- Those who desire to give up Freedom in order to gain Security, will not have, nor do

Re: Warning messages about failed inlines in 1.3.x

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 03:39:00PM +0100, Angus Leeming wrote: > I have just trawled through the 700+ messages on the devel list > posted since I last looked in this direction. It seems that there Whoo hoo ! Welcome back ;) john

Re: [PATCH] fix bug 1156

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 04:01:58PM +0200, Juergen Spitzmueller wrote: > (qtabular width input) > http://bugzilla.lyx.org/show_bug.cgi?id=1156 > > OK to apply? Looks good to me regards john

Re: Re: Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 01:41:35PM +0200, Andre Poenitz wrote: > For my part, I won't type long sermons in an web > frontend as long as this is not as comfortable as > email or such. Then use the email interface ... john

Boost warnings

2003-07-15 Thread Angus Leeming
Compiling 1.3.x I get these warnings when compiling the boost regex library. (gcc 3.3) Lars, are the boost developers interested in such things? If so could you pass them on? File ../../../../../boost/libs/regex/src/c_regex_traits.cpp ../../../../../boost/libs/regex/src/cpp_regex_traits.cpp: In

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 01:44:53PM +0200, Andre Poenitz wrote: > I think so. There's a tex2lyx component assigned to you now john

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 05:12:43PM +0200, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | On Tue, Jul 15, 2003 at 04:53:07PM +0200, Lars Gullik Bjønnes wrote: > | > I'd really like to know when the regression was introduced first. > | > Just to pause until we find out.

Re: [patch] redraw more...

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 11:43:58AM +0200, Andre Poenitz wrote: > [The only problem that might have cropped up would be speed but I simply > don't see it. So this is either premature optimization or some > optimization that's no more relevant on a PIII/667] Probably a selection of other changes ov

Re: why cut->paste text from other documents = blue underlined?

2003-07-15 Thread John Levon
On Tue, Jul 15, 2003 at 04:08:12PM +0300, Dekel Tsur wrote: > How about allowing the user to choose in the preferences dialog whether the > language should be changed on paste (but only between dialects of the same > language) ? Bleh. Remember the mantra: a pref means that we've run out of design

Re: [patch] more drawing

2003-07-15 Thread Garst R. Reese
Lars Gullik Bjønnes wrote: > Depends... if we understand why the regressions is there or not... > My experience with marking tabular has been a bit weird. Sometimes it is very slow, others not, and I can't determine what I may be doing different! It would look like a regression -- then I would r

Re: [patch] more drawing

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Tue, Jul 15, 2003 at 04:53:07PM +0200, Lars Gullik Bjønnes wrote: | > I'd really like to know when the regression was introduced first. | > Just to pause until we find out. | | I am building now '1 week ago', I am doing 2003-07-01 and 2003-06-01 |

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 04:53:07PM +0200, Lars Gullik Bjønnes wrote: > I'd really like to know when the regression was introduced first. > Just to pause until we find out. I am building now '1 week ago', but to be honest, I think this is a waste of time. Andre' -- Those who desire to give up Fr

Re: Warning messages about failed inlines in 1.3.x

2003-07-15 Thread Angus Leeming
Lars Gullik Bjønnes wrote: > Angus Leeming <[EMAIL PROTECTED]> writes: > > | Hello all, > | > | $ gcc --version > | gcc (GCC) 3.3.1 20030626 (Debian prerelease) > | > | I find many warning messages like > > Many? That was an understatement... thousands upon thousands you > mean... (at least wi

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 04:53:07PM +0200, Lars Gullik Bjønnes wrote: > I'd really like to know when the regression was introduced first. > > Just to pause until we find out. Hm... ok. > We can divede the binary search between us to make it faster, just > tell me the dates to check for (or the in

Re: [patch] more drawing

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | So given that this is an "old" bug, ok to proceed with the | redraw/update changes? I'd really like to know when the regression was introduced first. Just to pause until we find out. We can divede the binary search between us to make it faster, just t

Re: Warning messages about failed inlines in 1.3.x

2003-07-15 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Hello all, | | $ gcc --version | gcc (GCC) 3.3.1 20030626 (Debian prerelease) | | I find many warning messages like Many? That was an understatement... thousands upon thousands you mean... (at least with gcc 3.4) | ../../../../../boost/libs/regex/sr

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 04:34:24PM +0200, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | On Tue, Jul 15, 2003 at 03:01:23PM +0200, Andre' Poenitz wrote: > | > > | > This moves closer towards full redraw but there are regressions: > | > Selection in tables gets slow a

Warning messages about failed inlines in 1.3.x

2003-07-15 Thread Angus Leeming
Hello all, I have just trawled through the 700+ messages on the devel list posted since I last looked in this direction. It seems that there have been some impassioned discussions as well as the more standard fare. It all makes intersting reading ;-) More prosaically, I have just tried to comp

Re: [patch] more drawing

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Tue, Jul 15, 2003 at 03:01:23PM +0200, Andre' Poenitz wrote: | > | > This moves closer towards full redraw but there are regressions: | > Selection in tables gets slow and when selecting bottom up, the | > selection anchor "moves" in parallel to the

Re: [patch] more drawing

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 03:01:23PM +0200, Andre' Poenitz wrote: > > This moves closer towards full redraw but there are regressions: > Selection in tables gets slow and when selecting bottom up, the > selection anchor "moves" in parallel to the cursor. I just noticed that this kind of cursor move

[PATCH] fix bug 1156

2003-07-15 Thread Juergen Spitzmueller
(qtabular width input) http://bugzilla.lyx.org/show_bug.cgi?id=1156 OK to apply? Juergen. Index: src/frontends/qt2/QTabular.C === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/frontends/qt2/QTabular.C,v retrieving revision 1.22 diff

Re: why cut->paste text from other documents = blue underlined?

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 04:08:12PM +0300, Dekel Tsur wrote: > How about allowing the user to choose in the preferences dialog whether the > language should be changed on paste (but only between dialects of the same > language) ? Against. We can not have a preference for everything everytime we ca

Re: why cut->paste text from other documents = blue underlined?

2003-07-15 Thread Dekel Tsur
On Tue, Jul 15, 2003 at 12:39:48AM +0100, John Levon wrote: > > >, it is desired that LyX should automatically change the language of > > the copied text to American, and then the user should fix the spelling. > > And what if it's *not* intended to be a single-language document ? You > just broke

[patch] more drawing

2003-07-15 Thread Andre Poenitz
This moves closer towards full redraw but there are regressions: Selection in tables gets slow and when selecting bottom up, the selection anchor "moves" in parallel to the cursor. Nested minipages, however, work just fine "at full speed", so this does not look like a problem of 'full redraw' and

Re: BranchList class for comment

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 03:48:41PM +0300, Martin Vermeer wrote: > using std::vector; > using std::bind2nd; > using std::remove_if; You want list::remove_if, not the general one. > void BranchList::remove(string const & s) > { > remove_if(list.begin(), list.end(), bind2nd(pfn(), s)); > } v

meeting

2003-07-15 Thread Andre Poenitz
So who's actually coming? So far I expect Alfredo(2), Angus, Asger, Jean-Marc, Jose' and Lars. Not coming are Juergen and John(?). Asger (Wed-Sun) and Jean-Marc (Thu-Tue) already sent details concerning flights/trains. Anything wrong with that lists? [I'd like send out address etc by private

Re: BranchList class for comment

2003-07-15 Thread Martin Vermeer
On Tue, Jul 15, 2003 at 09:18:47AM +0200, Andre Poenitz spake thusly: ... > Theoretically, I'd expect this to be undefined behaviour as 'it' is > invalid after its 'pointee' is erased. > > Practically, there is a std::list<>::remove_if(Pred) somewhere. > > Andre' Thanks! Ok, see attached, whi

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Lars Gullik Bjønnes
<[EMAIL PROTECTED]> writes: | Andre Poenitz <[EMAIL PROTECTED]> schrieb am 15.07.2003, 13:20:37: | > On Tue, Jul 15, 2003 at 01:12:01PM +0200, [EMAIL PROTECTED] wrote: | > > PS: If you do not want me to fill bugzilla with | > > tex2lyx reports, please shout loudly. | > | > No, that's fine. My 'in

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:41:56PM +0200, Lars Gullik Bjønnes wrote: > | [But certainly less often than on lyx-devel...] > > So, perhaps even if you do not check that often, tex2lyx bugs should > be default assigned to you? I think so. Unless somebody thinks that Jean-Marc is more likely to fix

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Tue, Jul 15, 2003 at 01:13:41PM +0200, Lars Gullik Bjønnes wrote: | > | Now why are tex2lyx bugs assigned to Jean-Marc? | > | > he is default... and you have paid no interest in bugzilla. | | Well, I look from time to time | | [But certainly le

Re: Re: Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:26:02PM +0200, [EMAIL PROTECTED] wrote: > > On Tue, Jul 15, 2003 at 01:12:01PM +0200, [EMAIL PROTECTED] wrote: > > > PS: If you do not want me to fill bugzilla with > > > tex2lyx reports, please shout loudly. > > > > No, that's fine. My 'inbox based bugzilla substitute'

Re: Re: Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread michael.schmitt
Andre Poenitz <[EMAIL PROTECTED]> schrieb am 15.07.2003, 13:20:37: > On Tue, Jul 15, 2003 at 01:12:01PM +0200, [EMAIL PROTECTED] wrote: > > PS: If you do not want me to fill bugzilla with > > tex2lyx reports, please shout loudly. > > No, that's fine. My 'inbox based bugzilla substitute' is far le

[patch] remove unneeded refresh_row

2003-07-15 Thread Andre Poenitz
As LyXText::refresh_row was only used by REFRESH_ROW (now merged into REFRESH_AREA), the variable and the surrounding logig can go... Andre' -- Those who desire to give up Freedom in order to gain Security, will not have, nor do they deserve, either one. (T. Jefferson or B. Franklin or both

Re: Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:12:01PM +0200, [EMAIL PROTECTED] wrote: > PS: If you do not want me to fill bugzilla with > tex2lyx reports, please shout loudly. No, that's fine. My 'inbox based bugzilla substitute' is far less reliable (but more convenient...) > I am not looking for tex2lyx bugs dire

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:13:41PM +0200, Lars Gullik Bjønnes wrote: > | Now why are tex2lyx bugs assigned to Jean-Marc? > > he is default... and you have paid no interest in bugzilla. Well, I look from time to time [But certainly less often than on lyx-devel...] Andre' -- Those who desir

Re: Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread michael.schmitt
Andre Poenitz <[EMAIL PROTECTED]> schrieb am 15.07.2003, 13:06:27: > Now why are tex2lyx bugs assigned to Jean-Marc? Because there is no component "tex2lyx" right now. I used "general" for the classification of the bug reports. I informed John about the problem already. Michael PS: If you do

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Tue, Jul 15, 2003 at 01:02:01PM +0200, [EMAIL PROTECTED] wrote: | > | > Weitergeleitete Nachricht von [EMAIL PROTECTED] vom 15.07.2003, | > 12:58:07: | > > http://bugzilla.lyx.org/show_bug.cgi?id=1261 | > > | > >Summary: tex2lyx adds spa

Re: [patch] refresh_status tristate -> bool

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:07:07PM +0200, Lars Gullik Bjønnes wrote: > | void LyXText::clearPaint() > | { > | - refresh_status_ = REFRESH_NONE; > | + need_refresh_ = true; > | refresh_row = rows().end(); > | refresh_y = 0; > | } > > Is this correct? No. > Shouldn't it be false? Y

Re: [patch] refresh_status tristate -> bool

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | Index: text2.C | === | RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/text2.C,v | retrieving revision 1.384 | diff -u -p -r1.384 text2.C | --- text2.C 15 Jul 2003 08:26:16 - 1.384

Re: Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 01:02:01PM +0200, [EMAIL PROTECTED] wrote: > > Weitergeleitete Nachricht von [EMAIL PROTECTED] vom 15.07.2003, > 12:58:07: > > http://bugzilla.lyx.org/show_bug.cgi?id=1261 > > > >Summary: tex2lyx adds space around itemize items > >Product: LyX > >

Fw: [Bug 1261] New: tex2lyx adds space around itemize items

2003-07-15 Thread michael.schmitt
Weitergeleitete Nachricht von [EMAIL PROTECTED] vom 15.07.2003, 12:58:07: > http://bugzilla.lyx.org/show_bug.cgi?id=1261 > >Summary: tex2lyx adds space around itemize items >Product: LyX >Version: 1.4.0cvs > Platform: All > OS/Version: Linux >

[patch] refresh_status tristate -> bool

2003-07-15 Thread Andre Poenitz
This replaces the refresh_status tristate by a 'bool need_refresh_' [Which will go in the end entirely...] Andre' -- Those who desire to give up Freedom in order to gain Security, will not have, nor do they deserve, either one. (T. Jefferson or B. Franklin or both...) Index: lyxtext.h

Re: [patch] redraw more...

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 11:38:54AM +0200, Braunstein Alfredo wrote: > Andre Poenitz wrote: > > > The attached patch collapses 'REFRESH_AREA' and 'REFRESH_ROW' handling. > > Goal is still simple two-phase drawing with 'redraw everything'. > > > > I don't see any noticable sluggishness when jumping

Re: [patch] redraw more...

2003-07-15 Thread Braunstein Alfredo
Andre Poenitz wrote: > The attached patch collapses 'REFRESH_AREA' and 'REFRESH_ROW' handling. > Goal is still simple two-phase drawing with 'redraw everything'. > > I don't see any noticable sluggishness when jumping around in the > UserGuide (was to be expected I think) > > Incidentally, this

[patch] redraw more...

2003-07-15 Thread Andre Poenitz
The attached patch collapses 'REFRESH_AREA' and 'REFRESH_ROW' handling. Goal is still simple two-phase drawing with 'redraw everything'. I don't see any noticable sluggishness when jumping around in the UserGuide (was to be expected I think) Incidentally, this fixes the redraw bug when clicking

Re: BranchList class for comment

2003-07-15 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Mon, Jul 14, 2003 at 10:05:51PM +0300, Martin Vermeer wrote: | > void BranchList::remove(string const & s) | > { | > List::iterator it = list.begin(); | > List::iterator end = list.end(); | > for (; it != end; it++) { | > if (i

Re: why cut->paste text from other documents = blue underlined?

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 12:39:48AM +0100, John Levon wrote: > I do not see a way we can change language behind the user's back in a > reliable manner, and I don't think we should. I agree with John. A Russian word does not magically convert to Chinese just because it is pasted in a Chinese documen

Re: Assert

2003-07-15 Thread Andre Poenitz
On Tue, Jul 15, 2003 at 07:58:53AM +0200, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | I think there are a few places in InsetText that are never reached. > | Should I put in a lyxerr<< or an Assert or ...? > | > | > | I.e. > > I think Assert. > > perhaps we sh

Re: BranchList class for comment

2003-07-15 Thread Andre Poenitz
On Mon, Jul 14, 2003 at 10:05:51PM +0300, Martin Vermeer wrote: > void BranchList::remove(string const & s) > { > List::iterator it = list.begin(); > List::iterator end = list.end(); > for (; it != end; it++) { > if (it->getBranch() == s) list.erase(it); > } >

Re: another resize problem

2003-07-15 Thread Andre Poenitz
On Mon, Jul 14, 2003 at 09:01:52PM +0200, Alfredo Braunstein wrote: > Effectively, it seems there's no need for a difference. The attached patch > seems to work fine (will test it a little more though). Good boy ;-) Just commit if you are finished with testing. Andre', wondering whether the sky

Re: patches

2003-07-15 Thread Andre Poenitz
On Mon, Jul 14, 2003 at 08:39:28PM +0200, Alfredo Braunstein wrote: > Juergen Spitzmueller wrote: > > > This would be it. Works very well in text3.C (normal text) and insettext > > (tested inside footnotes, minipages). > > OK to apply? I think so. Andre' -- Those who desire to give up Freedom

Re: [PATCH] Branch/Note "final"

2003-07-15 Thread Andre Poenitz
On Mon, Jul 14, 2003 at 09:52:12PM +0300, Martin Vermeer wrote: > int i = 0; > if (pt == "Comment") os << "%\n\\begin{comment}\n"; // remember to validate > - if (pt == "Greyedout") os << "%\n\\textcolor[gray]{0.8}{"; > + if (pt == "Greyedout") os << "%\n\\color[gray]{0.8}"; Be