diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2017-07-22 13:20:05 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-07-22 14:16:03 +0200 |
| commit | 1cf7a4c39e701023472a16467bd81aeacf5ff053 (patch) | |
| tree | 56cd40ccafddaf28738283abd4d8035f1fe8950b | |
| parent | 2b4cb88c029cc085e06effb32a2631033ccd8a50 (diff) | |
Typo: "disposeAndClear", not "clearAndDispose"
Change-Id: I9f7f30fce9324f7ccb79e9a8ef0c13e091d7c3a3
Reviewed-on: https://gerrit.libreoffice.org/40304
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
| -rw-r--r-- | compilerplugins/clang/unusedfields.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/compilerplugins/clang/unusedfields.cxx b/compilerplugins/clang/unusedfields.cxx index a775a979deb3..66c959d519a5 100644 --- a/compilerplugins/clang/unusedfields.cxx +++ b/compilerplugins/clang/unusedfields.cxx @@ -472,10 +472,10 @@ void UnusedFields::checkWriteOnly(const FieldDecl* fieldDecl, const Expr* member if (startswith(name, "read") || name.find(">>=") != std::string::npos) // this is a write-only call ; - else if (name == "clear" || name == "dispose" || name == "clearAndDispose" || name == "swap") + else if (name == "clear" || name == "dispose" || name == "disposeAndClear" || name == "swap") // we're abusing the write-only analysis here to look for fields which don't have anything useful // being done to them, so we're ignoring things like std::vector::clear, std::vector::swap, - // and VclPtr::clearAndDispose + // and VclPtr::disposeAndClear ; else bPotentiallyReadFrom = true; |