Re: [patch] fix crash on command line with branches

2007-10-08 Thread Jürgen Spitzmüller
Abdelrazak Younes wrote: > this _or_ (use_gui) is fine. But (theApp()) is probably better in case > we develop a non-gui app someday. I chose theApp() (it was easier to access from BranchList). Jürgen

Re: [patch] fix crash on command line with branches

2007-10-08 Thread Abdelrazak Younes
Abdelrazak Younes wrote: Jürgen Spitzmüller wrote: http://bugzilla.lyx.org/show_bug.cgi?id=4255 The patch looks straightforward to me. Abdel, what do you think? Yes, this of (use_gui) is fine. this _or_ (use_gui) is fine. But (theApp()) is probably better in case we develop a non-gui app s

Re: [patch] fix crash on command line with branches

2007-10-08 Thread Abdelrazak Younes
Jürgen Spitzmüller wrote: http://bugzilla.lyx.org/show_bug.cgi?id=4255 The patch looks straightforward to me. Abdel, what do you think? Yes, this of (use_gui) is fine. Abdel.

[patch] fix crash on command line with branches

2007-10-08 Thread Jürgen Spitzmüller
http://bugzilla.lyx.org/show_bug.cgi?id=4255 The patch looks straightforward to me. Abdel, what do you think? Jürgen Index: src/BranchList.cpp === --- src/BranchList.cpp (Revision 20838) +++ src/BranchList.cpp (Arbeitskopie) @@ -24,7