SQLite Forum

Unnecessary lemon_sprintf in lemon.c?

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][1].

[1]: src:/info/6d2f95a474a0c196