aboutsummaryrefslogtreecommitdiffstats
path: root/bin
diff options
context:
space:
mode:
authorDanilo M. <danix@danix.xyz>2026-09-17 08:52:14 +0200
committerDanilo M. <danix@danix.xyz>2026-09-17 08:52:14 +0200
commit6832a816f9965e273c104f7641c662d6d5d57ced (patch)
tree02a25a3cff9b9ca5734ac51d8744e94bec11c7bd /bin
parent159fabf66ca1b7e0a745165de0ef56e2631f79b8 (diff)
downloadconky-theme-udt-6832a816f9965e273c104f7641c662d6d5d57ced.tar.gz
conky-theme-udt-6832a816f9965e273c104f7641c662d6d5d57ced.zip
fix: guard a nil timestamp in the icon lookup
icon(800, nil, 1500, 1900) compared nil against a number and raised, which in this project is a blank dashboard with no message. The module's own header promises nothing raises, and is_day() already guards the same case, so the inconsistency was the bug. Not reachable from the widget as written: it passes os.time(), and parse() refuses a response without sunrise and sunset, so no caller can currently produce the failing combination. Fixed anyway, because the contract is what future callers will rely on and the fix is one word. The test file also now says why the codepoint VALUES carry no assertion: comparing M.ICON.snow against a literal would only prove the escape was typed twice. Whether E31A draws a snow cloud is what the glyph sheet is for, and mutation testing confirms the assertions cannot see it. Found by code review. The same fix is applied to the plan, so Tasks 2-5 inherit the guarded version. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Diffstat (limited to 'bin')
0 files changed, 0 insertions, 0 deletions