search

cgmb / guardonce

Utilities for converting from C/C++ include guards to #pragma once and back again.
MIT License
142 stars 3 forks source link

once2guard --help throws exception #24

Closed cgmb closed 6 years ago

cgmb commented 6 years ago

The use of % in argparse help strings needs to be escaped. I caught this the first time I wrote the help for --endif-style, but I went back and forth on what symbol to use, and it got missed in b7ac6709d34cbb6e8030f23c1cb385150cee3479.

This was an embarrassingly simple oversight. The unit tests for guardonce have been focused on the algorithms handling searching and substitution, but until yesterday there were no integration tests checking that they're glued together correctly. That has been corrected, and this will not be missed again. There's only a few integration tests at the moment, but they will be expanded upon as development continues.

cgmb commented 6 years ago

Fixed in 06b361b272cbd19aea57d0333bafb91af708d9ee.