From 9db9a0d22515f90a75ec09deb900308e7604ad5a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Ekl=C3=B6f?= <daniel@ekloef.se>
Date: Sat, 16 Jan 2021 15:39:44 +0100
Subject: [PATCH] =?UTF-8?q?config:=20add=20=E2=80=98none=E2=80=99=20as=20a?=
 =?UTF-8?q?=20possible=20value=20for=20=E2=80=98selection-target=E2=80=99?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When ‘selection-target’ is set to ‘none’, selecting text does not copy
the text to _any_ clipboard.

This patch also refactors the value parsing to be data driven.
---
 config.c    | 30 ++++++++++++++++++------------
 config.h    |  1 +
 selection.c |  3 +++
 3 files changed, 22 insertions(+), 12 deletions(-)

diff --git a/config.c b/config.c
index 08a46a4a..63d62ea3 100644
--- a/config.c
+++ b/config.c
@@ -603,19 +603,25 @@ parse_section_main(const char *key, const char *value, struct config *conf,
     }
 
     else if (strcmp(key, "selection-target") == 0) {
-        if (strcasecmp(value, "primary") == 0)
-            conf->selection_target = SELECTION_TARGET_PRIMARY;
-        else if (strcasecmp(value, "clipboard") == 0)
-            conf->selection_target = SELECTION_TARGET_CLIPBOARD;
-        else if (strcasecmp(value, "both") == 0)
-            conf->selection_target = SELECTION_TARGET_BOTH;
-        else {
-            LOG_AND_NOTIFY_ERR(
-                "%s:%d: [default]: %s: invalid 'selection-target'; "
-                "must be one of 'primary', 'clipboard' or 'both",
-                path, lineno, value);
-            return false;
+        static const char *const values[] = {
+            [SELECTION_TARGET_NONE] = "none",
+            [SELECTION_TARGET_PRIMARY] = "primary",
+            [SELECTION_TARGET_CLIPBOARD] = "clipboard",
+            [SELECTION_TARGET_BOTH] = "both",
+        };
+
+        for (size_t i = 0; i < ALEN(values); i++) {
+            if (strcasecmp(value, values[i]) == 0) {
+                conf->selection_target = i;
+                return true;
+            }
         }
+
+        LOG_AND_NOTIFY_ERR(
+            "%s:%d: [default]: %s: invalid 'selection-target'; "
+            "must be one of 'none', 'primary', 'clipboard' or 'both",
+            path, lineno, value);
+        return false;
     }
 
     else {
diff --git a/config.h b/config.h
index db9a5b39..753a3386 100644
--- a/config.h
+++ b/config.h
@@ -167,6 +167,7 @@ struct config {
     bool presentation_timings;
     bool hold_at_exit;
     enum {
+        SELECTION_TARGET_NONE,
         SELECTION_TARGET_PRIMARY,
         SELECTION_TARGET_CLIPBOARD,
         SELECTION_TARGET_BOTH
diff --git a/selection.c b/selection.c
index fd1b1661..2b209ae2 100644
--- a/selection.c
+++ b/selection.c
@@ -657,6 +657,9 @@ selection_finalize(struct seat *seat, struct terminal *term, uint32_t serial)
     assert(term->selection.start.row <= term->selection.end.row);
 
     switch (term->conf->selection_target) {
+    case SELECTION_TARGET_NONE:
+        break;
+
     case SELECTION_TARGET_PRIMARY:
         selection_to_primary(seat, term, serial);
         break;