Make Postscript renderer deterministic by Shane32 · Pull Request #643 · Shane32/QRCoder
103-106: LGTM! Date parameter correctly removed.
The format call has been updated to remove the DateTime.Now parameter, achieving the goal of deterministic output. The placeholder indices {0}, {1}, {2} correctly map to width, pointsPerModule, and EPSF string respectively.
135-155: LGTM! PS_HEADER format string is correct.
The raw string literal format is clean and the placeholder indices are correctly positioned:
- {2} for EPSF version token (line 136)
- {0} for width/size values (lines 141, 142, 148)
- {1} for scale/pointsPerModule (line 149)
157-193: LGTM! PS_FUNCTIONS format string is correct.
The raw string literal format is clean and all placeholders {0}-{6} are correctly positioned for RGB colors and module count.
195-201: LGTM! PS_FOOTER format string is correct.
The raw string literal format is clean and requires no placeholders.