Review Request 126782: Fix almost all the remaining Clazy warnings in KDevPlatform
Sergey Popov
sergobot at protonmail.com
Mon Jan 18 17:21:26 UTC 2016
> On Янв. 18, 2016, 5:03 п.п., Aleix Pol Gonzalez wrote:
> > plugins/cvs/cvsplugin.cpp, line 466
> > <https://git.reviewboard.kde.org/r/126782/diff/1-2/?file=433719#file433719line466>
> >
> > `QString()`? Or is it a QByteArray?
Yes, that should be QString()
- Sergey
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126782/#review91275
-----------------------------------------------------------
On Янв. 18, 2016, 5:20 п.п., Sergey Popov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126782/
> -----------------------------------------------------------
>
> (Updated Янв. 18, 2016, 5:20 п.п.)
>
>
> Review request for KDevelop and Aleix Pol Gonzalez.
>
>
> Repository: kdevplatform
>
>
> Description
> -------
>
> I fixed almost all the remaining warnings in KDevPlatform but there are some warnings, which are still not fixed. "Missing Q_OBJECT macro" in some files, generated while building, for example.
>
> Related GCI-2015 task: [https://codein.withgoogle.com/tasks/5200084669038592/](https://codein.withgoogle.com/tasks/5200084669038592/)
>
>
> Diffs
> -----
>
> debugger/breakpoint/breakpointdetails.cpp 1f5a564
> debugger/breakpoint/breakpointwidget.cpp 30a7a4c
> debugger/util/treeitem.cpp c58798f
> debugger/variable/variablecollection.h 2520944
> debugger/variable/variablecollection.cpp 2ade907
> debugger/variable/variabletooltip.cpp ce95f46
> interfaces/contextmenuextension.cpp 191e183
> interfaces/ipartcontroller.cpp 5948371
> interfaces/iplugin.cpp 76da996
> interfaces/isourceformatter.cpp 7a295ca
> language/assistant/renameaction.cpp 8d1c383
> language/assistant/renameassistant.cpp 62f58a0
> language/backgroundparser/backgroundparser.cpp affd601
> language/backgroundparser/documentchangetracker.cpp cbb6359
> language/backgroundparser/parsejob.cpp b38ae36
> language/backgroundparser/parseprojectjob.cpp 1417ac4
> language/backgroundparser/tests/test_backgroundparser.cpp e91182c
> language/backgroundparser/tests/testlanguagesupport.cpp 8b8109b
> language/classmodel/classmodel.cpp 74e4f02
> language/classmodel/classmodelnode.cpp 9d7faab
> language/classmodel/documentclassesfolder.cpp 0d8750c
> language/classmodel/projectfolder.cpp 613529a
> language/codecompletion/codecompletion.cpp c90da5c
> language/codecompletion/codecompletionhelper.cpp f43b25e
> language/codecompletion/codecompletionmodel.cpp aa9c242
> language/codecompletion/normaldeclarationcompletionitem.cpp 09f5342
> language/codegen/applychangeswidget.cpp 6fa12f9
> language/codegen/basicrefactoring.h 9017da5
> language/codegen/basicrefactoring.cpp 57f1b96
> language/codegen/codedescription.cpp 68a5f39
> language/codegen/codedescriptionmetatypes.h ff4bfe4
> language/codegen/coderepresentation.cpp 9652717
> language/codegen/documentchangeset.cpp 1c38a1c
> language/codegen/sourcefiletemplate.cpp 1707d55
> language/codegen/templateclassgenerator.cpp 85dbf36
> language/codegen/templateengine.cpp 1a7adc0
> language/codegen/templaterenderer.cpp 8a6e7ac
> language/codegen/templatesmodel.cpp 125daa9
> language/codegen/tests/test_documentchangeset.cpp baca701
> language/codegen/tests/test_templateclassgenerator.cpp d8136b2
> language/codegen/tests/test_templaterenderer.cpp 86e6288
> language/codegen/tests/test_templatesmodel.cpp 64a01e9
> language/codegen/utilities.h 73033e7
> language/duchain/classdeclaration.cpp ea57238
> language/duchain/declaration.cpp 0d0726f
> language/duchain/declarationid.cpp 206c445
> language/duchain/definitions.cpp 6029d34
> language/duchain/duchain.cpp d0f1e4e
> language/duchain/duchaindumper.cpp 463c1f6
> language/duchain/duchainutils.cpp 7096e05
> language/duchain/ducontext.cpp 0b8fca2
> language/duchain/dumpdotgraph.cpp c3492b4
> language/duchain/functiondefinition.cpp d62f0aa
> language/duchain/identifier.cpp 2e40eb4
> language/duchain/importers.cpp 3c84eff
> language/duchain/instantiationinformation.cpp 1ce3d0b
> language/duchain/navigation/abstractdeclarationnavigationcontext.h 2356416
> language/duchain/navigation/abstractdeclarationnavigationcontext.cpp c2983d8
> language/duchain/navigation/abstractincludenavigationcontext.h ffd84e0
> language/duchain/navigation/abstractincludenavigationcontext.cpp 836d7a7
> language/duchain/navigation/abstractnavigationcontext.cpp 905ac97
> language/duchain/navigation/abstractnavigationwidget.cpp b32ff5b
> language/duchain/navigation/problemnavigationcontext.h ffd71af
> language/duchain/navigation/problemnavigationcontext.cpp fa57987
> language/duchain/navigation/usescollector.cpp 120da9d
> language/duchain/navigation/usesnavigationcontext.h bab42c0
> language/duchain/navigation/usesnavigationcontext.cpp fd0a70a
> language/duchain/navigation/useswidget.cpp 2cb96c2
> language/duchain/parsingenvironment.cpp 4252775
> language/duchain/persistentsymboltable.cpp 21f39bb
> language/duchain/problem.cpp 4c647af
> language/duchain/tests/test_duchain.cpp a3f9721
> language/duchain/tests/test_duchainshutdown.cpp ea090e1
> language/duchain/tests/test_identifier.cpp b5de64c
> language/duchain/topducontext.cpp f3a9a5f
> language/duchain/types/abstracttype.cpp c764fc6
> language/duchain/types/constantintegraltype.cpp 1b75371
> language/duchain/types/functiontype.cpp cfe4044
> language/duchain/types/integraltype.cpp 15fc8a8
> language/duchain/types/pointertype.cpp 1e9fe88
> language/duchain/types/referencetype.cpp 12715ca
> language/duchain/types/structuretype.cpp d6f85f0
> language/duchain/types/typealiastype.cpp d3cc9b8
> language/duchain/types/typerepository.cpp dc81ba3
> language/duchain/types/unsuretype.cpp b3e14bd
> language/duchain/uses.cpp f4a5600
> language/editor/modificationrevisionset.cpp af23f32
> language/highlighting/codehighlighting.cpp d674741
> language/highlighting/colorcache.cpp fa9c7d5
> language/highlighting/configurablecolors.cpp 964eb9f
> language/interfaces/quickopendataprovider.cpp 415574b
> language/interfaces/quickopenfilter.h 0110008
> language/util/setrepository.cpp 14497dc
> outputview/outputdelegate.h 6fedf5f
> outputview/outputexecutejob.cpp 2fd80fd
> outputview/outputfilteringstrategies.cpp 4fdb684
> outputview/tests/test_filteringstrategy.cpp 63409ac
> outputview/tests/test_outputmodel.cpp 7125f05
> outputview/tests/testlinebuilderfunctions.h 40266dc
> plugins/appwizard/appwizardplugin.cpp 87c9a65
> plugins/appwizard/projectselectionpage.cpp 949dd04
> plugins/appwizard/projecttemplatesmodel.h 4bb46d1
> plugins/appwizard/projecttemplatesmodel.cpp d9bb7df
> plugins/appwizard/projectvcspage.cpp d411320
> plugins/bazaar/bazaarplugin.cpp a10a33c
> plugins/bazaar/bazaarutils.cpp edf13aa
> plugins/bazaar/bzrannotatejob.cpp b668563
> plugins/classbrowser/classbrowserplugin.cpp 67d07b3
> plugins/classbrowser/classwidget.cpp a471fa0
> plugins/codeutils/codeutilsplugin.cpp c1e793c
> plugins/contextbrowser/browsemanager.h cbe22bd
> plugins/contextbrowser/browsemanager.cpp c23b428
> plugins/contextbrowser/contextbrowser.cpp 5bc9a6a
> plugins/contextbrowser/contextbrowserview.cpp 57c966f
> plugins/cvs/checkoutdialog.cpp 421f5ad
> plugins/cvs/cvsannotatejob.cpp 8ba726e
> plugins/cvs/cvsdiffjob.cpp 927ba3b
> plugins/cvs/cvsjob.cpp 68f9909
> plugins/cvs/cvsmainview.cpp 4fe0c4f
> plugins/cvs/cvsplugin.cpp 356eb85
> plugins/cvs/cvsproxy.h 7792938
> plugins/cvs/cvsproxy.cpp d6810c4
> plugins/cvs/cvsstatusjob.cpp 11d6e10
> plugins/cvs/editorsview.cpp bb1c6be
> plugins/cvs/importdialog.cpp 23cd9f3
> plugins/documentswitcher/documentswitcherplugin.h 7f7e8af
> plugins/documentswitcher/documentswitcherplugin.cpp 938264a
> plugins/documentswitcher/documentswitchertreeview.h ea349e8
> plugins/documentview/kdevdocumentview.cpp 1fa8156
> plugins/documentview/kdevdocumentviewplugin.cpp abbcbef
> plugins/execute/executeplugin.cpp 770e33e
> plugins/execute/nativeappconfig.cpp a47dff7
> plugins/execute/nativeappjob.cpp 20f97c6
> plugins/execute/projecttargetscombobox.cpp a4d962c
> plugins/executescript/executescriptplugin.cpp d62533e
> plugins/executescript/scriptappconfig.h 9a044fa
> plugins/executescript/scriptappconfig.cpp d4d6c8e
> plugins/executescript/scriptappjob.cpp 7dd2420
> plugins/externalscript/externalscriptjob.cpp 06e4830
> plugins/externalscript/externalscriptplugin.cpp 6838522
> plugins/externalscript/externalscriptview.cpp ee717a6
> plugins/filemanager/kdevfilemanagerplugin.cpp a39f71c
> plugins/filetemplates/classmemberspage.cpp dcd7ca6
> plugins/filetemplates/filetemplatesplugin.cpp 4c734b1
> plugins/filetemplates/licensepage.cpp ddd5829
> plugins/filetemplates/main.cpp 058510f
> plugins/filetemplates/outputpage.cpp aaac553
> plugins/filetemplates/overridespage.cpp 5e39627
> plugins/filetemplates/templateclassassistant.cpp 08309d1
> plugins/filetemplates/templateoptionspage.cpp 2d95f33
> plugins/filetemplates/templatepreview.cpp 705964b
> plugins/filetemplates/templateselectionpage.cpp 70db5e2
> plugins/filetemplates/tests/test_generationtest.cpp 04cb4d6
> plugins/genericprojectmanager/test/test_projectload.cpp 65a4782
> plugins/git/gitjob.h 22fd444
> plugins/git/gitplugin.cpp f06ef4e
> plugins/git/stashmanagerdialog.cpp 308d36c
> plugins/git/tests/test_git.cpp c80a1d3
> plugins/grepview/grepdialog.cpp ee212f1
> plugins/grepview/grepoutputdelegate.h f9a1a64
> plugins/grepview/grepoutputmodel.cpp 47bac97
> plugins/grepview/grepoutputview.cpp cc25669
> plugins/grepview/greputil.cpp 0447a26
> plugins/grepview/grepviewplugin.cpp b59217a
> plugins/grepview/tests/test_findreplace.cpp 3b080d4
> plugins/konsole/kdevkonsoleviewplugin.cpp 6908a7b
> plugins/openwith/openwithplugin.cpp 7603b5a
> plugins/outlineview/outlinenode.cpp 4e722c9
> plugins/outlineview/outlinewidget.cpp 2bc82c4
> plugins/patchreview/localpatchsource.cpp 006aab2
> plugins/patchreview/patchhighlighter.cpp 0d3a757
> plugins/patchreview/patchreview.cpp ef13c0d
> plugins/patchreview/patchreviewtoolview.cpp cbf8320
> plugins/problemreporter/problemhighlighter.cpp 65b3aee
> plugins/problemreporter/problemnavigationcontext.h 5c6a520
> plugins/problemreporter/problemnavigationcontext.cpp db38d42
> plugins/problemreporter/problemreporterplugin.cpp c9ebbb3
> plugins/problemreporter/problemsview.cpp 659ff6d
> plugins/problemreporter/problemtreeview.cpp 59f6166
> plugins/projectfilter/tests/test_projectfilter.cpp 6da4222
> plugins/projectmanagerview/projectbuildsetwidget.cpp 7108a92
> plugins/projectmanagerview/projectmanagerview.cpp 0a8f701
> plugins/projectmanagerview/projectmanagerviewplugin.cpp bdd5dd3
> plugins/projectmanagerview/projectmodelsaver.cpp 3d74426
> plugins/projectmanagerview/projecttreeview.cpp 81ba3ab
> plugins/projectmanagerview/vcsoverlayproxymodel.cpp 4ee9987
> plugins/quickopen/actionsquickopenprovider.h 85d5e9e
> plugins/quickopen/actionsquickopenprovider.cpp ec38c4d
> plugins/quickopen/declarationlistquickopen.h edeab56
> plugins/quickopen/documentationquickopenprovider.h cd96225
> plugins/quickopen/duchainitemquickopen.h 5de72ae
> plugins/quickopen/duchainitemquickopen.cpp f915f6f
> plugins/quickopen/expandingtree/expandingdelegate.cpp 4be0b26
> plugins/quickopen/expandingtree/expandingtree.h 5bd7c97
> plugins/quickopen/expandingtree/expandingwidgetmodel.cpp 34be20b
> plugins/quickopen/projectfilequickopen.cpp d9e14c2
> plugins/quickopen/projectitemquickopen.h fa1a834
> plugins/quickopen/projectitemquickopen.cpp a9a57f5
> plugins/quickopen/quickopenplugin.h 8818d0c
> plugins/quickopen/quickopenplugin.cpp 509623d
> plugins/quickopen/tests/quickopentestbase.cpp 0b0565c
> plugins/quickopen/tests/test_quickopen.cpp 333fbba
> plugins/standardoutputview/outputwidget.cpp 45d65e5
> plugins/standardoutputview/tests/test_standardoutputview.cpp 7522901
> plugins/subversion/kdevsvnplugin.cpp 0287bd7
> plugins/subversion/svnclient.cpp d090551
> plugins/subversion/svncommitjob.cpp b595cd3
> plugins/subversion/svndiffjob.cpp f2e5a53
> plugins/subversion/svninternaljobbase.cpp 5758194
> plugins/subversion/svnjobbase.cpp d3c752c
> plugins/subversion/svnlocationwidget.h 983d2a2
> plugins/subversion/svnssldialog.cpp b654d7c
> plugins/subversion/tests/svnimport.cpp 6fd5c80
> plugins/subversion/tests/svnrecursiveadd.cpp 947f0e6
> plugins/switchtobuddy/switchtobuddyplugin.cpp 3678a45
> plugins/testview/testview.cpp ef8a3ef
> plugins/testview/testviewplugin.cpp 67d200e
> plugins/vcschangesview/vcschangesviewplugin.cpp d55b80a
> plugins/welcomepage/welcomepagedocument.cpp ec7759a
> plugins/welcomepage/welcomepageplugin.cpp 3032f8e
> plugins/welcomepage/welcomepageview.h 4a0f264
> plugins/welcomepage/welcomepageview.cpp 0fbd31d
> project/abstractfilemanagerplugin.cpp feb104a
> project/projectmodel.cpp 17bf418
> project/tests/projectmodelperformancetest.cpp 6adcc83
> project/tests/test_projectmodel.cpp d73f4fd
> serialization/tests/test_indexedstring.cpp 4c424c1
> serialization/tests/test_itemrepository.cpp 4a09417
> shell/assistantpopup.cpp efd8249
> template/filters/kdevfilters.cpp 175726d
> tests/json/delayedoutput.cpp 1483d4f
> tests/json/jsondeclarationtests.h fed47b1
> tests/json/jsonducontexttests.h 11dbbe5
> tests/json/jsontesthelpers.h 564c5a7
> tests/json/jsontypetests.h 6934069
> tests/json/testsuite.h b28708a
> tests/testhelpers.h 1bb7242
> tests/testproject.h 7f37619
> tests/testproject.cpp 4032110
> util/duchainify/main.cpp 452e918
> util/environmentgrouplist.cpp e2e100b
> util/formattinghelpers.h c25bba8
> util/kdevstringhandler.cpp fbb5f64
> util/path.cpp b646fef
> util/shellutils.cpp 34dfead
> util/tests/test_embeddedfreetree.cpp 08e4a40
> util/tests/test_environment.cpp c20200d
> util/tests/test_path.cpp d2a8781
> vcs/dvcs/dvcsjob.cpp fb456bb
> vcs/dvcs/tests/test_dvcsjob.cpp 2b19958
> vcs/models/brancheslistmodel.cpp c78d29a
> vcs/models/tests/test_models.cpp d3f7111
> vcs/vcsrevision.cpp e06abe5
> vcs/widgets/vcsdiffpatchsources.h 8668480
> vcs/widgets/vcsdiffpatchsources.cpp a77bf13
>
> Diff: https://git.reviewboard.kde.org/r/126782/diff/
>
>
> Testing
> -------
>
> Suceessfuly built.
> Tests passed.
>
>
> Thanks,
>
> Sergey Popov
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160118/21dd55a8/attachment-0001.html>
More information about the KDevelop-devel
mailing list