diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-06-29 12:25:02 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-02 08:47:48 +0200 |
commit | d98f1e4e0373782ad71b945dcc92c1c3d6dcf6c8 (patch) | |
tree | a39deddf841d478d7baa5e96829891447383a597 /compilerplugins/clang | |
parent | 683fa7b68673c8c6563ee493b8b5682bb6fd1b8a (diff) |
loplugin:unusedfields improve checking for write-only fields
we trade off a little accuracy for finding more possible write-only
fields.
Change-Id: I9f7edba99481fe4ded0a9d8e45e911b0ee99d269
Reviewed-on: https://gerrit.libreoffice.org/56715
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'compilerplugins/clang')
5 files changed, 126 insertions, 104 deletions
diff --git a/compilerplugins/clang/unusedfields.cxx b/compilerplugins/clang/unusedfields.cxx index 159d8544b635..8dca2f549933 100644 --- a/compilerplugins/clang/unusedfields.cxx +++ b/compilerplugins/clang/unusedfields.cxx @@ -524,13 +524,16 @@ void UnusedFields::checkWriteOnly(const FieldDecl* fieldDecl, const Expr* member } else if (op == UO_AddrOf || op == UO_Deref || op == UO_Plus || op == UO_Minus - || op == UO_Not || op == UO_LNot - || op == UO_PreInc || op == UO_PostInc - || op == UO_PreDec || op == UO_PostDec) + || op == UO_Not || op == UO_LNot) { bPotentiallyReadFrom = true; break; } + /* The following are technically reads, but from a code-sense they're more of a write/modify, so + ignore them to find interesting fields that only modified, not usefully read: + UO_PreInc / UO_PostInc / UO_PreDec / UO_PostDec + But we still walk up in case the result of the expression is used in a read sense. + */ walkupUp(); } else if (auto caseStmt = dyn_cast<CaseStmt>(parent)) @@ -651,7 +654,14 @@ void UnusedFields::checkWriteOnly(const FieldDecl* fieldDecl, const Expr* member MyFieldInfo fieldInfo = niceName(fieldDecl); if (bPotentiallyReadFrom) + { readFromSet.insert(fieldInfo); + if (fieldInfo.fieldName == "nNextElementNumber") + { + parent->dump(); + memberExpr->dump(); + } + } } void UnusedFields::checkReadOnly(const FieldDecl* fieldDecl, const Expr* memberExpr) diff --git a/compilerplugins/clang/unusedfields.only-used-in-constructor.results b/compilerplugins/clang/unusedfields.only-used-in-constructor.results index 4a6acd1bc740..f6a684c5ea75 100644 --- a/compilerplugins/clang/unusedfields.only-used-in-constructor.results +++ b/compilerplugins/clang/unusedfields.only-used-in-constructor.results @@ -196,7 +196,7 @@ include/LibreOfficeKit/LibreOfficeKitGtk.h:33 _LOKDocView aDrawingArea GtkDrawingArea include/LibreOfficeKit/LibreOfficeKitGtk.h:38 _LOKDocViewClass parent_class GtkDrawingAreaClass -include/oox/export/shapes.hxx:122 +include/oox/export/shapes.hxx:123 oox::drawingml::ShapeExport maShapeMap oox::drawingml::ShapeExport::ShapeHashMap include/registry/registry.hxx:35 Registry_Api acquire void (*)(RegHandle) @@ -216,8 +216,6 @@ include/svx/itemwin.hxx:34 SvxLineBox aDelayTimer class Timer include/xmloff/shapeimport.hxx:140 SdXML3DLightContext mbSpecular _Bool -include/xmloff/XMLTextMasterPageContext.hxx:34 - XMLTextMasterPageContext sIsPhysical const class rtl::OUString jvmfwk/plugins/sunmajor/pluginlib/sunversion.hxx:102 jfw_plugin::SunVersion usVersion class rtl::OUString libreofficekit/qa/gtktiledviewer/gtv-application-window.cxx:35 @@ -314,15 +312,15 @@ sc/inc/compiler.hxx:264 ScCompiler::AddInMap pUpper const char * sc/inc/formulalogger.hxx:42 sc::FormulaLogger maMessages std::vector<OUString> -sc/inc/interpretercontext.hxx:23 +sc/inc/interpretercontext.hxx:31 ScInterpreterContext mrDoc const class ScDocument & -sc/inc/token.hxx:399 +sc/inc/token.hxx:397 SingleDoubleRefModifier aDub struct ScComplexRefData sc/qa/unit/ucalc_column.cxx:104 aInputs aName const char * -sc/source/core/data/document.cxx:1248 +sc/source/core/data/document.cxx:1251 (anonymous namespace)::BroadcastRecalcOnRefMoveHandler aSwitch sc::AutoCalcSwitch -sc/source/core/data/document.cxx:1249 +sc/source/core/data/document.cxx:1252 (anonymous namespace)::BroadcastRecalcOnRefMoveHandler aBulk class ScBulkBroadcast sc/source/filter/inc/htmlpars.hxx:614 ScHTMLQueryParser mnUnusedId ScHTMLTableId @@ -372,8 +370,6 @@ sc/source/ui/inc/msgpool.hxx:46 ScMessagePool aCondFormatDlgItem class ScCondFormatDlgItem sc/source/ui/inc/notemark.hxx:43 ScNoteMarker m_aTimer class Timer -sc/source/ui/inc/sortkeydlg.hxx:46 - ScSortKeyWindow m_nItemHeight sal_Int32 sccomp/source/solver/DifferentialEvolution.hxx:35 DifferentialEvolutionAlgorithm maRandomDevice std::random_device sccomp/source/solver/ParticelSwarmOptimization.hxx:56 @@ -394,7 +390,7 @@ sd/source/ui/remotecontrol/ZeroconfService.hxx:36 sd::ZeroconfService port uint sd/source/ui/table/TableDesignPane.hxx:106 sd::TableDesignPane aImpl class sd::TableDesignWidget -sd/source/ui/view/DocumentRenderer.cxx:1321 +sd/source/ui/view/DocumentRenderer.cxx:1322 sd::DocumentRenderer::Implementation mxObjectShell SfxObjectShellRef sd/source/ui/view/viewshel.cxx:1215 sd::KeepSlideSorterInSyncWithPageChanges m_aDrawLock sd::slidesorter::view::class SlideSorterView::DrawLock @@ -472,12 +468,10 @@ svx/source/customshapes/EnhancedCustomShapeFunctionParser.cxx:1091 (anonymous namespace)::ExpressionGrammar::definition modifierReference ::boost::spirit::rule<ScannerT> svx/source/customshapes/EnhancedCustomShapeFunctionParser.cxx:1092 (anonymous namespace)::ExpressionGrammar::definition identifier ::boost::spirit::rule<ScannerT> -svx/source/dialog/framelinkarray.cxx:377 +svx/source/dialog/framelinkarray.cxx:376 svx::frame::MergedCellIterator mnFirstRow size_t svx/source/dialog/imapwnd.hxx:78 IMapWindow maItemInfos struct SfxItemInfo [1] -svx/source/dialog/orienthelper.cxx:36 - svx::OrientationHelper_Impl mrCtrlDial class svx::DialControl & svx/source/gallery2/galbrws2.cxx:116 (anonymous namespace)::GalleryThemePopup maBuilder class VclBuilder svx/source/stbctrls/pszctrl.cxx:93 @@ -488,15 +482,13 @@ svx/source/stbctrls/zoomctrl.cxx:55 ZoomPopup_Impl m_aBuilder class VclBuilder svx/source/svdraw/svdcrtv.cxx:49 ImplConnectMarkerOverlay maObjects sdr::overlay::OverlayObjectList -svx/source/svdraw/svdopath.cxx:93 - ImpSdrPathDragData bIsPrevControl _Bool svx/source/xml/xmleohlp.cxx:71 OutputStorageWrapper_Impl aTempFile class utl::TempFile sw/inc/unosett.hxx:144 SwXNumberingRules m_pImpl ::sw::UnoImplPtr<Impl> sw/qa/core/test_ToxTextGenerator.cxx:134 ToxTextGeneratorWithMockedChapterField mChapterFieldType class SwChapterFieldType -sw/qa/extras/uiwriter/uiwriter.cxx:3995 +sw/qa/extras/uiwriter/uiwriter.cxx:4001 IdleTask maIdle class Idle sw/source/core/crsr/crbm.cxx:64 (anonymous namespace)::CursorStateHelper m_aSaveState class SwCursorSaveState @@ -506,7 +498,7 @@ sw/source/core/layout/dbg_lay.cxx:169 SwImplEnterLeave nAction enum DbgAction sw/source/core/text/inftxt.hxx:683 SwTextSlot aText class rtl::OUString -sw/source/core/text/porfld.cxx:142 +sw/source/core/text/porfld.cxx:141 SwFieldSlot aText class rtl::OUString sw/source/filter/html/htmlcss1.cxx:77 SwCSS1ItemIds nFormatBreak sal_uInt16 @@ -522,7 +514,7 @@ sw/source/uibase/inc/chrdlg.hxx:43 SwCharDlg m_nCharUrlId sal_uInt16 sw/source/uibase/inc/chrdlg.hxx:45 SwCharDlg m_nCharBrdId sal_uInt16 -sw/source/uibase/inc/cption.hxx:56 +sw/source/uibase/inc/cption.hxx:44 SwCaptionDialog m_aTextFilter class TextFilterAutoConvert sw/source/uibase/inc/glossary.hxx:95 SwGlossaryDlg m_aNoSpaceFilter class TextFilter @@ -530,6 +522,8 @@ sw/source/uibase/inc/olmenu.hxx:77 SwSpellPopup m_aBuilder class VclBuilder sw/source/uibase/inc/olmenu.hxx:86 SwSpellPopup m_nLangParaMenuId sal_uInt16 +sw/source/uibase/inc/optload.hxx:186 + SwCaptionOptPage m_aTextFilter class TextFilterAutoConvert sw/source/uibase/inc/regionsw.hxx:256 SwInsertSectionTabDialog m_nNotePageId sal_uInt16 sw/source/uibase/inc/regionsw.hxx:276 @@ -564,6 +558,12 @@ vcl/inc/opengl/RenderList.hxx:30 Vertex lineData glm::vec4 vcl/inc/opengl/zone.hxx:46 OpenGLVCLContextZone aZone class OpenGLZone +vcl/inc/qt5/Qt5Menu.hxx:74 + Qt5MenuItem mbVisible _Bool +vcl/inc/qt5/Qt5Object.hxx:34 + Qt5Object m_aSystemData struct SystemEnvData +vcl/inc/qt5/Qt5Object.hxx:36 + Qt5Object m_pParent class Qt5Frame * vcl/inc/salmenu.hxx:42 SalMenuButtonItem mnId sal_uInt16 vcl/inc/salmenu.hxx:43 @@ -584,15 +584,13 @@ vcl/inc/unx/i18n_ic.hxx:46 SalI18N_InputContext maSwitchIMCallback XIMCallback vcl/inc/unx/i18n_ic.hxx:47 SalI18N_InputContext maDestroyCallback XIMCallback -vcl/inc/unx/i18n_xkb.hxx:32 - SalI18N_KeyboardExtension mnErrorBase int -vcl/source/app/salvtables.cxx:1249 +vcl/source/app/salvtables.cxx:1344 SalInstanceEntry m_aTextFilter class SalInstanceEntry::WeldTextFilter vcl/source/gdi/jobset.cxx:34 ImplOldJobSetupData cDeviceName char [32] vcl/source/gdi/jobset.cxx:35 ImplOldJobSetupData cPortName char [32] -vcl/unx/gtk3/gtk3gtkinst.cxx:2166 +vcl/unx/gtk3/gtk3gtkinst.cxx:2221 CrippledViewport viewport GtkViewport vcl/unx/gtk/a11y/atkhypertext.cxx:29 (anonymous) atk_hyper_link AtkHyperlink @@ -606,8 +604,6 @@ vcl/unx/gtk/glomenu.cxx:20 GLOMenu parent_instance GMenuModel writerfilter/source/dmapper/PropertyMap.hxx:198 writerfilter::dmapper::SectionPropertyMap m_nDebugSectionNumber sal_Int32 -xmloff/source/draw/ximpstyl.hxx:144 - SdXMLPresentationPageLayoutContext msName class rtl::OUString xmloff/source/text/XMLTextListBlockContext.hxx:35 XMLTextListBlockContext msListStyleName class rtl::OUString xmloff/source/text/XMLTextListBlockContext.hxx:41 diff --git a/compilerplugins/clang/unusedfields.readonly.results b/compilerplugins/clang/unusedfields.readonly.results index bdef9f1e915a..6fe680af3a7b 100644 --- a/compilerplugins/clang/unusedfields.readonly.results +++ b/compilerplugins/clang/unusedfields.readonly.results @@ -52,7 +52,7 @@ bridges/source/jni_uno/jni_java2uno.cxx:150 jni_uno::largest p void * bridges/source/jni_uno/jni_java2uno.cxx:151 jni_uno::largest a uno_Any -chart2/source/model/main/DataPoint.hxx:107 +chart2/source/model/main/DataPoint.hxx:108 chart::DataPoint m_bNoParentPropAllowed _Bool connectivity/source/drivers/evoab2/EApi.h:125 (anonymous) po char * @@ -242,7 +242,7 @@ framework/inc/xml/menudocumenthandler.hxx:160 framework::OReadMenuHandler m_bMenuPopupMode _Bool framework/inc/xml/menudocumenthandler.hxx:190 framework::OReadMenuPopupHandler m_bMenuMode _Bool -framework/source/fwe/classes/addonsoptions.cxx:299 +framework/source/fwe/classes/addonsoptions.cxx:301 framework::AddonsOptions_Impl m_aEmptyAddonToolBar Sequence<Sequence<struct com::sun::star::beans::PropertyValue> > i18npool/inc/textconversion.hxx:80 i18npool::(anonymous) code sal_Unicode @@ -314,20 +314,12 @@ include/svl/ondemand.hxx:58 OnDemandLocaleDataWrapper aSysLocale class SvtSysLocale include/svtools/editsyntaxhighlighter.hxx:33 MultiLineEditSyntaxHighlight m_aColorConfig svtools::ColorConfig -include/svtools/valueset.hxx:440 - SvtValueSet maSelectHdl Link<class SvtValueSet *, void> -include/svx/dialcontrol.hxx:164 - svx::SvxDialControl m_aText class rtl::OUString -include/svx/dialcontrol.hxx:207 - svx::SvxDialControl::DialControl_Impl maModifyHdl Link<class svx::SvxDialControl *, void> -include/svx/paraprev.hxx:118 - ParaPrevWindow nFirstLineOfst short -include/svx/paraprev.hxx:120 - ParaPrevWindow nUpper sal_uInt16 -include/svx/paraprev.hxx:121 - ParaPrevWindow nLower sal_uInt16 -include/svx/paraprev.hxx:127 - ParaPrevWindow eLine enum SvxPrevLineSpace +include/svx/colorbox.hxx:105 + ColorListBox m_nSlotId sal_uInt16 +include/svx/colorbox.hxx:106 + ColorListBox m_bShowNoneButton _Bool +include/svx/fntctrl.hxx:93 + FontPrevWindow maText class rtl::OUString include/svx/sdr/overlay/overlayanimatedbitmapex.hxx:51 sdr::overlay::OverlayAnimatedBitmapEx mbOverlayState _Bool include/svx/sdr/overlay/overlaymanager.hxx:73 @@ -340,18 +332,6 @@ include/svx/svdoedge.hxx:161 SdrEdgeObj mbBoundRectCalculationRunning _Bool include/svx/svdpntv.hxx:170 SdrPaintView maDrawinglayerOpt class SvtOptionsDrawinglayer -include/svx/swframeexample.hxx:119 - SwFrameExample nWrap css::text::WrapTextMode -include/svx/swframeexample.hxx:121 - SwFrameExample bTrans _Bool -include/test/sheet/xdatapilottable.hxx:31 - apitest::XDataPilotTable xCellForChange css::uno::Reference<css::table::XCell> -include/test/sheet/xdatapilottable.hxx:32 - apitest::XDataPilotTable xCellForCheck css::uno::Reference<css::table::XCell> -include/test/sheet/xnamedranges.hxx:38 - apitest::XNamedRanges xSheet css::uno::Reference<css::sheet::XSpreadsheet> -include/test/sheet/xspreadsheets2.hxx:46 - apitest::XSpreadsheets2 xDocument css::uno::Reference<css::sheet::XSpreadsheetDocument> include/unoidl/unoidl.hxx:443 unoidl::ConstantValue union unoidl::ConstantValue::(anonymous at /home/noel/libo/include/unoidl/unoidl.hxx:443:5) include/unoidl/unoidl.hxx:444 @@ -442,6 +422,8 @@ sc/inc/formulagroup.hxx:39 sc::FormulaGroupEntry::(anonymous) mpCells class ScFormulaCell ** sc/inc/reordermap.hxx:21 sc::ColRowReorderMapType maData sc::ColRowReorderMapType::DataType +sc/qa/extras/anchor.cxx:53 + sc_apitest::ScAnchorTest mxComponent uno::Reference<lang::XComponent> sc/source/core/inc/adiasync.hxx:41 ScAddInAsync::(anonymous) pStr class rtl::OUString * sc/source/core/inc/interpre.hxx:105 @@ -588,9 +570,9 @@ sw/inc/swevent.hxx:71 SwCallMouseEvent::(anonymous union)::(anonymous) pFormat const class SwFrameFormat * sw/source/core/access/accfrmobjmap.hxx:100 SwAccessibleChildMap maMap std::map<key_type, mapped_type, key_compare> -sw/source/core/doc/swstylemanager.cxx:58 - SwStyleManager aAutoCharPool class StylePool sw/source/core/doc/swstylemanager.cxx:59 + SwStyleManager aAutoCharPool class StylePool +sw/source/core/doc/swstylemanager.cxx:60 SwStyleManager aAutoParaPool class StylePool sw/source/core/doc/tblrwcl.cxx:83 CpyTabFrame::(anonymous) nSize SwTwips @@ -616,18 +598,8 @@ sw/source/uibase/inc/fldmgr.hxx:77 SwInsertField_Data m_aDBDataSource css::uno::Any sw/source/uibase/inc/labimg.hxx:49 SwLabItem m_aBin class rtl::OUString -sw/source/uibase/inc/numprevw.hxx:31 - NumberingPreview pActNum const class SwNumRule * -sw/source/uibase/inc/numprevw.hxx:33 - NumberingPreview nPageWidth long -sw/source/uibase/inc/numprevw.hxx:34 - NumberingPreview pOutlineNames const class rtl::OUString * -sw/source/uibase/inc/numprevw.hxx:35 - NumberingPreview bPosition _Bool -sw/source/uibase/inc/optload.hxx:94 +sw/source/uibase/inc/optload.hxx:107 CaptionComboBox aDefault class rtl::OUString -sw/source/uibase/inc/prcntfld.hxx:111 - SwPercentField bLockAutoCalculation _Bool toolkit/source/awt/vclxtoolkit.cxx:433 (anonymous namespace)::VCLXToolkit mxSelection css::uno::Reference<css::datatransfer::clipboard::XClipboard> ucb/source/ucp/gio/gio_mount.hxx:46 @@ -642,7 +614,7 @@ ucb/source/ucp/gio/gio_mount.hxx:52 OOoMountOperationClass _gtk_reserved4 void (*)(void) ucb/source/ucp/hierarchy/hierarchydatasupplier.cxx:73 hierarchy_ucp::DataSupplier_Impl m_aIterator class HierarchyEntry::iterator -ucbhelper/source/client/proxydecider.cxx:118 +ucbhelper/source/client/proxydecider.cxx:127 ucbhelper::proxydecider_impl::InternetProxyDecider_Impl m_aEmptyProxy const struct ucbhelper::InternetProxyServer unoidl/source/sourceprovider-scanner.hxx:147 unoidl::detail::SourceProviderInterfaceTypeEntityPad directMandatoryBases std::vector<DirectBase> @@ -696,11 +668,9 @@ vcl/inc/svdata.hxx:284 ImplSVNWFData mbNoActiveTabTextRaise _Bool vcl/inc/svdata.hxx:286 ImplSVNWFData mbProgressNeedsErase _Bool -vcl/inc/svdata.hxx:295 - ImplSVNWFData mbRolloverMenubar _Bool vcl/inc/unx/i18n_status.hxx:56 vcl::I18NStatus m_aCurrentIM class rtl::OUString -vcl/inc/unx/saldisp.hxx:266 +vcl/inc/unx/saldisp.hxx:288 SalDisplay m_aInvalidScreenData struct SalDisplay::ScreenData vcl/source/filter/jpeg/Exif.hxx:62 Exif::TiffHeader byteOrder sal_uInt16 @@ -714,11 +684,11 @@ vcl/source/filter/jpeg/transupp.h:148 (anonymous) crop_xoffset_set JCROP_CODE vcl/source/filter/jpeg/transupp.h:150 (anonymous) crop_yoffset_set JCROP_CODE -vcl/source/fontsubset/sft.cxx:1054 +vcl/source/fontsubset/sft.cxx:1048 vcl::_subHeader2 firstCode sal_uInt16 -vcl/source/fontsubset/sft.cxx:1055 +vcl/source/fontsubset/sft.cxx:1049 vcl::_subHeader2 entryCount sal_uInt16 -vcl/source/fontsubset/sft.cxx:1056 +vcl/source/fontsubset/sft.cxx:1050 vcl::_subHeader2 idDelta sal_uInt16 vcl/source/gdi/dibtools.cxx:50 (anonymous namespace)::CIEXYZ aXyzX FXPT2DOT30 diff --git a/compilerplugins/clang/unusedfields.untouched.results b/compilerplugins/clang/unusedfields.untouched.results index e87e298f7c70..c8156c084c3d 100644 --- a/compilerplugins/clang/unusedfields.untouched.results +++ b/compilerplugins/clang/unusedfields.untouched.results @@ -62,8 +62,28 @@ cui/source/inc/cuicharmap.hxx:108 SvxCharacterMap m_xShowSetArea std::unique_ptr<weld::CustomWeld> cui/source/inc/cuicharmap.hxx:110 SvxCharacterMap m_xSearchSetArea std::unique_ptr<weld::CustomWeld> +cui/source/inc/cuitabarea.hxx:704 + SvxColorTabPage m_xCtlPreviewOld std::unique_ptr<weld::CustomWeld> +cui/source/inc/cuitabarea.hxx:705 + SvxColorTabPage m_xCtlPreviewNew std::unique_ptr<weld::CustomWeld> +cui/source/inc/cuitabarea.hxx:706 + SvxColorTabPage m_xValSetColorListWin std::unique_ptr<weld::CustomWeld> +cui/source/inc/cuitabarea.hxx:707 + SvxColorTabPage m_xValSetRecentListWin std::unique_ptr<weld::CustomWeld> cui/source/inc/insrc.hxx:31 SvxInsRowColDlg m_xBeforeBtn std::unique_ptr<weld::RadioButton> +cui/source/inc/page.hxx:105 + SvxPageDescPage m_xOrientationFT std::unique_ptr<weld::Label> +cui/source/inc/page.hxx:135 + SvxPageDescPage m_xBspWin std::unique_ptr<weld::CustomWeld> +cui/source/inc/paragrph.hxx:67 + SvxStdParagraphTabPage m_xRightLabel std::unique_ptr<weld::Label> +cui/source/inc/paragrph.hxx:91 + SvxStdParagraphTabPage m_xExampleWin std::unique_ptr<weld::CustomWeld> +cui/source/inc/paragrph.hxx:154 + SvxParaAlignTabPage m_xExampleWin std::unique_ptr<weld::CustomWeld> +cui/source/inc/paragrph.hxx:159 + SvxParaAlignTabPage m_xPropertiesFL std::unique_ptr<weld::Widget> cui/source/inc/swpossizetabpage.hxx:88 SvxSwPosSizeTabPage m_xExampleWN std::unique_ptr<weld::CustomWeld> cui/source/inc/textattr.hxx:69 @@ -80,6 +100,8 @@ emfio/source/emfuno/xemfparser.cxx:60 emfio::emfreader::XEmfParser context_ uno::Reference<uno::XComponentContext> extensions/source/scanner/scanner.hxx:44 ScannerManager maProtector osl::Mutex +filter/source/pdf/impdialog.hxx:186 + ImpPDFTabGeneralPage mxSelectedSheets std::unique_ptr<weld::Label> filter/source/xsltdialog/xmlfiltertabpagebasic.hxx:36 XMLFilterTabPageBasic m_xContainer std::unique_ptr<weld::Widget> filter/source/xsltdialog/xmlfiltertabpagexslt.hxx:48 @@ -110,6 +132,12 @@ include/svtools/unoevent.hxx:162 SvEventDescriptor xParentRef css::uno::Reference<css::uno::XInterface> include/svx/ClassificationDialog.hxx:63 svx::ClassificationDialog m_nInsertMarkings sal_Int16 +include/svx/colorwindow.hxx:133 + ColorWindow mxColorSetWin std::unique_ptr<weld::CustomWeld> +include/svx/colorwindow.hxx:134 + ColorWindow mxRecentColorSetWin std::unique_ptr<weld::CustomWeld> +include/svx/hdft.hxx:87 + SvxHFPage m_xBspWin std::unique_ptr<weld::CustomWeld> include/vcl/uitest/uiobject.hxx:268 TabPageUIObject mxTabPage VclPtr<class TabPage> include/xmloff/formlayerexport.hxx:173 @@ -166,15 +194,15 @@ sal/osl/unx/thread.cxx:112 osl_thread_global_st m_priority struct osl_thread_priority_st sc/inc/formulalogger.hxx:42 sc::FormulaLogger maMessages std::vector<OUString> -sc/inc/interpretercontext.hxx:23 +sc/inc/interpretercontext.hxx:31 ScInterpreterContext mrDoc const class ScDocument & sc/qa/unit/ucalc_column.cxx:104 aInputs aName const char * -sc/source/core/data/document.cxx:1248 +sc/source/core/data/document.cxx:1251 (anonymous namespace)::BroadcastRecalcOnRefMoveHandler aSwitch sc::AutoCalcSwitch -sc/source/core/data/document.cxx:1249 +sc/source/core/data/document.cxx:1252 (anonymous namespace)::BroadcastRecalcOnRefMoveHandler aBulk class ScBulkBroadcast -sc/source/core/data/formulacell.cxx:1756 +sc/source/core/data/formulacell.cxx:1793 (anonymous namespace)::StackCleaner pInt std::unique_ptr<ScInterpreter> sc/source/filter/inc/sheetdatacontext.hxx:61 oox::xls::SheetDataContext aReleaser class SolarMutexReleaser @@ -182,7 +210,7 @@ sc/source/ui/inc/crdlg.hxx:33 ScColOrRowDlg m_xBtnRows std::unique_ptr<weld::RadioButton> sc/source/ui/inc/delcodlg.hxx:39 ScDeleteContentsDlg m_xBtnOk std::unique_ptr<weld::Button> -sc/source/ui/inc/docsh.hxx:460 +sc/source/ui/inc/docsh.hxx:462 ScDocShellModificator mpProtector std::unique_ptr<ScRefreshTimerProtector> sc/source/ui/inc/instbdlg.hxx:66 ScInsertTableDlg m_xBtnBehind std::unique_ptr<weld::RadioButton> @@ -202,7 +230,7 @@ sd/source/ui/slidesorter/view/SlsLayouter.cxx:61 sd::slidesorter::view::Layouter::Implementation mpTheme std::shared_ptr<view::Theme> sd/source/ui/table/TableDesignPane.hxx:106 sd::TableDesignPane aImpl class sd::TableDesignWidget -sd/source/ui/view/DocumentRenderer.cxx:1321 +sd/source/ui/view/DocumentRenderer.cxx:1322 sd::DocumentRenderer::Implementation mxObjectShell SfxObjectShellRef sd/source/ui/view/viewshel.cxx:1215 sd::KeepSlideSorterInSyncWithPageChanges m_aDrawLock sd::slidesorter::view::class SlideSorterView::DrawLock @@ -216,9 +244,9 @@ sd/source/ui/view/ViewShellBase.cxx:192 sd::ViewShellBase::Implementation mpPageCacheManager std::shared_ptr<slidesorter::cache::PageCacheManager> sfx2/source/doc/doctempl.cxx:114 DocTempl::DocTempl_EntryData_Impl mxObjShell class SfxObjectShellLock -sfx2/source/view/viewfrm.cxx:158 +sfx2/source/view/viewfrm.cxx:160 (anonymous namespace)::SfxEditDocumentDialog m_xEditDocument std::unique_ptr<weld::Button> -sfx2/source/view/viewfrm.cxx:159 +sfx2/source/view/viewfrm.cxx:161 (anonymous namespace)::SfxEditDocumentDialog m_xCancel std::unique_ptr<weld::Button> starmath/inc/dialog.hxx:347 SmSymbolDialog m_xSymbolSetDisplayArea std::unique_ptr<weld::CustomWeld> @@ -268,6 +296,8 @@ sw/source/ui/misc/impfnote.hxx:44 SwEndNoteOptionPage m_xPosFT std::unique_ptr<weld::Label> sw/source/uibase/inc/dbui.hxx:31 PrintMonitor m_xDocName std::unique_ptr<weld::Label> +sw/source/uibase/inc/drpcps.hxx:141 + SwDropCapsPage m_xPict std::unique_ptr<weld::CustomWeld> sw/source/uibase/inc/insfnote.hxx:38 SwInsFootNoteDlg m_xNumberFrame std::unique_ptr<weld::Widget> sw/source/uibase/inc/num.hxx:55 @@ -296,9 +326,13 @@ uui/source/unknownauthdlg.hxx:36 UnknownAuthDialog m_xOptionButtonDontAccept std::unique_ptr<weld::RadioButton> vcl/inc/opengl/zone.hxx:46 OpenGLVCLContextZone aZone class OpenGLZone +vcl/inc/qt5/Qt5Menu.hxx:74 + Qt5MenuItem mbVisible _Bool +vcl/inc/qt5/Qt5Object.hxx:34 + Qt5Object m_aSystemData struct SystemEnvData vcl/inc/salprn.hxx:44 SalPrinterQueueInfo mpPortName std::unique_ptr<OUString> -vcl/source/app/salvtables.cxx:463 +vcl/source/app/salvtables.cxx:494 SalInstanceContainer m_xContainer VclPtr<vcl::Window> vcl/source/gdi/jobset.cxx:34 ImplOldJobSetupData cDeviceName char [32] @@ -310,7 +344,7 @@ vcl/unx/generic/print/prtsetup.hxx:73 RTSPaperPage m_xContainer std::unique_ptr<weld::Widget> vcl/unx/generic/print/prtsetup.hxx:108 RTSDevicePage m_xContainer std::unique_ptr<weld::Widget> -vcl/unx/gtk3/gtk3gtkinst.cxx:2166 +vcl/unx/gtk3/gtk3gtkinst.cxx:2221 CrippledViewport viewport GtkViewport vcl/unx/gtk/a11y/atkhypertext.cxx:29 (anonymous) atk_hyper_link AtkHyperlink diff --git a/compilerplugins/clang/unusedfields.writeonly.results b/compilerplugins/clang/unusedfields.writeonly.results index 8087001fff19..8ad41ca74132 100644 --- a/compilerplugins/clang/unusedfields.writeonly.results +++ b/compilerplugins/clang/unusedfields.writeonly.results @@ -1,6 +1,6 @@ basctl/source/basicide/basicbox.hxx:69 basctl::DocListenerBox m_aNotifier class basctl::DocumentEventNotifier -basctl/source/inc/basidesh.hxx:88 +basctl/source/inc/basidesh.hxx:87 basctl::Shell m_aNotifier class basctl::DocumentEventNotifier basctl/source/inc/bastype2.hxx:180 basctl::TreeListBox m_aNotifier class basctl::DocumentEventNotifier @@ -82,6 +82,8 @@ codemaker/source/cppumaker/dependencies.hxx:114 codemaker::cppumaker::Dependencies m_floatDependency _Bool codemaker/source/cppumaker/dependencies.hxx:115 codemaker::cppumaker::Dependencies m_doubleDependency _Bool +comphelper/qa/container/comphelper_ifcontainer.cxx:45 + ContainerListener m_pStats struct ContainerStats * configmgr/source/components.cxx:164 configmgr::Components::WriteThread reference_ rtl::Reference<WriteThread> * connectivity/source/drivers/mork/MorkParser.hxx:133 @@ -158,6 +160,8 @@ cui/source/inc/cuihyperdlg.hxx:57 SvxHlinkCtrl aRdOnlyForwarder class SfxStatusForwarder cui/source/inc/cuihyperdlg.hxx:77 SvxHpLinkDlg maCtrl class SvxHlinkCtrl +cui/source/options/cfgchart.hxx:33 + SvxChartColorTable nNextElementNumber int dbaccess/source/core/dataaccess/documentdefinition.cxx:287 dbaccess::LifetimeCoupler m_xClient Reference<class com::sun::star::uno::XInterface> desktop/qa/desktop_lib/test_desktop_lib.cxx:178 @@ -206,6 +210,8 @@ emfio/inc/mtftools.hxx:128 emfio::LOGFONTW lfClipPrecision sal_uInt8 emfio/inc/mtftools.hxx:129 emfio::LOGFONTW lfQuality sal_uInt8 +emfio/inc/wmfreader.hxx:50 + emfio::WmfReader mnCurrentAction sal_uInt32 emfio/source/reader/emfreader.cxx:310 (anonymous namespace)::BLENDFUNCTION aBlendOperation unsigned char emfio/source/reader/emfreader.cxx:311 @@ -320,7 +326,7 @@ sc/inc/compiler.hxx:262 ScCompiler::AddInMap pEnglish const char * sc/inc/compiler.hxx:264 ScCompiler::AddInMap pUpper const char * -sc/inc/document.hxx:2514 +sc/inc/document.hxx:2509 ScMutationDisable mpDocument class ScDocument * sc/inc/pivot.hxx:75 ScDPLabelData mnFlags sal_Int32 @@ -328,7 +334,7 @@ sc/inc/pivot.hxx:78 ScDPLabelData mbIsValue _Bool sc/source/core/data/cellvalues.cxx:25 sc::(anonymous namespace)::BlockPos mnEnd size_t -sc/source/core/data/column4.cxx:1313 +sc/source/core/data/column4.cxx:1314 (anonymous namespace)::StartListeningFormulaCellsHandler mnStartRow SCROW sc/source/filter/excel/xltoolbar.hxx:23 TBCCmd cmdID sal_uInt16 @@ -384,12 +390,8 @@ sc/source/ui/inc/AccessibleText.hxx:194 ScAccessiblePreviewHeaderCellTextData mbRowHeader _Bool sc/source/ui/inc/datastream.hxx:108 sc::DataStream mnSettings sal_uInt32 -sc/source/ui/inc/filtdlg.hxx:198 - ScSpecialFilterDlg pOptionsMgr class ScFilterOptionsMgr * sc/source/ui/inc/preview.hxx:47 ScPreview nTabPage long -sc/source/ui/inc/sortkeydlg.hxx:46 - ScSortKeyWindow m_nItemHeight sal_Int32 sc/source/ui/inc/tabvwsh.hxx:129 ScTabViewShell pPivotSource class ScArea * sd/source/filter/eppt/eppt.hxx:142 @@ -402,18 +404,14 @@ sd/source/filter/ppt/ppt97animations.hxx:50 Ppt97AnimationInfoAtom nUnknown1 sal_uInt8 sd/source/filter/ppt/ppt97animations.hxx:51 Ppt97AnimationInfoAtom nUnknown2 sal_uInt8 -sd/source/ui/inc/animobjs.hxx:123 - sd::AnimationWindow pControllerItem class sd::AnimationControllerItem * -sd/source/ui/inc/navigatr.hxx:123 - SdNavigatorWin mpNavigatorCtrlItem class SdNavigatorControllerItem * -sd/source/ui/inc/navigatr.hxx:124 - SdNavigatorWin mpPageNameCtrlItem class SdPageNameControllerItem * sd/source/ui/remotecontrol/Receiver.hxx:36 sd::Receiver pTransmitter class sd::Transmitter * sd/source/ui/sidebar/MasterPageContainerProviders.hxx:136 sd::sidebar::TemplatePreviewProvider msURL class rtl::OUString sd/source/ui/sidebar/SlideBackground.hxx:99 sd::sidebar::SlideBackground m_pContainer VclPtr<class VclVBox> +sfx2/inc/bitset.hxx:28 + IndexBitSet nCount sal_uInt16 sfx2/source/view/classificationcontroller.cxx:59 sfx2::ClassificationCategoriesController m_aPropertyListener class sfx2::ClassificationPropertyListener slideshow/source/engine/opengl/TransitionImpl.hxx:296 @@ -464,20 +462,24 @@ svx/source/sidebar/line/LinePropertyPanel.hxx:105 svx::sidebar::LinePropertyPanel maEdgeStyle sfx2::sidebar::ControllerItem svx/source/sidebar/line/LinePropertyPanel.hxx:106 svx::sidebar::LinePropertyPanel maCapStyle sfx2::sidebar::ControllerItem -svx/source/svdraw/svdpdf.hxx:188 - ImpSdrPdfImport mbLastObjWasLine _Bool -svx/source/svdraw/svdpdf.hxx:195 +svx/source/svdraw/svdpdf.hxx:192 ImpSdrPdfImport mdPageWidthPts double svx/source/table/tablertfimporter.cxx:53 sdr::table::RTFCellDefault maItemSet class SfxItemSet -sw/inc/ndgrf.hxx:45 - SwGrfNode bGraphicArrived _Bool sw/inc/shellio.hxx:145 SwReader aFileName class rtl::OUString sw/source/core/doc/tblafmt.cxx:185 SwAfVersions m_nVerticalAlignmentVersion sal_uInt16 +sw/source/core/inc/swfont.hxx:975 + SvStatistics nGetTextSize sal_uInt16 +sw/source/core/inc/swfont.hxx:976 + SvStatistics nDrawText sal_uInt16 sw/source/core/inc/swfont.hxx:977 SvStatistics nGetStretchTextSize sal_uInt16 +sw/source/core/inc/swfont.hxx:978 + SvStatistics nDrawStretchText sal_uInt16 +sw/source/core/inc/swfont.hxx:979 + SvStatistics nChangeFont sal_uInt16 sw/source/core/text/xmldump.cxx:34 XmlPortionDumper ofs TextFrameIndex sw/source/filter/html/htmlcss1.cxx:77 @@ -514,6 +516,10 @@ vcl/inc/opengl/RenderList.hxx:29 Vertex color glm::vec4 vcl/inc/opengl/RenderList.hxx:30 Vertex lineData glm::vec4 +vcl/inc/qt5/Qt5Frame.hxx:59 + Qt5Frame m_pSalMenu class Qt5Menu * +vcl/inc/qt5/Qt5Graphics.hxx:50 + Qt5Graphics m_pFontCollection class PhysicalFontCollection * vcl/inc/salmenu.hxx:34 SalItemParams nBits enum MenuItemBits vcl/inc/salmenu.hxx:42 @@ -540,10 +546,12 @@ vcl/inc/salwtype.hxx:213 SalQueryCharPositionEvent mnCursorBoundHeight long vcl/inc/salwtype.hxx:248 SalSwipeEvent mnVelocityY double -vcl/inc/sft.hxx:462 +vcl/inc/sft.hxx:461 vcl::TrueTypeFont mapper sal_uInt32 (*)(const sal_uInt8 *, sal_uInt32, sal_uInt32) vcl/inc/unx/gtk/gtkframe.hxx:215 GtkSalFrame m_nFloatFlags enum FloatWinPopupFlags +vcl/inc/window.h:293 + WindowImpl mnLockCount sal_uInt16 vcl/opengl/salbmp.cxx:435 (anonymous namespace)::ScanlineWriter mpCurrentScanline sal_uInt8 * vcl/source/filter/graphicfilter.cxx:1008 @@ -568,5 +576,9 @@ vcl/unx/gtk/hudawareness.cxx:20 (anonymous) connection GDBusConnection * vcl/unx/gtk/hudawareness.cxx:23 (anonymous) notify GDestroyNotify +vcl/unx/kde4/KDEXLib.hxx:49 + KDEXLib::SocketData pending YieldFunc +vcl/unx/kde4/KDEXLib.hxx:50 + KDEXLib::SocketData queued YieldFunc writerfilter/source/dmapper/PropertyMap.hxx:198 writerfilter::dmapper::SectionPropertyMap m_nDebugSectionNumber sal_Int32 |