diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2015-02-12 16:11:04 +0100 |
---|---|---|
committer | Justin M. Keyes <justinkz@gmail.com> | 2015-02-18 20:54:13 -0500 |
commit | 94db26edbdce5a2c147329fc8e8da580216d2a1c (patch) | |
tree | e3e2065023698e2dc73bd1cb4ec07e92eeacd89c /clint.py | |
parent | 7dd48d7af08613255bc95b63f5b6b0f096a98d22 (diff) | |
download | rneovim-94db26edbdce5a2c147329fc8e8da580216d2a1c.tar.gz rneovim-94db26edbdce5a2c147329fc8e8da580216d2a1c.tar.bz2 rneovim-94db26edbdce5a2c147329fc8e8da580216d2a1c.zip |
Enable -Wconversion: indent.c.
Note: Clint was failing because of recommending not to use long. But
converting to long is the proper refactoring here, in as far as other
longs exist. We could, then, disable clint rule, or remove this file
from checking. We choose the former, as it's being discussed what to do
with longs, but a decision has not been taken. So, it seems most
reasonable to allow longs for now, to enable proper refactorings, and
then, when a decision is taken, refactor all longs to some other thing.
Diffstat (limited to 'clint.py')
-rwxr-xr-x | clint.py | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -2776,7 +2776,7 @@ def CheckLanguage(filename, clean_lines, linenum, file_extension, # TODO(unknown): figure out if they're using default arguments in fn proto. # Check if people are using the verboten C basic types. - match = Search(r'\b(short|long(?! +double)|long long)\b', line) + match = Search(r'\b(short|long long)\b', line) if match: error(filename, linenum, 'runtime/int', 4, 'Use int16_t/int64_t/etc, rather than the C type %s' |