From: Mischa POSLAWSKY <perl@shiar.org>
Date: Sun, 9 Apr 2017 01:21:39 +0000 (+0200)
Subject: charset: no .X on empty glyph cells
X-Git-Tag: v1.10~190
X-Git-Url: http://git.shiar.nl/sheet.git/commitdiff_plain/28ac2dea2dcafa09f6d7246c8703225fb96efeea

charset: no .X on empty glyph cells

Keep empty class for missing/unassigned code points.
---

diff --git a/Shiar_Sheet/FormatChar.pm b/Shiar_Sheet/FormatChar.pm
index 8054655..ef0445d 100644
--- a/Shiar_Sheet/FormatChar.pm
+++ b/Shiar_Sheet/FormatChar.pm
@@ -8,7 +8,7 @@ use utf8;
 use Data::Dump 'pp';
 use PLP::Functions 'EscapeHTML';
 
-our $VERSION = '1.06';
+our $VERSION = '1.07';
 
 our $uc = do 'unicode-char.inc.pl';
 
@@ -34,12 +34,12 @@ sub glyph_html {
 	my ($class, $name, $mnem, $entity, $string) = @$info;
 
 	my $cell = EscapeHTML($string || $char);
-	my $title = sprintf 'U+%04X%s', $codepoint, $name && " ($name)";
+	my $title = sprintf 'U+%04X%s', $codepoint, !!$name && " ($name)";
 
-	$cell = "<span>$cell</span>" if $class =~ /\bZs\b/;
+	$cell = "<span>$cell</span>" if $class and $class =~ /\bZs\b/;
 	$cell = '&nbsp;' if $cell eq '';
 
-	return ($cell, EscapeHTML($title), "X $class", $mnem, $entity);
+	return ($cell, EscapeHTML($title), !!$class && "X $class", $mnem, $entity);
 }
 
 sub glyphs_html {
@@ -109,7 +109,7 @@ sub cell {
 		}
 
 		if ($self->{style} eq 'di') {
-			if ($mnem =~ /…/) {
+			if ($mnem and $mnem =~ /…/) {
 				# incomplete representation, usually partial
 			}
 			elsif ($class =~ /\bu-di\b/) {