Andre Poenitz wrote:
On Fri, Apr 27, 2007 at 11:05:06PM -0500, Bo Peng wrote:
You can have a look at my 'rename summary' post. I think a few more
files need to be renamed.
I propose to rename class InsetOld to Inset, and rename as follows.

The idea is to remove 'InsetOld' entrirely be moving stuff either up
and down in the inset hierarchy. This is not 1.5.0 business anymore.

Note that I've done some cleanup in there recently. There's not a lot remaining:

        void setInsetName(docstring const & s) { name_ = s; }
        virtual docstring const & getInsetName() const { return name_; }
        void setPosCache(PainterInfo const &, int, int) const;

setPosCache() is also implemented in InsetMathDim which is inherited by most of the math insets; basically all except for InsetMathChar and InsetMathSymbol. I've (in my local tree) transferred setPosCache() to InsetBase for a week now and I didn't see any problem with that. Actually I am almost sure that this is the right thing to do. Doing that means that we can delete InsetMathDim and that all that will remain in InsetOld is set/getInsetName().

What do you think?

Abdel.

Reply via email to