Re: [Patch] gtk Include dialog

2004-12-06 Thread John Spray
On Sun, 2004-12-05 at 15:43 +, Angus Leeming wrote: > > In GInclude::onOpenClicked, in order to apply changes ready for loading > > the document (which closes the dialog), I do > > + apply(); > > + controller().dispatchParams(); > > Yuck! Well, damned if I know what's going on. It seems to

Re: autogen.sh and FC3

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Angus> Note also that running ./autogen.sh on the 1.3.x tree produces > Angus> additional warnings > Angus> acinclude.m4:9: warning: underquoted definition of > Angus> LYX_CHECK_VERSION > > Does the following patch cure the warnings? It does indeed, although I note

[PATCH 13x]: pragmas stripped

2004-12-06 Thread Angus Leeming
JMarc, I went ahead and applied my script to the 13x tree. It actually took longer to check the resulting patch (attached) than it did to create it ;-) Regards, -- Angus pragma.diff.bz2 Description: BZip2 compressed data

Re: Hyphenation mess :-(

2004-12-06 Thread Georg Baum
Am Montag, 6. Dezember 2004 18:12 schrieb Jean-Marc Lasgouttes: > There are a number of regexps for that, but in this case, it is the > opening parenthesis that triggers, I guess. > > I would do as follows: > > if I find a line with opening parenthesis so that what follows does > not form an exi

Re: [patch] cursor

2004-12-06 Thread Angus Leeming
Alfredo Braunstein wrote: > Is this ok [self-explaining patch attached], or are there some kind of > moral reasons for not showing half a cursor? Looks sensible to me. -- Angus

[patch] cursor

2004-12-06 Thread Alfredo Braunstein
Is this ok [self-explaining patch attached], or are there some kind of moral reasons for not showing half a cursor? Alfredo Index: screen.C === RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/frontends/screen.C,v retrieving revision 1.

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Angus Leeming
Andre Poenitz wrote: > On Mon, Dec 06, 2004 at 12:27:40PM +, Angus Leeming wrote: >> What functionality do we need of bash that the POSIX-standard Bourne >> shell does not provide? I suspect that we need nothing that we can't get >> using perfectly standard means. > > Certainly. However, ther

Re: [patch] undo

2004-12-06 Thread Alfredo Braunstein
Andre Poenitz wrote: >> Cool! now we have a developer with even a development machine! That seems >> an explosive combination ;-) > > Not excatly. I stopped the checkout after 1:15h over the 56k line. If I burn a cdrom with an updated tree and ship it from South America, would it arrive sooner?

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 12:27:40PM +, Angus Leeming wrote: > What functionality do we need of bash that the POSIX-standard Bourne shell > does not provide? I suspect that we need nothing that we can't get using > perfectly standard means. Certainly. However, there are some cases where the so

Re: [patch] undo

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 02:47:19PM +0100, Alfredo Braunstein wrote: > Andre Poenitz wrote: > > > On Sun, Dec 05, 2004 at 08:27:00PM +0100, Alfredo Braunstein wrote: > >> This patch solves two crashes with undo, and simplifies a bit undo.cursor > >> handling in textUndoOrRedo. > > > > Whohey. And

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 02:11:55PM +0100, Lars Gullik Bjønnes wrote: > Andre Poenitz <[EMAIL PROTECTED]> writes: > > | Could you please add that to the coding style rules. IIRC the general > | rule is to avoid unnecessary braces in general but I agree that this is > | a case where the unnecessary

