From 02ce237f911f7b970ea9b093226a9e7d09fffbf4 Mon Sep 17 00:00:00 2001
From: Simon Tatham <anakin@pobox.com>
Date: Sun, 24 Apr 2016 07:29:25 +0100
Subject: [PATCH] Improve 'illegal colour' error message in icon.pl.

It actually went off this morning, after an upgrade of ImageMagick,
and I found that it contained both unprintable characters in the
colour description and the wrong variable in the coordinate display.
---
 icons/icon.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/icons/icon.pl b/icons/icon.pl
index b9cce7e..d0c5a8e 100755
--- a/icons/icon.pl
+++ b/icons/icon.pl
@@ -203,7 +203,8 @@ sub readicon {
 		$currbits += $depth;
 		if ($x < $w && defined ($pix = $data->[$y*$w+$x])) {
 		    if (!defined $pal{$pix}) {
-			die "illegal colour value $pix at pixel $i in $filename\n";
+                        my $pixprintable = unpack "H*", $pix;
+			die "illegal colour value $pixprintable at pixel ($x,$y) in $filename\n";
 		    }
 		    $currbyte |= $pal{$pix};
 		}