Jose' Matos wrote:

> The attached patch unifies the API used for the different backends
> in insets.
> 
> In inset.h we have:
> 
> int latex(Buffer const &, std::ostream &,LatexRunParams const &);
> int ascii(Buffer const &, std::ostream &,LatexRunParams const &);
> int linuxdoc(Buffer const &, std::ostream &,LatexRunParams const &);
> int docbook(Buffer const &, std::ostream &,LatexRunParams const &);

[snip...]

>   Comments?

Good! Especially the getting rid of 'nice'. Could you see if you can 
can constify the non-const buffer member functions that alter 'nice' 
too. Maybe in a separate iteration of course...

You should rename LatexRunParams though. ExportParams?

And it really seems like parargraph_funcs.C is getting too big...

-- 
Angus

Reply via email to