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'