On Fri, Mar 21, 2008 at 02:24:06AM -0400, rgheck wrote:
>
> OK, here's my attempt once again to fix bug 2178. I've tested this a bit, 
> but I'm not an extensive user of tables, so I'm sure there are bugs here. 
> But the general idea seems to work.
>
> Note that this is intended to be a "minimal patch", involving as few 
> changes as possible to the existing code. It may well be that there are 
> better ways to do this, longer term, but I am NOT about to start messing 
> with the InsetTabular code in any serious way. The list of names at the 
> start of InsetTabular.cpp is enough to scare me off for good. ;-)
>
> Comments welcome.

I would have prefered if the InsetTableCell and the CellData were
combined in a single class (assumng they were always 1:1) and the
whole stuff located in InsetTabular.cpp instead of a separate .h/.cpp
file pair as this is pretty much an implementation detail of table
insets.

But feel free to commit what you have. If it annoys me too much I could
always "fix" it myself ;-)

Andre'

Reply via email to