[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/3.5/kdevelop/languages/cpp/debugger
From:       David Nolden <david.nolden.kde () art-master ! de>
Date:       2007-06-28 12:19:00
Message-ID: 1183033140.265500.28879.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 681279 by zwabel:

use warningYesNo instead with a do-no-show-again checkbox instead of error-messages, \
so the workflow isn't destroyed when the debugger starts bugging around BUG:147320

 M  +2 -2      dbgpsdlg.cpp  
 M  +4 -4      debuggerpart.cpp  
 M  +13 -13    gdbcontroller.cpp  
 M  +4 -4      variablewidget.cpp  


--- branches/KDE/3.5/kdevelop/languages/cpp/debugger/dbgpsdlg.cpp #681278:681279
@@ -166,14 +166,14 @@
         {
             if(ps_output_line.search(item) == -1)
             {
-                KMessageBox::error(
+                KMessageBox::warningYesNo(
                     this, 
                     // FIXME: probably should XML-escape 'item' before passing it
                     // to 'arg'.
                     i18n("<b>Could not parse output from the <tt>ps</tt> \
command!</b>"  "<p>The following line could not be parsed:"
                          "<b><tt>%1</tt>").arg(item),
-                    i18n("Internal error"));
+                    i18n("Internal error"), KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error" );  break;
             }
             
--- branches/KDE/3.5/kdevelop/languages/cpp/debugger/debuggerpart.cpp #681278:681279
@@ -652,10 +652,10 @@
         }
         if( !info.exists() )
         {
-            KMessageBox::error(
+            KMessageBox::warningYesNo(
                 mainWindow()->main(),
                 i18n("Could not locate the debugging shell '%1'.").arg( \
                shell_without_args ),
-                i18n("Debugging Shell Not Found") );
+                i18n("Debugging Shell Not Found"),KStdGuiItem::ok(), \
KStdGuiItem::cont(), "gdb_error" );  return false;
         }
     }
@@ -761,12 +761,12 @@
 {
     mainWindow()->raiseView(gdbOutputWidget);
 
-    KMessageBox::error(
+    KMessageBox::warningYesNo(
         mainWindow()->main(),
         i18n("<b>GDB exited abnormally</b>"
              "<p>This is likely a bug in GDB. "
              "Examine the gdb output window and then stop the debugger"),
-        i18n("GDB exited abnormally"));
+        i18n("GDB exited abnormally"), KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  
     // Note: we don't stop the debugger here, becuse that will hide gdb
     // window and prevent the user from finding the exact reason of the
--- branches/KDE/3.5/kdevelop/languages/cpp/debugger/gdbcontroller.cpp #681278:681279
@@ -291,11 +291,11 @@
 {
     if (stateIsOn(s_dbgNotStarted))
     {
-        KMessageBox::error(
+        KMessageBox::warningYesNo(
             0,
             i18n("<b>Gdb command sent when debugger is not running</b><br>"
             "The command was:<br> %1").arg(cmd->initialString()),
-            i18n("Internal error"));
+            i18n("Internal error"),KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  return;
     }
 
@@ -388,9 +388,9 @@
     }
     if (bad_command)
     {
-        KMessageBox::error(0, i18n("<b>Invalid debugger command</b><br>")
+        KMessageBox::warningYesNo(0, i18n("<b>Invalid debugger command</b><br>")
                            + message,
-                           i18n("Invalid debugger command"));
+                           i18n("Invalid debugger command"),KStdGuiItem::ok(), \
KStdGuiItem::cont(), "gdb_error");  return;
     }
 
@@ -646,7 +646,7 @@
     raiseEvent(program_exited);
 
     if (msgBox)
-        KMessageBox::error(0, i18n("gdb message:\n")+msg);
+        KMessageBox::warningYesNo(0, i18n("gdb message:\n")+msg,"Warning", \
KStdGuiItem::ok(), KStdGuiItem::cont(), "gdb_error");  
     emit dbgStatus (msg, state_);
 }
@@ -707,7 +707,7 @@
         // FIXME: throw an exception here. Move reporting
         // to the caller, who knows the gdb output.
 #if 0
-        KMessageBox::error(
+        KMessageBox::warningYesNo(
             0,
             i18n("Invalid gdb reply\n"
                  "Command was: %1\n"
@@ -716,7 +716,7 @@
             .arg(currentCmd_->rawDbgCommand())
             .arg(buf)
             .arg(r.reason),
-            i18n("Invalid gdb reply"));
+            i18n("Invalid gdb reply"),KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  #endif
     }
 
@@ -838,13 +838,13 @@
     if (!dbgProcess_->start( KProcess::NotifyOnExit,
                              KProcess::Communication(KProcess::All)))
     {
-        KMessageBox::error(
+        KMessageBox::warningYesNo(
             0,
             i18n("<b>Could not start debugger.</b>"
                  "<p>Could not run '%1'. "
                  "Make sure that the path name is specified correctly."
                 ).arg(dbgProcess_->args()[0]),
-            i18n("Could not start debugger"));
+            i18n("Could not start debugger"),KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  
         return false;
     }
@@ -1083,11 +1083,11 @@
         QString tty(tty_->getSlave());
         if (tty.isEmpty())
         {
-            KMessageBox::error(0, i18n("GDB cannot use the tty* or pty* devices.\n"
+            KMessageBox::warningYesNo(0, i18n("GDB cannot use the tty* or pty* \
                devices.\n"
                                        "Check the settings on /dev/tty* and \
                /dev/pty*\n"
                                        "As root you may need to \"chmod ug+rw\" tty* \
                and pty* devices "
                                        "and/or add the user to the tty group using "
-                                       "\"usermod -G tty username\"."));
+                                       "\"usermod -G tty username\"."), "Warning", \
KStdGuiItem::ok(), KStdGuiItem::cont(), "gdb_error");  
             delete tty_;
             tty_ = 0;
@@ -1330,12 +1330,12 @@
         return;
     }
 
-    KMessageBox::error(
+    KMessageBox::warningYesNo(
         0,
         i18n("<b>Debugger error</b>"
              "<p>Debugger reported the following error:"
              "<p><tt>") + result["msg"].literal(),
-        i18n("Debugger error"));
+        i18n("Debugger error"), KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  
     // Error most likely means that some change made in GUI
     // was not communicated to the gdb, so GUI is now not
--- branches/KDE/3.5/kdevelop/languages/cpp/debugger/variablewidget.cpp \
#681278:681279 @@ -758,12 +758,12 @@
     }
     else
     {
-        KMessageBox::error(
+        KMessageBox::warningYesNo(
             0,
             "<b>Can't get frame id</b>"
             "Could not found frame id from output of 'info frame'. "
             "Further debugging can be unreliable. ",
-            i18n("Internal error"));
+            i18n("Internal error"), KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  }
 
     if (frameIdChanged)
@@ -895,10 +895,10 @@
         {
             // FIXME: merge all output lines together.
             // FIXME: add 'debuggerError' to debuggerpart.
-            KMessageBox::error(
+            KMessageBox::warningYesNo(
                 0,
                 i18n("<b>Debugger error</b><br>") + lines[1],
-                i18n("Debugger error"));
+                i18n("Debugger error"), KStdGuiItem::ok(), KStdGuiItem::cont(), \
"gdb_error");  }
     }
 }


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic