tecnickcom / gogen / 28875040662

100%

push

github

fix(strsplit): prevent empty chunks, refactor into helpers, and add fuzz tests and benchmarks

- Fix ChunkLine emitting a blank chunk for whitespace-only input shorter than size (e.g. ChunkLine("   ", 5, -1) returned [""]); the new appendChunk helper drops empty results on every path so a produced chunk is never blank.
- Refactor Chunk and ChunkLine into small single-purpose helpers (runeSafeEnd, separatorEnd, appendChunk, appendLine, limitReached), removing the //nolint:gocognit,gocyclo,cyclop suppressions while keeping the output byte-for-byte identical.
- Pre-size the result slices with a capacity hint to reduce reallocations.
- Clarify the package and function docs: byte (not rune) size semantics, the wider-than-size rune caveat, newline-only ("\n") boundaries with "\r" and other Unicode line separators handled only via trimming, blank-line collapsing, and the nil-vs-empty return contract.
- Add FuzzChunk and FuzzChunkLine asserting no empty chunks, trimmed output, the n upper bound, UTF-8 validity, the size bound, and non-whitespace content preservation for valid UTF-8 input, with regression seeds for negative size and invalid UTF-8 bytes.
- Add benchmarks covering ASCII, separator-free, punctuation, Unicode, and multiline inputs.
- Add tests for CRLF, lone CR, blank-line collapse, whitespace-only input, leading whitespace with a small size, and a multi-byte (U+2003) space separator.
- Rename example_strplit_test.go to example_strsplit_test.go and fix comment typos.

54 of 54 new or added lines in 1 file covered. (100.0%)

13887 of 13891 relevant lines covered (99.97%)

978.56 hits per line