From e0b2994847e8a1340a19013edfb5667086409d6c Mon Sep 17 00:00:00 2001
From: Ernie Pasveer <epasveer@att.net>
Date: Sat, 25 Nov 2023 10:39:38 -0600
Subject: [PATCH] Fixed breakpoint conditions to handle string comparisons.

---
 src/SeerBreakpointCreateDialog.cpp |  5 +-
 src/SeerBreakpointCreateDialog.ui  | 74 ++++++++++--------------------
 src/SeerGdbWidget.cpp              |  4 +-
 3 files changed, 31 insertions(+), 52 deletions(-)

diff --git a/src/SeerBreakpointCreateDialog.cpp b/src/SeerBreakpointCreateDialog.cpp
index d8827aad..abc7f8fc 100644
--- a/src/SeerBreakpointCreateDialog.cpp
+++ b/src/SeerBreakpointCreateDialog.cpp
@@ -180,7 +180,10 @@ QString SeerBreakpointCreateDialog::breakpointText () const {
 
     if (conditionalEnabled()) {
         if (conditionalText() != "") {
-            breakpointParameters += " -c \"" + conditionalText() + "\"";
+
+            QString str = conditionalText().replace('"', "\\\""); // Quote " characters.
+
+            breakpointParameters += " -c \"" + str + "\"";
         }
     }
 
diff --git a/src/SeerBreakpointCreateDialog.ui b/src/SeerBreakpointCreateDialog.ui
index bf848933..92ad4a7f 100644
--- a/src/SeerBreakpointCreateDialog.ui
+++ b/src/SeerBreakpointCreateDialog.ui
@@ -146,26 +146,13 @@
        </widget>
       </item>
       <item row="0" column="1">
-       <spacer name="temporaryHorizontalSpacer">
-        <property name="orientation">
-         <enum>Qt::Horizontal</enum>
-        </property>
-        <property name="sizeHint" stdset="0">
-         <size>
-          <width>40</width>
-          <height>20</height>
-         </size>
-        </property>
-       </spacer>
-      </item>
-      <item row="0" column="2">
        <widget class="QCheckBox" name="conditionalCheckBox">
         <property name="text">
          <string>Condition if:</string>
         </property>
        </widget>
       </item>
-      <item row="0" column="3">
+      <item row="0" column="2" colspan="2">
        <widget class="QLineEdit" name="conditionalLineEdit">
         <property name="clearButtonEnabled">
          <bool>true</bool>
@@ -180,86 +167,73 @@
        </widget>
       </item>
       <item row="1" column="1">
-       <spacer name="hardwareHorizontalSpacer">
-        <property name="orientation">
-         <enum>Qt::Horizontal</enum>
-        </property>
-        <property name="sizeHint" stdset="0">
-         <size>
-          <width>40</width>
-          <height>20</height>
-         </size>
-        </property>
-       </spacer>
-      </item>
-      <item row="1" column="2">
        <widget class="QCheckBox" name="ignoreCountCheckBox">
         <property name="text">
          <string>Ignore Count</string>
         </property>
        </widget>
       </item>
-      <item row="1" column="3">
+      <item row="1" column="2">
        <widget class="QLineEdit" name="ignoreCountLineEdit">
         <property name="clearButtonEnabled">
          <bool>true</bool>
         </property>
        </widget>
       </item>
-      <item row="2" column="0">
-       <widget class="QCheckBox" name="pendingCheckBox">
-        <property name="text">
-         <string>Pending</string>
-        </property>
-       </widget>
-      </item>
-      <item row="2" column="1">
-       <spacer name="pendingHorizontalSpacer">
+      <item row="1" column="3">
+       <spacer name="horizontalSpacer">
         <property name="orientation">
          <enum>Qt::Horizontal</enum>
         </property>
         <property name="sizeHint" stdset="0">
          <size>
-          <width>40</width>
+          <width>131</width>
           <height>20</height>
          </size>
         </property>
        </spacer>
       </item>
-      <item row="2" column="2">
+      <item row="2" column="0">
+       <widget class="QCheckBox" name="pendingCheckBox">
+        <property name="text">
+         <string>Pending</string>
+        </property>
+       </widget>
+      </item>
+      <item row="2" column="1">
        <widget class="QCheckBox" name="threadIdCheckBox">
         <property name="text">
          <string>Thread Id</string>
         </property>
        </widget>
       </item>
-      <item row="2" column="3">
+      <item row="2" column="2">
        <widget class="QLineEdit" name="threadIdLineEdit">
         <property name="clearButtonEnabled">
          <bool>true</bool>
         </property>
        </widget>
       </item>
-      <item row="3" column="0">
-       <widget class="QCheckBox" name="disabledCheckBox">
-        <property name="text">
-         <string>Disabled</string>
-        </property>
-       </widget>
-      </item>
-      <item row="3" column="1">
-       <spacer name="disabledHorizontalSpacer">
+      <item row="2" column="3">
+       <spacer name="horizontalSpacer_2">
         <property name="orientation">
          <enum>Qt::Horizontal</enum>
         </property>
         <property name="sizeHint" stdset="0">
          <size>
-          <width>40</width>
+          <width>131</width>
           <height>20</height>
          </size>
         </property>
        </spacer>
       </item>
+      <item row="3" column="0">
+       <widget class="QCheckBox" name="disabledCheckBox">
+        <property name="text">
+         <string>Disabled</string>
+        </property>
+       </widget>
+      </item>
      </layout>
     </widget>
    </item>
diff --git a/src/SeerGdbWidget.cpp b/src/SeerGdbWidget.cpp
index 751a1863..62468c0c 100644
--- a/src/SeerGdbWidget.cpp
+++ b/src/SeerGdbWidget.cpp
@@ -1983,7 +1983,9 @@ void SeerGdbWidget::handleGdbBreakpointCondition (QString breakpoint, QString co
         return;
     }
 
-    handleGdbCommand("-break-condition " + breakpoint + " " + condition);
+    QString str = condition.replace('"', "\\\""); // Quote " characters.
+
+    handleGdbCommand("-break-condition " + breakpoint + " \"" + condition + "\"");
     handleGdbGenericpointList();
 }