summaryrefslogtreecommitdiff
path: root/compilerplugins
AgeCommit message (Collapse)AuthorFilesLines
2013-07-24Adapt to PPCallbacks.h changes on Clang trunk towards 3.4Stephan Bergmann1-1/+7
Change-Id: I5611ffd36f9c6d1560a6ada6118a1efaeb6dc522
2013-07-23adjust for upstreaming of warn_unused attributeLuboš Luňák1-0/+9
The warn_unused attribute has been upstream to GCC and Clang, so use it if present. Still warn about STL types if those do not use it yet (which is the status as of now). Change-Id: I3c003e44c08d1d141e23bba38cf92e663a5ac353
2013-07-16mention the explicit bool conversion from C++11 for pointer-to-bool checkLuboš Luňák1-3/+4
Change-Id: I050019b30d2ec96bbd25916624f8cb88d72110e4
2013-07-15compiler plugin warning about unwanted silent pointer-to-bool conversionsLuboš Luňák1-0/+163
C/C++ silently converts pointers to booleans, which is presumably seen as a good idea by lazy people, but it can occassionally silently break code in strange ways, most notably by selecting a bool overload when no matching overload for the pointer exists (OUStringBuffer::append() can break like that, e.g. in ba37e4062f538db7e51d6a64ba544eeddbc567cf, other cases are 8e3bf1598fa95ac8d099e45ae4252e7654a6f590 or 28e4c0250e67a344b4d6088bdca2e680a4bffad0). So far the plugin checks only conversions in function call arguments, there's disabled code that could check more places, but I'm not aware so far of any place where anything actually broke because of that, and it'd require fixups to be explicit in some places (e.g. 'bool b = returns_pointer();' would require '!= NULL' added), so for it'll be only the simple various and it can be made more strict if wanted. Change-Id: I6a5d207daf925e6c2d1bf684060536795ecfcc35
2013-07-15fix finding all parents for AST nodesLuboš Luňák1-5/+17
Ctor bodies can also have code inside of member variables initialization, which is not considered to be inside function body. Change-Id: Id68960093a51396b9486f1364b1a361526c3431d
2013-06-20simplify bodynotinblock plugin using parentStmt()Luboš Luňák2-47/+43
Change-Id: Ia2fe10af2ca555c7b88348e7ed571f1176586580
2013-06-20simplify postfixincrementfix plugin using parentStmt()Luboš Luňák2-67/+27
Change-Id: I93fa422afe7f3e1e10576dd64af9d57b2302f44e
2013-06-20make it easy to get a parent of an AST nodeLuboš Luňák2-0/+74
Clang API doesn't provide this, but it's occasionally needed, and so far the way has been inspecting the highest possible node in AST and walking down and remembering, which is complicated, error-prone and annoying. Change-Id: Id5b72cb5ebfc069e90efe6d673c0ef18ebcdab61
2013-06-13work around the C++11 __float128 problem with libstdc++ headers and clangLuboš Luňák1-1/+1
Change-Id: Ia1b443d22b3b7f6f93f1ad8c5fa760b0f1da3b83
2013-06-05Ignore warnings in Clang headersStephan Bergmann1-1/+1
(as occur at least in trunk Clang builds) Change-Id: I7e74200ae1042133fa04477e39e46fd2f01cc1b0
2013-06-05Adapt UPDATE_FILES=<module> to headers being moved to include/Stephan Bergmann1-1/+12
Change-Id: I6af5236358a51a51514d646c809147db56b2464b
2013-06-02fix plugin build with clang < 3.3Luboš Luňák1-6/+6
Change-Id: If2eb7595ca7d25f6d491e332bb3060061d3b42a7
2013-05-31make compiler plugin warnings follow WaELuboš Luňák1-2/+0
Change-Id: I72f1a11ce86d66e773e66116c725310df77fce39
2013-05-31obey --enable-werror when building clang pluginLuboš Luňák1-1/+5
Change-Id: I8ca9b09a6ffd4b2f00740563fa9682fdabb26b3d
2013-05-31reoder stuff in makefileLuboš Luňák1-2/+2
At the top should be stuff to set manually if needed. Change-Id: I516ac6df8102f00f6fc34c4c87d040688fcc2e99
2013-05-31update for PPCallbacks API changes in Clang3.3Luboš Luňák2-2/+43
Change-Id: I8493afa8e2264a42fba090c9c72f36f36cb2d165
2013-05-31mark all virtuals in the clang plugin as overrideLuboš Luňák15-21/+21
In order to help catch Clang API changes. Change-Id: I5738f5dff405f7a2dd377c983fb32c1c7a2ff921
2013-05-22Use the new type-checking Reference constructor to reduce code noiseNoel Grandin2-0/+225
Also create a Clang compiler plugin to detect such cases. Change-Id: I61ad1a1d6b1c017eeb51f226d2dde0e9bb7f1752 Reviewed-on: https://gerrit.libreoffice.org/4001 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2013-05-06check for double modifications in compiler pluginsLuboš Luňák2-0/+17
A different way to do 1c0669af2f1f58e6431b5e489ac48a883e242ba7. Sometimes one piece of code can be represented several times in the AST, e.g. with default function arguments. Change-Id: Ic7799fa0bd918a638bdc8ebef69e6aa91d355bdc
2013-05-06compiler plugin for RTL_CONSTASCII_USTRINGPARAM removalLuboš Luňák1-0/+140
Change-Id: Ibc487aa78d9eba3f936b4f4cc0298adfaef60d5c
2013-05-06Revert "ignore already seen locations in compiler plugins"Luboš Luňák2-16/+6
This does not always work well, e.g. when building a return value in a return statement from a temporary, there is CXXConstructExpr containing CXXTemporaryObjectExpr, which both share the same location. This reverts commit 1c0669af2f1f58e6431b5e489ac48a883e242ba7.
2013-05-06ignore already seen locations in compiler pluginsLuboš Luňák2-6/+16
Change-Id: Icba8dfa61aee7237af569b2414b1cf4ce93ca476
2013-05-06Fix handling range in removeText().Luboš Luňák2-2/+15
Turns out removeText( SourceRange ) treats it as a token range, so it's not always character-exact if used for removal of only several characters from a token (e.g. an identifier). Change-Id: I0223d52da90f9535d9ef1d48b0f56d69131536c8
2013-05-02add constLuboš Luňák20-28/+28
Change-Id: I71236b9ca6300372ba00c85401cf19f6c0e7ac99
2013-04-24Update location of log-areas.doxTor Lillqvist1-3/+3
Change-Id: If54a3d7047f13ae9c9345c21737a89afee645403
2013-04-22base for unusedcode compiler pluginLuboš Luňák1-0/+75
Needs work to actually do something useful, but the basics are there. Change-Id: I193922f2f5572760c8c20def0f9b830138f47fef
2013-04-07removeforwardstringdecl clang plugin has been used, move itLuboš Luňák2-0/+0
Change-Id: I3bccfa773448e17ac3c8ece6b99f85c6d399b0cf
2013-04-04compiler plugin to check incorrect #ifdef for config_XXX.h macrosLuboš Luňák1-0/+100
http://lists.freedesktop.org/archives/libreoffice/2013-March/047769.html Change-Id: Ibba9d3dcc87d6d6eee58ab3690a8f87339d00956
2013-04-04don't explicitly delete PPCallbacks-based compiler plugin actionsLuboš Luňák1-1/+6
If the class is based on PPCallbacks too, Clang internals will delete the instance. Change-Id: I6dd83d800e6cca17eb0b5de23c8994f11c087fd5
2013-03-30silence compilerplugin buildPeter Foley1-3/+5
Change-Id: I4397575e4665e8a257d0bd3e15245d996a48a1f4
2013-03-28adjust unused compiler plugins for ASTContext to CompilerInstance change tooLuboš Luňák8-16/+16
Change-Id: I415ed25586408d7e7df9457f7c637a8c6d13d35d
2013-03-28pass around CompilerInstance rather than ASTContextLuboš Luňák15-65/+67
It's possible to get the latter from the former, and the former is useful for other things too (access to the preprocessor, for example). Change-Id: I708d709129fd3a35bf7c63da4de09c2e696b382d
2013-03-28-Wextra for compiler pluginsLuboš Luňák4-5/+5
Change-Id: I246dcf1d2c099c510435d21f0dd47e3c098456f2
2013-03-19A plugin to find suboptimal equalsIgnoreAsciiCaseAscii[L] callsStephan Bergmann1-0/+111
Change-Id: Id2572982ca899223b89016ee7482ccb960032805
2013-03-19Enable -std=c++11 for compilerplugins/clangStephan Bergmann1-1/+1
Change-Id: I4849656b16c2a7103217870050b0de3ccb97af56
2013-02-22s/the the/the/Tor Lillqvist2-2/+2
Change-Id: Iadacffaad832c6ff06757e8567e24f929f24a4c3
2013-02-11Fixed compilation error of Clang plugins on Ubuntu/MintMarc-André Laverdière1-0/+2
Change-Id: Ib08c5075034e0e16c5541f05425ef15d2cc3a6f6 Reviewed-on: https://gerrit.libreoffice.org/2045 Reviewed-by: Luboš Luňák <l.lunak@suse.cz> Tested-by: Luboš Luňák <l.lunak@suse.cz>
2013-02-09make it possible to limit what files will be modified by a compiler pluginLuboš Luňák3-25/+90
Change-Id: I4e3e8f5ca5e5b5b59b1bd067281f90940dd893b1
2013-02-09fix check whether a file is in a directoryLuboš Luňák1-4/+4
Make sure / is appended, otherwise /foo might match /foobar/file . Change-Id: I36469916b72b407c8f0c9c255099ee671039cf17
2013-02-09treat all source files as not up to date only with FORCE_COMPILE_ALLLuboš Luňák1-0/+2
So running the compiler plugin can be done either with FORCE_COMPILE_ALL=1 for simplicity, running the plugin on all source files for the target, or it's possible to use make's -W option (or touch files manually) to run it only for selected files. Change-Id: Ifa82bbe91dc1e179af9db8fbf99271bcf03a1d47
2013-02-07create an output dir that's already needed by $(shell ...)Luboš Luňák1-1/+2
Otherwise there's an error message e.g. during a clean build. Change-Id: I42efd08e014a4ebe25cfd1a497f622e5ea5c39ad
2013-02-07make sure the default target for compilerplugins is buildingLuboš Luňák1-1/+3
CLANGFORCE was the first in the makefile after c6ffe17631cccf11fbe00479d2169116d494a7da and therefore the default Change-Id: Id3b86a1ea2c27479067f7f55a977da3e6e8a2dfb
2013-02-02tutorial examples for writing new Clang plugin actionsLuboš Luňák9-1/+380
http://wiki.documentfoundation.org/Clang_plugins Change-Id: Ieb4fc186490e81ab961c094ca0a7fcdabc0f348f
2013-02-02a bit more reliable detection of removed/added sourcesLuboš Luňák1-0/+4
Change-Id: I7fdde46334c648ab6cba54210e31fada87dcfb55
2013-02-02remove not needed #includeLuboš Luňák9-18/+1
Change-Id: I0fa791733407199db5be2cc9606ac9be1da64188
2013-02-02append [loplugin] automatically in report() rathen than manually everywhereLuboš Luňák6-20/+21
Change-Id: I2f98622f152ae0c7ac8d1113d6380f686ac7234c
2013-02-02always use the report() helperLuboš Luňák5-29/+30
Change-Id: I2966fdb5bd98b1ddf718079584acf90a3e3a3700
2013-02-02move clang plugin that's no longer neededLuboš Luňák3-0/+3
But still keep it in case it'd be needed again, or simply as a starting reference. Change-Id: If519e1320e1bd6dce7746b57172324b33504051e
2013-02-02move documentation of plugins to the .cxx filesLuboš Luňák6-32/+28
It's mostly there already anyway, no need to duplicate it. Change-Id: I5b066f90725a064fb0746e1411900e835e3f66c3
2013-02-02automatically link all .cxx files in clang/ into the pluginLuboš Luňák1-12/+18
This means that just moving sources to the dir will enable the plugin action in that source, without modifying any sources, so those changes would not be accidentally committed when using a rewriter to change LO sources. Change-Id: Ic5a9c52dbf1939a1e78ad39ed6691ce3a1f399df
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-01 18:24:57 +0000