Mirrors/miller
1
0
Fork 0
mirror of https://github.com/johnkerl/miller.git synced 2026-07-17 16:38:54 +00:00
Code Issues Projects Releases Wiki Activity
miller/pkg
History
John Kerl a723f6cdaa
Fix PPRINT alignment with multi-character OFS (#1819) (#2063) 
* Fix column alignment for wide and combining Unicode chars (#1520, #379)

PPRINT, markdown-aligned, and XTAB writers measured column widths with
utf8.RuneCountInString, which counts codepoints rather than terminal
display columns. East-Asian fullwidth characters (counted as 1 but
displayed as 2) and zero-width combining marks (counted as 1 but
displayed as 0) both caused misalignment.

Add lib.DisplayWidth wrapping uniseg.StringWidth, and use it from the
three writers. The XTAB right-aligned path also drops fmt.Sprintf with
%*s since Go's %*s pads by rune count too.

UTF8Strlen is unchanged so DSL strlen semantics are preserved.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Fix PPRINT alignment with multi-character OFS (#1819)

Previously, writePadding repeated the OFS string for each padding slot, so
non-space or multi-character OFS values produced misaligned columns and
leaked the separator into padding (e.g. --ofs=XY yielded aXYXYXYXYb).

Pad with spaces instead, leaving OFS to act only as the column separator.
The default single-space OFS is unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Use non-space multi-char OFS in test 0021 for Windows portability

PowerShell collapses single-quoted '  ' in command lines, which broke
the test on the windows-latest CI runner. Switch to --ofs XY, which
exercises the same multi-character padding code path without shell
quoting hazards.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-17 12:45:49 -04:00
..
auxents Performance and style fixes (#1981) 2026-02-18 09:19:31 -05:00
bifs Fix mlr -s shebang doc and reject arrays/maps in contains/index (#1658) (#2058) 2026-05-17 11:40:39 -04:00
cli Add --omd-aligned flag for column-padded markdown output (#2057) 2026-05-16 11:41:01 -04:00
climain Default to "cat" verb when none is supplied (#2060) 2026-05-17 12:13:21 -04:00
colorizer Multiple style updates (#1974) 2026-02-16 15:49:21 -05:00
dkvpx Add DKVPX file format (#2002) 2026-03-02 22:35:08 -05:00
dsl Fix mlr -s shebang doc and reject arrays/maps in contains/index (#1658) (#2058) 2026-05-17 11:40:39 -04:00
entrypoint Improve error propagation (#1975) 2026-02-16 16:48:41 -05:00
go-csv Some fixes for staticcheck (#2006) 2026-03-03 09:27:03 -05:00
input Fix data dropped from ragged implicit-header TSV/CSV-lite input (#1749) (#2059) 2026-05-17 11:46:59 -04:00
lib Fix column alignment for wide and combining Unicode chars (#1520, #379) (#2061) 2026-05-17 12:12:54 -04:00
mlrval Fix issue #1983 (#2040) 2026-04-19 10:44:53 -04:00
output Fix PPRINT alignment with multi-character OFS (#1819) (#2063) 2026-05-17 12:45:49 -04:00
parsing Replace GOCC parser-generator with PGPG (#2015) 2026-03-15 22:28:57 -04:00
pbnjay-strptime Performance and style fixes (#1981) 2026-02-18 09:19:31 -05:00
platform Performance and style fixes (#1981) 2026-02-18 09:19:31 -05:00
runtime Experiment with mlr script (#2009) 2026-03-05 20:14:32 -05:00
scan Performance and style fixes (#1981) 2026-02-18 09:19:31 -05:00
stream Multiple style updates (#1974) 2026-02-16 15:49:21 -05:00
terminals Default to "cat" verb when none is supplied (#2060) 2026-05-17 12:13:21 -04:00
transformers Apply join prefixes/rename to unpaired records (#1821) (#2062) 2026-05-17 12:35:58 -04:00
types Update performance docs (#1991) 2026-02-22 17:04:35 -05:00
version Post-6.18.1 release: back to 6.18.1-dev 2026-04-19 12:03:05 -04:00
README.md Export library code in pkg/ (#1391) 2023-09-10 17:15:13 -04:00

README.md

Please see ../../README-dev.md for an overview; please see each subdirectory for details about it.