| Commit message (Collapse) | Author | Age |
|
|
|
| |
And re-run IWYU, adjust #includes.
|
| |
|
|
|
|
|
|
|
|
| |
The autoconf-generated config.h contains a number of directives which
may alter the behaviour of system headers on certain platforms. Always
include it in every C++ file as the first include.
Closes #2993.
|
|
|
|
|
| |
This removes some pointless parentheses but the primary focus is removing
redundancies like unnecessary "else" clauses.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I missed restyling a few "switch" blocks to make them consistent with the rest
of the code base. This fixes that oversight. This should be the final step in
restyling the C++ code to have a consistent style. This also includes a few
trivial cleanups elsewhere.
I also missed restyling the "complete" module when working my way from a to z
so this final change includes restyling that module.
Total lint errors decreased 36%. Cppcheck errors went from 47 to 24. Oclint P2
errors went from 819 to 778. Oclint P3 errors went from 3252 to 1842.
Resolves #2902.
|
|
|
|
|
|
|
|
|
| |
For this change I decided to bundle the remaining modules that need to be
resytyled because only two were large enough to warrant doing on their own.
Reduces lint errors from 225 to 162 (-28%). Line count from 3073 to 2465 (-20%).
Another step in resolving issue #2902.
|
|
|
|
|
|
|
|
|
| |
Remove the "make iwyu" build target. Move the functionality into the
recently introduced lint.fish script. Fix a lot, but not all, of the
include-what-you-use errors. Specifically, it fixes all of the IWYU errors
on my OS X server but only removes some of them on my Ubuntu 14.04 server.
Fixes #2957
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Prior to this fix, when completing a command that doesn't have a /, we
would prepend each component of PATH and then expand the whole thing. So
any special characters in the PATH would be interpreted when performing
tab completion.
With this fix, we pull the PATH resolution out of complete.cpp and
migrate it to expand.cpp. This unifies nicely with the CDPATH resolution
already in that file. This requires introducing a new expand flag
EXPAND_SPECIAL_FOR_COMMAND, which is analogous to EXPAND_SPECIAL_CD
(which is renamed to EXPAND_SPECIAL_FOR_CD). This flag tells expand to
resolve paths against PATH instead of the working directory.
Fixes #952
|
|
|
|
| |
Noticed by cppcheck
|
| |
|
| |
|
| |
|
|
|
|
|
| |
When expanding an executable with $PATH, don't attempt to
interpret the directories in PATH as fuzzy matching. Fixes #2413.
|
| |
|
|
|
|
|
| |
When ascending out of a directory, we need to clear the directory
from the visited set. Fixes #2414.
|
|
|
|
| |
Fixes the wrong sense in a waccess test.
|
|
|
|
| |
Eliminates some stat calls, speeding up wildcard expansion
|
| |
|
|
|
|
| |
e.g. /u/l/b<tab> may be expanded to /usr/local/bin
|
|
|
|
|
| |
Preparation for zsh-style intermediate component
tab completion
|
|
|
|
| |
Preparation for zsh-style intermediate fuzzy matching
|
| |
|
| |
|
|
|
|
| |
Simplifies code yet further
|
| |
|
|
|
|
| |
This is yet clearer
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
In a few places, we need to add a prefix to completions that
replace the token. This change factors that logic into its
own function prepend_token_prefix.
|
| |
|
| |
|
| |
|
|
|
|
| |
Not yet turned on.
|
|\
| |
| |
| | |
http://include-what-you-use.org/
|
| |
| |
| |
| |
| | |
Replace uses of vector<completion_t>& with vector<completion_t>*
This makes it clear at the call site that the object may be mutated.
|
|/ |
|
|
|
|
|
|
|
|
| |
This change moves source files into a src/ directory,
and puts object files into an obj/ directory. The Makefile
and xcode project are updated accordingly.
Fixes #1866
|
|
This change moves source files into a src/ directory,
and puts object files into an obj/ directory. The Makefile
and xcode project are updated accordingly.
Fixes #1866
|