diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-04-24 21:55:39 +0200 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2018-04-25 14:27:01 +0200 |
| commit | 90c91b824c2b362b43bdd2f8d8d647867d8fe612 (patch) | |
| tree | 243cff7af1b6301b987f36e2f63f80c5ceaece2d /bin | |
| parent | c72d69d4b0d28497bc6f70b085dfdf2fbc672cbb (diff) | |
sw: fix remaining IWYU warnings in inc/*.hxx
Also check for not needed forward declarations.
Change-Id: I92759f3f40d9458fd192665b39b87a78d8b97e5a
Reviewed-on: https://gerrit.libreoffice.org/53418
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Diffstat (limited to 'bin')
| -rwxr-xr-x | bin/find-unneeded-includes | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/bin/find-unneeded-includes b/bin/find-unneeded-includes index de91a3570231..7cc933ff6c9a 100755 --- a/bin/find-unneeded-includes +++ b/bin/find-unneeded-includes @@ -145,8 +145,15 @@ def processIWYUOutput(iwyuOutput, moduleRules): toRemove.append("%s:%s: %s" % (currentFileName, lineno, include)) continue + match = re.match("- (.*); // lines (.*)-.*", line) + if match: + fwdDecl = match.group(1) + lineno = match.group(2) + if not ignoreRemoval(fwdDecl, toAdd, currentFileName, moduleRules): + toRemove.append("%s:%s: %s" % (currentFileName, lineno, fwdDecl)) + for remove in toRemove: - print("ERROR: %s: remove not needed include" % remove) + print("ERROR: %s: remove not needed include / forward declaration" % remove) return len(toRemove) |