From e5a35b808a2e0f6d7107a3f79067524de89b6be1 Mon Sep 17 00:00:00 2001
From: Mischa POSLAWSKY <perl@shiar.org>
Date: Sun, 13 Nov 2022 03:28:32 +0100
Subject: [PATCH] protest against unknown palette presets

---
 barcat                                         | 7 ++++++-
 t/t1731-unknown_palette_--palette=none_?64.out | 1 +
 2 files changed, 7 insertions(+), 1 deletion(-)
 create mode 100644 t/t1731-unknown_palette_--palette=none_?64.out

diff --git a/barcat b/barcat
index 2339bc7..b939ea9 100755
--- a/barcat
+++ b/barcat
@@ -57,6 +57,7 @@ GetOptions(\%opt,
 	'indicators:s',
 	'palette=s' => sub {
 		$opt{palette} = {
+			''     => [],
 			fire   => [qw( 90 31 91 33 93 97 96 )],
 			fire256=> [map {"38;5;$_"} qw(
 				235  52 88 124 160 196
@@ -75,7 +76,11 @@ GetOptions(\%opt,
 				(map { 201 - $_     } 0..4), # -b
 				196,
 			],
-		}->{$_[1]} // [ split /[^0-9;]/, $_[1] ];
+		}->{$_[1]} // do {
+			my @vals = split /[^0-9;]/, $_[1]
+				or die "Empty palette resulting from \"$_[1]\"\n";
+			\@vals;
+		};
 	},
 	'stat|s!',
 	'signal-stat=s',
diff --git a/t/t1731-unknown_palette_--palette=none_?64.out b/t/t1731-unknown_palette_--palette=none_?64.out
new file mode 100644
index 0000000..87046e4
--- /dev/null
+++ b/t/t1731-unknown_palette_--palette=none_?64.out
@@ -0,0 +1 @@
+Empty palette resulting from "none"
-- 
2.30.0