Unidata / UDUNITS-2

API and utility for arithmetic manipulation of units of physical quantities
http://www.unidata.ucar.edu/software/udunits
Other
62 stars 36 forks source link

nit: whitespace clean #72

Closed schwehr closed 6 years ago

schwehr commented 6 years ago

This one is not super important, but it would be nice if the code formatting could be standardized. My ideal would be to pick a clang-format setup and enforce it. But baring that, it would be helpful if the code was at least trailing whitespace clean. e.g.

find . -type f | xargs perl -pi -e 's/\s+\n/\n/g'
semmerson commented 6 years ago

You're right: it's a nit.

I did it anyway.

Are you aware that JPL -- the institution responsible for deep space missions like Voyager, Mars Pathfinder, Opportunity, Curiosity, Cassini, Juno, etc. -- doesn't enforce any formatting standard for C code?

schwehr commented 6 years ago

I worked on Pathfinder, Opportunity, Curiosity, Phoenix, and Polar Lander. I recommend not following them as examples of what to do for robust software engineering :(

semmerson commented 6 years ago

@schwehr Noted.