On Fri, May 04, 2007 at 09:42:57AM +0200, Jean-Marc Lasgouttes wrote:
> >>>>> "Bo" == Bo Peng <[EMAIL PROTECTED]> writes:
> 
> >> Inset::MARGIN_CODE), > InsetName("float", Inset::FLOAT_CODE),
> 
> Bo> ERT_CODE dictates verbatim output which is exactly what
> Bo> InsetListings needs. I do not see a need for Inset::LISTINGS_CODE
> Bo> right now but this may turn out to be wrong.
> 
> Please NO! ERT_CODE identifies the ERT inset and nothing else. If some
> lazy programmer decided to hardcode some behaviour depending on ERT, I
> think now is the right time to cleanup the code (assuming that we
> really thing adding listings is a good idea of course) and add the
> relevant methods.

And the Right Method(tm) would probably a function 

        virtual bool InsetBase::whatever() const { ...}

with re-implementations oin InsetERT and InsetListing.

Andre'

Reply via email to