Unnecessary lemon_sprintf in lemon.c?
(1) By Zellyn Hunter (zellyn) on 2021-11-26 01:43:18 [link]
Is the `lemon_sprintf(line,…` on [line 4768](https://sqlite.org/src/file?ci=trunk&name=tool/lemon.c&ln=4768) in `ReportTable` completely spurious? I don't see `line` being used elsewhere in that function. Just curious. (I'm porting `lemon.c` to Go, fwiw.)
(2) By Larry Brasfield (larrybr) on 2022-01-14 22:37:50 in reply to 1
I agree that it provides no functionality in normal usage of Lemon. However, I know that when debugging it can be useful to get information into a readable form somewhere that never escapes the program except via display by the debugger. I see this motivation as more likely than a simple oversight by the code's author. Good luck with your port.
(3) By Richard Hipp (drh) on 2022-01-14 23:29:29 in reply to 1 [link]
Thanks for the report. Harmless extra code fixed [on trunk]. : src:/info/6d2f95a474a0c196