From 692c5ff90ecc047f4892ba3be0e697f642124767 Mon Sep 17 00:00:00 2001
From: Stefan Zipproth <s.zipproth@acrion.ch>
Date: Fri, 27 Jun 2025 22:41:40 +0200
Subject: [PATCH] Fix profile matching for unquoted output syntax

The match() function was still using the old quoted output syntax
("output \"%99[^\"]\"" pattern) which fails to match profiles
written with the new unquoted syntax (output DP-3).

This caused duplicate profiles to be created instead of updating
existing ones. Now supports both quoted (legacy) and unquoted
(current) output formats for backward compatibility.
---
 src/store.c | 46 ++++++++++++++++++++++++++++++++++++----------
 1 file changed, 36 insertions(+), 10 deletions(-)

diff --git a/src/store.c b/src/store.c
index 15d0021..8ecc9b6 100644
--- a/src/store.c
+++ b/src/store.c
@@ -142,17 +142,43 @@ struct profile_line match(char **descriptions, int num, const char *filename) {
           profileEndLine = lineCount;
           if (profileMatchedNum == num) ps = Found;
         } else {
-          char *on_start = strstr(buffer, "output");
-          on_start       = strchr(on_start, '"');
-          on_start++;
-          char *on_end     = strchr(on_start, '"');
-          char *outputName = strndup(on_start, on_end - on_start);
+          char outputName[MAX_NAME_LENGTH];
+          char *trimmedBuffer = buffer;
+          while (isspace(*trimmedBuffer)) {
+            trimmedBuffer++; // skip leading spaces
+          }
+          char tempName[MAX_NAME_LENGTH];
+          int matched_scan = 0;
+
+          // Try quoted format first (legacy): output "Long Description (DP-3)"
+          if (sscanf(trimmedBuffer, "output \"%255[^\"]\"", tempName) == 1) {
+            // Extract output name from parentheses if present: (DP-3) -> DP-3
+            char *paren_start = strrchr(tempName, '(');
+            char *paren_end = strrchr(tempName, ')');
+            if (paren_start && paren_end && paren_end > paren_start) {
+              size_t len = paren_end - paren_start - 1;
+              strncpy(outputName, paren_start + 1, len);
+              outputName[len] = '\0';
+              matched_scan = 1;
+            }
+          } else if (sscanf(trimmedBuffer, "output %99s", outputName) == 1) {
+            // Try unquoted format: output DP-3
+            matched_scan = 1;
+          }
+          
+          if (matched_scan != 1) continue; // Skip unparseable lines
+
           // check if the output name is in the descriptions
-          int i            = 0;
-          while (descriptions[i] != NULL && strcmp(outputName, descriptions[i])) i++;
-          if (descriptions[i] != NULL) {
-            profileMatchedNum++;
-          } else {
+          bool matched = false;
+          for (int i = 0; descriptions[i] != NULL; i++) {
+            if (strcmp(outputName, descriptions[i]) == 0) {
+              matched = true;
+              profileMatchedNum++;
+              break;
+            }
+          }
+
+          if (!matched) {
             // if any output is not matched, break
             profileMatchedNum = 0;
             ps                = Looking_for_profile;