chipsalliance/Surelog

Logging: Superfluous newlines eat up vertical space and make logs hard to read

Closed this issue · 1 comments

All log messages end with an unnecessary newline. This makes it hard to go through Surelog logfiles as they always use twice the vertical space.

So instead of a log-output like

[INF:CM0023] Creating log file /foo/bar/slpp_all/surelog.log.

[ERR:PP0101] /foo/bar/clk_rst_if.sv:24: Cannot open include file "/foo/bar/hello.svh".

[ERR:PP0101] /foo/bar/clk_rst_if.sv:25: Cannot open include file "/foo/bar/world.svh".

[ERR:PP0102] /foo/bar/clk_rst_if.sv:146:5: Unknown macro "quux".

[ERR:PP0102] /foo/bar/clk_rst_if.sv:160:7: Unknown macro "baz".

This would be much more manageable and readable if we have a log-message per line like other tools do:

[INF:CM0023] Creating log file /foo/bar/slpp_all/surelog.log.
[ERR:PP0101] /foo/bar/clk_rst_if.sv:24: Cannot open include file "/foo/bar/hello.svh".
[ERR:PP0101] /foo/bar/clk_rst_if.sv:25: Cannot open include file "/foo/bar/world.svh".
[ERR:PP0102] /foo/bar/clk_rst_if.sv:146:5: Unknown macro "quux".
[ERR:PP0102] /foo/bar/clk_rst_if.sv:160:7: Unknown macro "baz".

Fixed by #3889