On Thu, Feb 21, 2008 at 11:32:28AM -0500, rgheck wrote: > Andre Poenitz wrote: >> On Wed, Feb 20, 2008 at 03:08:03PM -0500, Richard Heck wrote: >> >>> Index: insets/InsetLayout.h >>> =================================================================== >>> --- insets/InsetLayout.h (revision 23079) >>> +++ insets/InsetLayout.h (working copy) >>> @@ -23,6 +23,12 @@ >>> /// >>> class InsetLayout { >>> public: >>> + InsetLayout() : + name("undefined"), >>> + labelstring(from_utf8("UNDEFINED")), >>> + font(sane_font), labelfont(sane_font), >>> + bgcolor(Color_error) >>> + { labelfont.setColor(Color_error); }; >>> std::string name; >>> std::string lyxtype; >>> docstring labelstring; >>> >> >> Non-critical function implementations of this length please in the .cpp >> file... >> >> > I'll do this change shortly. Doing it properly means making InsetLayout a > proper class rather than just a C-style struct, which is what it really is > right now.
Hm? Just moving the constuctor out-of-line does not change semantics... Andre'