fonts/nerd-fonts
1
0
Fork 0
mirror of https://github.com/ryanoasis/nerd-fonts.git synced 2025-01-06 21:49:40 +02:00
Code Issues Releases Activity

test-fonts: Fix missing codepoints and allow MacOS

Browse Source 
[why]
If a set has multiple ranges defined only the last of that ranges is
actually printed.

[how]
With commit
  7a4b5f872  Fixes build errors (ShellCheck)

a false positive of spellcheck lead to a 'correction' in the code that
actually broke it. The mapfile does not accumulate the sequences but
fills it in and so just the last sequence 'survives'.

Dropping mapfile also enables this script on MacOS, as that ancient
bash does not have mapfile.

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
This commit is contained in:
Fini Jastrow 2023-06-01 07:54:21 +02:00
parent 6f1cdb82d9
commit 6bb3b23d88
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
bin/scripts/test-fonts.sh
View File

@ -127,7 +127,8 @@ function print-unicode-ranges() {
local startDecimal=$((16#$start))
local endDecimal=$((16#$end))
mapfile -t combinedRanges < <(seq "$startDecimal" "$endDecimal")
# shellcheck disable=SC2207 # We DO WANT the output to be split
combinedRanges+=($(seq "$startDecimal" "$endDecimal"))
done