Le 19/04/2016 01:26, Guillaume Munch a écrit :
This turns InsetMathHull::isTable() into a redefinition of
Inset::isTable().
OK, I missed the definition of InsetMathHull::isTable() at that time.
As I see it, your version of isTable means 'does it handle tabular
features?'
Mine is more like: "is it a thing with lines and columns"? In maths, it
applies to anything that is a grid. Many of those actually implement
tabular features too (with some exceptions).
Therefore we have a conflict here.
Since InsetMathHull::isTable() is essentially private, what about
renaming it to hasTabularFeatures() or something like that? I can also
rename the other one, if you can think of a good name.
I can do it if you wish.
JMarc
