nagayon-935 / DrawlScan

80%

main: 89%

LAST BUILD ON BRANCH refactor/render-visibility-and-width

branch: refactor/render-visibility-and-width

CHANGE BRANCH

x

Reset

• refactor/render-visibility-and-width
• coverage-up
• main
• refactor/type-safe-options-and-cleanup
• release/v0.3.2
• releases/v0.1.0
• releases/v0.2.0
• releases/v0.2.1
• releases/v0.3.0
• releases/v0.3.1
• releases/v0.3.5
• releases/v0.3.6
• releases/v0.3.7
• releases/v0.3.8
• releases/v0.3.9
• releases/v0.4.0
• releases/v0.4.1
• releases/v0.4.2
• releases/v0.4.3
• releases/v0.4.4
• releases/v0.4.5
• releases/v0.5.0
• releases/v0.5.1
• v0.3.4

Build # 29065248032

Build Type

Pull #29

github

Committed by nagayon-935

Commit Message

refactor: replace magic visibility strings with (string, bool) and use rune-based widths

R-2: LayerHandler and every Print*Layer/LookupCountry now return
(string, bool) instead of relying on sentinel strings ("", "invisible")
to signal 'nothing to render'. Removes magic-string comparisons from
processAndPrintPacket and makes the 'no content' case a checked, typed
contract instead of a string convention.

R-4: RenderBlock and PrintHorizontalBlocks size their boxes using
utf8.RuneCountInString instead of byte length, so multi-byte UTF-8
content no longer inflates the computed box width.

Pull Request Pull Request #29: refactor: replace magic visibility strings with (string, bool); rune-based widths

Coverage Stats

31 of 34 new or added lines in 12 files covered. (91.18%)

395 of 491 relevant lines covered (80.45%)

5.94 hits per line