Re: [patch] clean up tabular logic

2003-10-06 Thread Lars Gullik Bjønnes
John Levon <[EMAIL PROTECTED]> writes: | On Mon, Oct 06, 2003 at 04:07:39PM +0200, Andre Poenitz wrote: > >> void foo(Par pat, bool thisorthat) > | I think experience even just with LyX source has shown it's generally a | bad idea. The preferable refactoring is obviously to share the code in | com

Re: [patch] clean up tabular logic

2003-10-06 Thread John Levon
On Mon, Oct 06, 2003 at 04:07:39PM +0200, Andre Poenitz wrote: > void foo(Par pat, bool thisorthat) I think experience even just with LyX source has shown it's generally a bad idea. The preferable refactoring is obviously to share the code in common between the two, then have two functions built

[patch] clean up tabular logic

2003-10-06 Thread Andre Poenitz
Some simplification around the reinit/Reinit business of LyXTabular which led to the removal of the LyXTabular::owner_ member. Andre' PS: Is anybody particular fond of void foo(Par pat, bool thisorthat) { if (thisorthat) do_this(); else do_that();