Code review comment for lp:~michihenning/unity/phablet-unity-api-whitespace-test

Revision history for this message
Michi Henning (michihenning) wrote :

I did the rename of Version.h.cmake to Version.h.in because cmake generates lots of .cmake files. Those files contain lots of trailing space. Version.h.cmake is not a generated file so it should have a different extension.

Right, the script doesn't work with file names containing spaces. Not sure how serious this is.

@Sawicz: What looks like a double space is actually a blank and a tab :-)

I can rewrite this in Python, if you like. Not totally sure whether it's worth it though for something as simple as this?

I don't know of a ready-made tool to do this. If there is one, I'd be happy to look at it.

« Back to merge proposal