Alfredo Braunstein wrote:
[*] in other words, if you iterate with DocIterator, the inset * member of
CUrsorSlice will never point to a tabular owned insettext but to the
tabular inset.
I didn't know that, thanks. But this is bad and inconsistent IMHO. We
should change that and implement a new virtual method in Inset:
/// \return the Inset at position idx or "this" if the Inset contains
/// no cell.
virtual Inset & cell(idx_type idx);
I see that InsetMath already contains a cell() method which returns a
MathData. We should merge that with Inset::cell(); either by making
MathData a proper Inset or by returning the englobing InsetMathNest (I
am not sure there is one).
Maybe the best would be to move the method to CursorSlice where it
belongs, there you don't have the constraint of keeping a valid cursor
position... In DocIterator now it's misleading because it's the only
forward* method that travels into an invalid cursor position...
good idea.
Don't worry, I can do this myself when I put my arse in gear ;-)
Good :-)
Abdel.
