X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Ffrontends%2Fxforms%2FFormError.C;h=0a6b5b2d1a583f3e926ddb05de99f861742dd32a;hb=9f29ab3aa5fb11baca9bc28dc3710076cb3a2645;hp=34b0b5077e566409218447dba50c128735101a06;hpb=c9dbe4fd82bf88c1b8c8abb7be24f86c6ec25e3f;p=lyx.git

diff --git a/src/frontends/xforms/FormError.C b/src/frontends/xforms/FormError.C
index 34b0b5077e..0a6b5b2d1a 100644
--- a/src/frontends/xforms/FormError.C
+++ b/src/frontends/xforms/FormError.C
@@ -19,11 +19,15 @@
 #include "FormError.h"
 #include "form_error.h"
 #include "insets/inseterror.h"
+#include "support/LAssert.h"
 
-FormError::FormError( LyXView * lv, Dialogs * d )
-	: FormInset( lv, d, _("LaTeX Error") ),
-	  dialog_(0), inset_(0), message_("")
+using SigC::slot;
+
+FormError::FormError(LyXView * lv, Dialogs * d)
+	: FormInset( lv, d, _("LaTeX Error")),
+	  inset_(0)
 {
+	Assert(lv && d);
 	// let the dialog be shown
 	// This is a permanent connection so we won't bother
 	// storing a copy because we won't be disconnecting.
@@ -31,15 +35,9 @@ FormError::FormError( LyXView * lv, Dialogs * d )
 }
 
 
-FormError::~FormError()
-{
-	delete dialog_;
-}
-
-
 FL_FORM * FormError::form() const
 {
-	if ( dialog_ ) return dialog_->form;
+	if (dialog_.get()) return dialog_->form;
 	return 0;
 }
 
@@ -47,14 +45,14 @@ FL_FORM * FormError::form() const
 void FormError::disconnect()
 {
 	inset_ = 0;
-	message_.empty();
+	message_.erase();
 	FormInset::disconnect();
 }
 
 
-void FormError::showInset( InsetError * inset )
+void FormError::showInset(InsetError * inset)
 {
-	if (inset == 0) return;
+	if (inset == 0) return; // Is this _really_ allowed? (Lgb)
 
 	// If connected to another inset, disconnect from it.
 	if (inset_)
@@ -62,7 +60,7 @@ void FormError::showInset( InsetError * inset )
 
 	inset_ = inset;
 	message_ = inset->getContents();
-	ih_ = inset->hide.connect(slot(this, &FormError::hide));
+	ih_ = inset->hideDialog.connect(slot(this, &FormError::hide));
 	show();
 }
 
@@ -75,9 +73,9 @@ void FormError::update()
 
 void FormError::build()
 {
-	dialog_ = build_error();
+	dialog_.reset(build_error());
 
-	// Workaround dumb xforms sizing bug
-	minw_ = form()->w;
-	minh_ = form()->h;
+        // Manage the cancel/close button
+	bc().setCancel(dialog_->button_cancel);
+	bc().refresh();
 }