diff options
author | Jacob Segal <jacob.e.segal@gmail.com> | 2018-09-10 18:46:18 -0700 |
---|---|---|
committer | Jacob Segal <jacob.e.segal@gmail.com> | 2018-11-17 04:04:39 -0800 |
commit | 1b3fa9828c79a6ca53561945348832de66cbab7b (patch) | |
tree | 1eb888a8debc1206e3c13749711597f2f9e14341 /ale_linters/kotlin | |
parent | 531868f759404e11d1f34f72e19dcd6112a88567 (diff) | |
download | ale-1b3fa9828c79a6ca53561945348832de66cbab7b.zip |
Fix bug where last c flag was ignored
There is currently a check that tries to prevent c-flags that contain
'-' in them from being unintentionally split and included in the list of
commands. For example, we wouldn't want "-fno-exceptions " to appear as
"-fno" and "-exceptions ". The way this check was done was by making sure
the last character of the split string was a space.
This meant that the very last option to appear in the compile command
was ignored (as it doesn't end with a space). This fix explicitly skips
the ends-with-space check on the last option in the command-line.
This isn't the best fix. Really we should be using the same
argument-processing rules as a shell would rather than just splitting on
'-'. That's a much larger and more complicated change though.
Diffstat (limited to 'ale_linters/kotlin')
0 files changed, 0 insertions, 0 deletions