What is the current state of spaces in the preprocessor

[The-MEO]

Describe the bug

Is it correct, that the preprocessor removes all spaces from defines?

To Reproduce

`define SVTEST(_NAME_) task automatic TEST_GITHUB();
`define SVTEST_END endtask

module github_issue;
`SVTEST(get_root)
`SVTEST_END
endmodule

Command to execute verible-verilog-preprocessor preprocess github.sv

Actual behavior:

`defineSVTEST(_NAME_)task automatic TEST_GITHUB();
`defineSVTEST_ENDendtask

module github_issue;
taskautomaticTEST_GITHUB();
endtask
endmodule

Expected behavior

I would expect that the spaces are preserved in define's after reading #1528.

`define SVTEST(_NAME_) task automatic TEST_GITHUB();
`define SVTEST_END endtask

module github_issue;
task automatic TEST_GITHUB();
endtask
endmodule

[jiao-x]

Hello，has this problem been solved❓

[hzeller]

I currently have not much time working on the project, so unless someone figures out the details and sends a Pull Request it might take a while.

I think there is a partial PR #1376 from a while back that was lost in review, but might be a starting point (it looks like it is full of merge conflicts, but I suppose most of it is due to files moving directories and underscores in filenames being dashes now, so the initial change might be easy to get started).