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'

Reply via email to