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

List:       kde-commits
Subject:    kdevelop/parts/valgrind
From:       Harald Fernengel <harald () trolltech ! com>
Date:       2003-01-07 4:01:07
[Download RAW message or body]

CVS commit by harald: 

make the dialog a bit smarter


  M +22 -4     valgrind_dialog.cpp   1.3


--- kdevelop/parts/valgrind/valgrind_dialog.cpp  #1.2:1.3
@@ -1,4 +1,5 @@
 #include <qlayout.h>
 #include <qcheckbox.h>
+#include <qregexp.h>
 
 #include <klocale.h>
@@ -54,13 +55,17 @@ void ValgrindDialog::setValExecutable( c
 }
 
+static const QString leakCheckParam( "--leak-check=yes" );
+static const QString reachableParam( "--show-reachable=yes" );
+static const QString childrenParam( "--trace-children=yes" );
+
 QString ValgrindDialog::valParams() const
 {
   QString params = w->valParamEdit->text();
   if ( w->memleakBox->isChecked() )
-    params += " --leak-check=yes";
+    params += " " + leakCheckParam;
   if ( w->reachableBox->isChecked() )
-    params += " --show-reachable=yes";
+    params += " " + reachableParam;
   if ( w->childrenBox->isChecked() )
-    params += " --trace-children=yes";
+    params += " " + childrenParam;
   
   return params;
@@ -69,5 +74,18 @@ QString ValgrindDialog::valParams() cons
 void ValgrindDialog::setValParams( const QString& params )
 {
-  w->valParamEdit->setText( params );
+  QString myParams = params;
+  if ( myParams.contains( leakCheckParam ) )
+    w->memleakBox->setChecked( true );
+  if ( myParams.contains( reachableParam ) )
+    w->reachableBox->setChecked( true );
+  if ( myParams.contains( childrenParam ) )
+    w->childrenBox->setChecked( true );
+  w->init();
+
+  myParams = myParams.replace( QRegExp( leakCheckParam ), "" );
+  myParams = myParams.replace( QRegExp( reachableParam ), "" );
+  myParams = myParams.replace( QRegExp( childrenParam ), "" );
+  myParams = myParams.stripWhiteSpace();
+  w->valParamEdit->setText( myParams );
 }
 


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

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