protest against unknown palette presets
authorMischa POSLAWSKY <perl@shiar.org>
Sun, 13 Nov 2022 02:28:32 +0000 (03:28 +0100)
committerMischa POSLAWSKY <perl@shiar.org>
Fri, 18 Nov 2022 00:25:46 +0000 (01:25 +0100)
barcat
t/t1731-unknown_palette_--palette=none_?64.out [new file with mode: 0644]

diff --git a/barcat b/barcat
index 2339bc7e2b8ed8dc8b604ec06e10ed557f9ebcfe..b939ea95ae88a91ddd0fe89b2017ecb1ae77259d 100755 (executable)
--- 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 (file)
index 0000000..87046e4
--- /dev/null
@@ -0,0 +1 @@
+Empty palette resulting from "none"