Re: Hyphenation mess :-(

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Angus> More importantly, tell me how I should recognize > Angus> .../preamble.tex is part of the file name. Ie, what > Angus> characteristic indicates that it is a file name? Is it the > Angus> blank line that follows or do I need to recognize the > Angus> characterist

Re: Hyphenation mess :-(

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Can you tell me where I should look in the LyX source to find Angus> the code that parses the log file for these dependencies. LaTeX::deplog in LaTeX.C Angus> More importantly, tell me how I should recognize Angus> .../preamble.te

Re: Hyphenation mess :-(

2004-12-06 Thread Georg Baum
Angus Leeming wrote: > Can you tell me where I should look in the LyX source to find the code > that parses the log file for these dependencies. I believe it is in src/LaTeX.C, in LaTeX::scanLogFile(). Another place that suffers from linebreaks is the extraction of error messages. > More importa

Re: Renaming README.Win32

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Andre> On Tue, Nov 30, 2004 at 10:29:34AM +0100, Jean-Marc Lasgouttes > Andre> wrote: >>> I would be willing to remove them if I had a good explanation of >>> the trade off involved. I understand they got removed from >>> 1.4.0cvs, but then they used to be a gain when

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 10:08:08AM +0100, Lars Gullik Bjønnes wrote: > | Is it possible to install bash on all systems we support? > > > | If so, wouldn't it be simpler to require a working 'bash' > | and drop all these 'portability hacks'? > > Depends... remember that even on Linux not everyone u

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 09:19:52AM +, Angus Leeming wrote: > Such a requirement will just lead to fewer people on these platforms > overcoming the hurdle of using the thing. I thought we were interested in > users. At least a little? We certainly are interested in users. But users are surprisi

Re: Hyphenation mess :-(

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: >> "Angus" == Angus Leeming <[EMAIL PROTECTED]> >> writes: > >>> Clever. Actually, I think we should do something like that when >>> parsing .log files, but I am not sure how. > > Angus> Can you give an example? > > This bug, for example: > http://bugzilla.l

Re: Assertion trigged while editing table

2004-12-06 Thread Alfredo Braunstein
Andreas Vox wrote: > Hi! > > The following is reproducible for me: create a table and start typing > rapidly > some chars and intersperse some LEFT keys to change the cell. > After approx. 20 cells LyX crashes (console output below) It this still there? I cannot reproduce with current cvs. If it

Re: Hyphenation mess :-(

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: >> Clever. Actually, I think we should do something like that when >> parsing .log files, but I am not sure how. Angus> Can you give an example? This bug, for example: http://bugzilla.lyx.org/show_bug.cgi?id=1027 JMarc

Re: [LyX/Mac] Setting PATH in property file.

2004-12-06 Thread Bennett Helm
On Nov 16, 2004, at 1:21 PM, Ronald Florence wrote: Jean-Marc Lasgouttes wrote: It seems that it is possible to set environment variables (like PATH maybe) of an application in its info.plist, using the LSElement key, as described here: http://developer.apple.com/documentation/MacOSX/Conceptual/ B

Re: Renaming README.Win32

2004-12-06 Thread Jean-Marc Lasgouttes
> "Andre" == Andre Poenitz <[EMAIL PROTECTED]> writes: Andre> On Tue, Nov 30, 2004 at 10:29:34AM +0100, Jean-Marc Lasgouttes Andre> wrote: >> I would be willing to remove them if I had a good explanation of >> the trade off involved. I understand they got removed from >> 1.4.0cvs, but then the

Re: Hyphenation mess :-(

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: >> "Angus" == Angus Leeming <[EMAIL PROTECTED]> >> writes: > > Angus> No, but that's why we use a loop in the sed script. The list of > Angus> languages always ends with 'loaded.', so get the next line if > Angus> the pattern space doesn't, remove the embedded

Re: Hyphenation mess :-(

2004-12-06 Thread Jean-Marc Lasgouttes
> "Martin" == Martin Vermeer <[EMAIL PROTECTED]> writes: Martin> Works as such :-) Martin> see attached. Martin> What next? Feel free to apply it. I asked about it on fr.comp.text.tex and it appears that there is no way to get this list from TeX... JMarc

Re: tex2lyx does not handle latex accents

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Jean-Marc Lasgouttes wrote: >> Latex accents like \'e are translated to InsetLatexAccent by >> tex2lyx. This is unfortunate, since some converters produce such >> latex accents (from swriter, I believe). Angus> Alternatively, go th

Re: Hyphenation mess :-(

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> No, but that's why we use a loop in the sed script. The list of Angus> languages always ends with 'loaded.', so get the next line if Angus> the pattern space doesn't, remove the embedded new line and go Angus> to the 'loop' label. H

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> But I didn't object to it. I was barely trying to point out that Lars> might not fix the problem it set out to fix (completely). Fair enough :) JMarc

Re: autogen.sh and FC3

2004-12-06 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | Lars> Am I objecting? I didn't notice. > | Well, you said that patch was basically useless before of that didn't | you? Yes :-) But I didn't object to it. I was barely trying to point out that might not fix the problem it set out to fix (complet

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: >>> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: >> Lars> | Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> Lars> writes: | | Yes, but I assume that fc3 provides t

Re: autogen.sh and FC3

2004-12-06 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: >> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: > | Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | | Yes, but I assume that fc3 provides the fixed m4 files already... > | Lars> For most packages, but apparently not for all

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Jean-Marc Lasgouttes wrote: >> Lars> Like this perhaps: >> >> + int foo = (forba() == hans ? some1() : some2()); >> + -NOT- >> + int foo = forba() == hans ? some1() : some2(); >> >> I would think that >> + int foo = (forba() == hans) ?

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Lars> Like this perhaps: > > + int foo = (forba() == hans ? some1() : some2()); > + -NOT- > + int foo = forba() == hans ? some1() : some2(); > > I would think that > + int foo = (forba() == hans) ? some1() : some2(); > is clearer in this case. Is it j

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Like this perhaps: + int foo = (forba() == hans ? some1() : some2()); + -NOT- + int foo = forba() == hans ? some1() : some2(); I would think that + int foo = (forba() == hans) ? some1() : some2(); is cl

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: | Yes, but I assume that fc3 provides the fixed m4 files already... Lars> For most packages, but apparently not for all. But we don't use ogg/ole/xmms, do we? I have to admit I

Re: [patch] undo

2004-12-06 Thread Alfredo Braunstein
Andre Poenitz wrote: > On Sun, Dec 05, 2004 at 08:27:00PM +0100, Alfredo Braunstein wrote: >> This patch solves two crashes with undo, and simplifies a bit undo.cursor >> handling in textUndoOrRedo. > > Whohey. And I just started a fresh checkout on my shining PII/333... > But now that xforms wor

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | Could you please add that to the coding style rules. IIRC the general | rule is to avoid unnecessary braces in general but I agree that this is | a case where the unnecessary braces might even be preferable. Like this perhaps: ? rules-1.diff Index: Cod

Re: [patch] selection_possible

2004-12-06 Thread Alfredo Braunstein
Alfredo Braunstein wrote: > Alfredo Braunstein wrote: > [...] > > And this is the actual patch ;-) > > Alfredo As nobody complained, I've commited this (please test that selection goes as usual, specially with gtk and Acqua) Alfredo

Re: autogen.sh and FC3

2004-12-06 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: >> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: > | Lars> | What kind of system m4s will have warnings? > | Lars> all kinds of stuff... ogg/ole/xmms you name it. > | Lars> If the strings has not been fixed (i.e. quoted), then you ge

Re: Lyx suggestion

2004-12-06 Thread Jean-Marc Lasgouttes
> "Alfredo" == Alfredo Braunstein <[EMAIL PROTECTED]> writes: Alfredo> Jean-Marc Lasgouttes wrote: Cyrille> Hi all, it would be nice to have the ability to tile the text Cyrille> window so several documents can be viewed at once. This would Cyrille> be particularly useful for merging text from

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> | What kind of system m4s will have warnings? Lars> all kinds of stuff... ogg/ole/xmms you name it. Lars> If the strings has not been fixed (i.e. quoted), then you get Lars> warnings. Yes, but I assume that fc3 provides the f

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Alfredo Braunstein
Lars Gullik BjÃnnes wrote: > Alfredo Braunstein <[EMAIL PROTECTED]> writes: > > | + int max_depth = beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0; > > Please add some parenthesis: int m = (foo ? dfd : fdf); > Even if not needed it is a tiny bit clearer. > > | + int max_depth = beg != 0 ?

Re: Lyx suggestion

2004-12-06 Thread Alfredo Braunstein
Jean-Marc Lasgouttes wrote: > Cyrille> Hi all, it would be nice to have the ability to tile the text > Cyrille> window so several documents can be viewed at once. This would > Cyrille> be particularly useful for merging text from two documents. > > This is something developers have in mind, but t

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Lars Gullik Bjønnes
Angus Leeming <[EMAIL PROTECTED]> writes: | Helge Hafting wrote: > >>>I am ambivalent. Since _I_ use bash all the time: no problem for me. >>>But there are a lot of systems where bash is not the system/preffered >>>shell. >>> >>> >> Preference is not a problem.   It  is  a  problem  only  if  bash

Re: autogen.sh and FC3

2004-12-06 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: >> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: > | Lars> Sure you fix some warnings, _but_ you will almost certainly | Lars> still have warnings from system m4s. So I am not sure this is | Lars> worth the effort. > | Lars> (on FC3

Re: [patch] undo

2004-12-06 Thread Andre Poenitz
On Sun, Dec 05, 2004 at 08:27:00PM +0100, Alfredo Braunstein wrote: > This patch solves two crashes with undo, and simplifies a bit undo.cursor > handling in textUndoOrRedo. Whohey. And I just started a fresh checkout on my shining PII/333... But now that xforms works again it might even fall into

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Andre Poenitz
On Mon, Dec 06, 2004 at 09:07:02AM +0100, Lars Gullik Bjønnes wrote: > Alfredo Braunstein <[EMAIL PROTECTED]> writes: > > | + int max_depth = beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0; > > Please add some parenthesis: int m = (foo ? dfd : fdf); > Even if not needed it is a tiny bit clear

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Angus Leeming
Helge Hafting wrote: >>I am ambivalent. Since _I_ use bash all the time: no problem for me. >>But there are a lot of systems where bash is not the system/preffered >>shell. >> >> > Preference is not a problem.   It  is  a  problem  only  if  bash  is >  hard  to  get for the system in question. W

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Helge Hafting
Lars Gullik BjÃnnes wrote: Andre Poenitz <[EMAIL PROTECTED]> writes: | On Sun, Dec 05, 2004 at 04:32:36PM +, Angus Leeming wrote: ksh: ${FOO${BAR}}: bad substitution [EMAIL PROTECTED]:aleem-> sh [EMAIL PROTECTED]:aleem-> BAR=bar [EMAIL PROTECTED]:aleem-> FOObar=w [EMAIL PROTECTED]:aleem-

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Lars" == Lars Gullik Bjønnes <[EMAIL PROTECTED]> writes: Lars> Sure you fix some warnings, _but_ you will almost certainly Lars> still have warnings from system m4s. So I am not sure this is Lars> worth the effort. Lars> (on FC3 you might be able to get rid of all warnings dependant Lars>

Re: autogen.sh and FC3

2004-12-06 Thread Lars Gullik Bjønnes
Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: >> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: > | Angus> Jean-Marc Lasgouttes wrote: Note also that running ./autogen.sh | Angus> on the 1.3.x tree produces additional warnings acinclude.m4:9: | Angus> warning: underquoted definition o

Re: Lyx suggestion

2004-12-06 Thread Jean-Marc Lasgouttes
> "Cyrille" == Cyrille Artho <[EMAIL PROTECTED]> writes: Cyrille> Hi all, it would be nice to have the ability to tile the text Cyrille> window so several documents can be viewed at once. This would Cyrille> be particularly useful for merging text from two documents. This is something develop

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Jean-Marc Lasgouttes wrote: Note also that running ./autogen.sh Angus> on the 1.3.x tree produces additional warnings acinclude.m4:9: Angus> warning: underquoted definition of LYX_CHECK_VERSION >> Does the following patch cure the

Re: autogen.sh and FC3

2004-12-06 Thread Angus Leeming
Jean-Marc Lasgouttes wrote: > Angus> Note also that running ./autogen.sh on the 1.3.x tree produces > Angus> additional warnings > Angus> acinclude.m4:9: warning: underquoted definition of > Angus> LYX_CHECK_VERSION > > Does the following patch cure the warnings? If you want a quick test, ask La

Re: autogen.sh and FC3

2004-12-06 Thread Jean-Marc Lasgouttes
> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes: Angus> Note also that running ./autogen.sh on the 1.3.x tree produces Angus> additional warnings Angus> acinclude.m4:9: warning: underquoted definition of Angus> LYX_CHECK_VERSION Does the following patch cure the warnings? JMarc Inde

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Angus Leeming
Lars Gullik Bjønnes wrote: >>> Sorry. You're out of look. Not portable to Tru64 unix. >> > | Is it possible to install bash on all systems we support? >> > | If so, wouldn't it be simpler to require a working 'bash' > | and drop all these 'portability hacks'? > > Depends... remember that even on L

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Lars Gullik Bjønnes
Andre Poenitz <[EMAIL PROTECTED]> writes: | On Sun, Dec 05, 2004 at 04:32:36PM +, Angus Leeming wrote: >> ksh: ${FOO${BAR}}: bad substitution >> [EMAIL PROTECTED]:aleem-> sh >> [EMAIL PROTECTED]:aleem-> BAR=bar >> [EMAIL PROTECTED]:aleem-> FOObar=w >> [EMAIL PROTECTED]:aleem-> echo ${FOO${

Re: [Patch] PCH patch for Angus.

2004-12-06 Thread Andre Poenitz
On Sun, Dec 05, 2004 at 04:32:36PM +, Angus Leeming wrote: > ksh: ${FOO${BAR}}: bad substitution > [EMAIL PROTECTED]:aleem-> sh > [EMAIL PROTECTED]:aleem-> BAR=bar > [EMAIL PROTECTED]:aleem-> FOObar=w > [EMAIL PROTECTED]:aleem-> echo ${FOO${BAR}} > bad substitution > > Sorry. You're out of

Re: [patch] remove getSelectionSpan

2004-12-06 Thread Lars Gullik Bjønnes
Alfredo Braunstein <[EMAIL PROTECTED]> writes: | + int max_depth = beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0; Please add some parenthesis: int m = (foo ? dfd : fdf); Even if not needed it is a tiny bit clearer. | + int max_depth = beg != 0 ? pars_[beg - 1].getMaxDepthAfter() : 0;