rgheck wrote:
Andre Poenitz wrote:
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...
What I meant was: I want to make this whole thing a real class anyway,
in which case there will shortly be an InsetLayout.cpp file, and it
seemed like too much trouble to put the implementation into
InsetCollapsable.cpp only to move it later.
Right, InsetLayout.cpp is the solution.
Abdel.
