On Wed, Feb 20, 2002 at 04:15:48PM +0100, Lars Gullik Bjønnes wrote:
 
> Martin Vermeer <[EMAIL PROTECTED]> writes:
> 
> | lyxlayout.C: In method `const class string & LyXLayout::name() const':
> | lyxlayout.C:740: warning: returning reference to temporary
> | lyxlayout.C: In method `const class string & LyXLayout::obsoleted_by() const':
> | lyxlayout.C:752: warning: returning reference to temporary
> 
> Fixing these fixed the problem.
> 
> I have now a patch that apperars to be working, but I have not tested
> switching layout<->layout, reading nonexistant layout etc.
> 
> The pach has been cleaned up a bit and is closer to ready for
> inclusion.
> 
> People, please test (and be careful)
> Report any and all problems to me...

I am still left with one nagging question... it was your patch that
moved name() and setName(...) of LyXLayout from inline in the .h file
to the .C file. Why? Does that make any difference for the bug, in 
other words, did this cause the bug to appear? If so, how?

Martin

Attachment: msg33278/pgp00000.pgp
Description: PGP signature

Reply via email to