Re: r19077 - in /lyx-devel/branches/personal/younes/mvc/src: ...

2007-07-18 Thread Abdelrazak Younes

Angus Leeming wrote:

[EMAIL PROTECTED] wrote:

Author: younes
Date: Sat Jul 14 15:15:51 2007
New Revision: 19077

URL: http://www.lyx.org/trac/changeset/19077
Log:
Polish and bug fixing

* Buffer::~Buffer(): transfer label updating from 
LyXView::closeBuffer() here for child documents.


Looks to me like you should wrap the entire contents of this destructor 
in a try/catch(std::exception) block...


I don't think there's any exception thrown in updateLabels() but you're 
right, it's safer. I'll do that, thanks.


Abdel.



Re: r19077 - in /lyx-devel/branches/personal/younes/mvc/src: ...

2007-07-18 Thread Angus Leeming

[EMAIL PROTECTED] wrote:

Author: younes
Date: Sat Jul 14 15:15:51 2007
New Revision: 19077

URL: http://www.lyx.org/trac/changeset/19077
Log:
Polish and bug fixing

* Buffer::~Buffer(): transfer label updating from LyXView::closeBuffer() here 
for child documents.


Looks to me like you should wrap the entire contents of this 
destructor in a try/catch(std::exception) block...


Angus