diff options
-rw-r--r-- | compilerplugins/clang/unusedfields.cxx | 16 | ||||
-rw-r--r-- | compilerplugins/clang/unusedfields.only-used-in-constructor.results | 44 | ||||
-rw-r--r-- | compilerplugins/clang/unusedfields.readonly.results | 66 | ||||
-rw-r--r-- | compilerplugins/clang/unusedfields.untouched.results | 54 | ||||
-rw-r--r-- | compilerplugins/clang/unusedfields.writeonly.results | 50 | ||||
-rw-r--r-- | emfio/inc/wmfreader.hxx | 1 | ||||
-rw-r--r-- | emfio/source/reader/wmfreader.cxx | 3 | ||||
-rw-r--r-- | include/svx/colorbox.hxx | 2 | ||||
-rw-r--r-- | include/svx/fntctrl.hxx | 2 | ||||
-rw-r--r-- | include/vcl/window.hxx | 3 | ||||
-rw-r--r-- | sd/source/ui/slidesorter/view/SlideSorterView.cxx | 2 | ||||
-rw-r--r-- | sfx2/inc/bitset.hxx | 1 | ||||
-rw-r--r-- | sfx2/source/bastyp/bitset.cxx | 3 | ||||
-rw-r--r-- | svx/source/dialog/fntctrl.cxx | 6 | ||||
-rw-r--r-- | svx/source/tbxctrls/tbcontrl.cxx | 24 | ||||
-rw-r--r-- | vcl/inc/window.h | 1 | ||||
-rw-r--r-- | vcl/source/window/dndeventdispatcher.cxx | 16 | ||||
-rw-r--r-- | vcl/source/window/window.cxx | 1 | ||||
-rw-r--r-- | vcl/source/window/window2.cxx | 13 |
19 files changed, 134 insertions, 174 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 diff --git a/emfio/inc/wmfreader.hxx b/emfio/inc/wmfreader.hxx index 4054141f0f0c..9e1059d63760 100644 --- a/emfio/inc/wmfreader.hxx +++ b/emfio/inc/wmfreader.hxx @@ -47,7 +47,6 @@ namespace emfio sal_uInt32 mnEMFSize; sal_uInt32 mnSkipActions; - sal_uInt32 mnCurrentAction; // eventually handed over external header const WmfExternal* mpExternalHeader; diff --git a/emfio/source/reader/wmfreader.cxx b/emfio/source/reader/wmfreader.cxx index 83e7ca6e81d7..f93eb0cbc31a 100644 --- a/emfio/source/reader/wmfreader.cxx +++ b/emfio/source/reader/wmfreader.cxx @@ -1364,7 +1364,6 @@ namespace emfio sal_uInt16 nFunction; mnSkipActions = 0; - mnCurrentAction = 0; mpEMFStream.reset(); mnEMFRecCount = 0; @@ -1388,7 +1387,6 @@ namespace emfio bool bEMFAvailable = false; while( true ) { - mnCurrentAction++; mpInputStream->ReadUInt32(mnRecSize).ReadUInt16( nFunction ); if ( @@ -1854,7 +1852,6 @@ namespace emfio , mnEMFRec(0) , mnEMFSize(0) , mnSkipActions(0) - , mnCurrentAction(0) , mpExternalHeader(pExternalHeader) { } diff --git a/include/svx/colorbox.hxx b/include/svx/colorbox.hxx index cb7c5e820128..6915f851e0a5 100644 --- a/include/svx/colorbox.hxx +++ b/include/svx/colorbox.hxx @@ -102,8 +102,6 @@ private: Color m_aAutoDisplayColor; Color m_aSaveColor; NamedColor m_aSelectedColor; - sal_uInt16 m_nSlotId; - bool m_bShowNoneButton; std::shared_ptr<PaletteManager> m_xPaletteManager; BorderColorStatus m_aBorderColorStatus; diff --git a/include/svx/fntctrl.hxx b/include/svx/fntctrl.hxx index 52db1503ba3e..e3c6b47276f5 100644 --- a/include/svx/fntctrl.hxx +++ b/include/svx/fntctrl.hxx @@ -90,7 +90,6 @@ class SAL_WARN_UNUSED SVX_DLLPUBLIC FontPrevWindow : public weld::CustomWidgetCo { private: std::unique_ptr<FontPrevWin_Impl> pImpl; - OUString maText; bool mbResetForeground : 1; bool mbResetBackground : 1; @@ -112,7 +111,6 @@ public: SvxFont& GetCJKFont(); SvxFont& GetCTLFont(); void SetBackColor( const Color& rColor ); - const OUString& GetText() const { return maText; } void Paint( vcl::RenderContext& rRenderContext, const tools::Rectangle& ) override; bool IsTwoLines() const; diff --git a/include/vcl/window.hxx b/include/vcl/window.hxx index 8ba941422514..9ef90177388f 100644 --- a/include/vcl/window.hxx +++ b/include/vcl/window.hxx @@ -832,9 +832,6 @@ public: ImplSVEvent * PostUserEvent( const Link<void*,void>& rLink, void* pCaller = nullptr, bool bReferenceLink = false ); void RemoveUserEvent( ImplSVEvent * nUserEvent ); - void IncrementLockCount(); - void DecrementLockCount(); - // returns the input language used for the last key stroke // may be LANGUAGE_DONTKNOW if not supported by the OS LanguageType GetInputLanguage() const; diff --git a/sd/source/ui/slidesorter/view/SlideSorterView.cxx b/sd/source/ui/slidesorter/view/SlideSorterView.cxx index 10c5fd0e3389..0688bfac45bc 100644 --- a/sd/source/ui/slidesorter/view/SlideSorterView.cxx +++ b/sd/source/ui/slidesorter/view/SlideSorterView.cxx @@ -611,11 +611,9 @@ void SlideSorterView::CompleteRedraw ( if (mnLockRedrawSmph == 0) { - mrSlideSorter.GetContentWindow()->IncrementLockCount(); if (mpLayeredDevice->HandleMapModeChange()) DeterminePageObjectVisibilities(); mpLayeredDevice->Repaint(rPaintArea); - mrSlideSorter.GetContentWindow()->DecrementLockCount(); } else { diff --git a/sfx2/inc/bitset.hxx b/sfx2/inc/bitset.hxx index d33ccf54bebe..fc29dbd67ecb 100644 --- a/sfx2/inc/bitset.hxx +++ b/sfx2/inc/bitset.hxx @@ -25,7 +25,6 @@ class IndexBitSet { private: sal_uInt16 nBlocks; - sal_uInt16 nCount; std::unique_ptr<sal_uInt32[]> pBitmap; IndexBitSet& operator|=( sal_uInt16 nBit ); diff --git a/sfx2/source/bastyp/bitset.cxx b/sfx2/source/bastyp/bitset.cxx index 495909f8f14b..9dc18f3e5c04 100644 --- a/sfx2/source/bastyp/bitset.cxx +++ b/sfx2/source/bastyp/bitset.cxx @@ -37,7 +37,6 @@ IndexBitSet& IndexBitSet::operator-=(sal_uInt16 nBit) if ( pBitmap[nBlock] & nBitVal ) { pBitmap[nBlock] &= ~nBitVal; - --nCount; } return *this; @@ -66,7 +65,6 @@ IndexBitSet& IndexBitSet::operator|=( sal_uInt16 nBit ) if ( (pBitmap[nBlock] & nBitVal) == 0 ) { pBitmap[nBlock] |= nBitVal; - ++nCount; } return *this; @@ -87,7 +85,6 @@ bool IndexBitSet::Contains( sal_uInt16 nBit ) const IndexBitSet::IndexBitSet() { - nCount = 0; nBlocks = 0; } diff --git a/svx/source/dialog/fntctrl.cxx b/svx/source/dialog/fntctrl.cxx index 311cfdd10d7b..0168c02d7e0c 100644 --- a/svx/source/dialog/fntctrl.cxx +++ b/svx/source/dialog/fntctrl.cxx @@ -1571,7 +1571,7 @@ void FontPrevWindow::Paint(vcl::RenderContext& rRenderContext, const tools::Rect else { if (pImpl->mbUseResText) - pImpl->maText = GetText(); + pImpl->maText = OUString(); else if (!pImpl->mbSelection && !pImpl->mbTextInited) { using namespace css::i18n::ScriptType; @@ -1611,7 +1611,7 @@ void FontPrevWindow::Paint(vcl::RenderContext& rRenderContext, const tools::Rect } if (pImpl->maText.isEmpty()) - pImpl->maText = GetText(); + pImpl->maText = OUString(); if (pImpl->maText.isEmpty()) { // fdo#58427: still no text? let's try that one... @@ -1620,7 +1620,7 @@ void FontPrevWindow::Paint(vcl::RenderContext& rRenderContext, const tools::Rect bool bEmpty = CleanAndCheckEmpty(pImpl->maText); if (bEmpty) - pImpl->maText = GetText(); + pImpl->maText = OUString(); if (pImpl->maText.getLength() > (TEXT_WIDTH - 1)) { diff --git a/svx/source/tbxctrls/tbcontrl.cxx b/svx/source/tbxctrls/tbcontrl.cxx index 325ca4359d7d..84a704ba8a75 100644 --- a/svx/source/tbxctrls/tbcontrl.cxx +++ b/svx/source/tbxctrls/tbcontrl.cxx @@ -3762,10 +3762,8 @@ ColorListBox::ColorListBox(weld::MenuButton* pControl, weld::Window* pTopLevel) , m_pTopLevel(pTopLevel) , m_aColorWrapper(this) , m_aAutoDisplayColor(Application::GetSettings().GetStyleSettings().GetDialogColor()) - , m_nSlotId(0) - , m_bShowNoneButton(false) { - m_aSelectedColor = GetAutoColor(m_nSlotId); + m_aSelectedColor = GetAutoColor(0); LockWidthRequest(); ShowPreview(m_aSelectedColor); } @@ -3793,15 +3791,13 @@ void ColorListBox::createColorWindow() OUString() /*m_aCommandURL*/, m_xPaletteManager, m_aBorderColorStatus, - m_nSlotId, + 0, // slotID xFrame, m_pTopLevel, m_xButton.get(), m_aColorWrapper)); SetNoSelection(); - if (m_bShowNoneButton) - m_xColorWindow->ShowNoneButton(); m_xButton->set_popover(m_xColorWindow->GetWidget()); m_xColorWindow->SelectEntry(m_aSelectedColor); } @@ -3865,20 +3861,10 @@ void ColorListBox::ShowPreview(const NamedColor &rColor) ScopedVclPtrInstance<VirtualDevice> xDevice; xDevice->SetOutputSize(aImageSize); const tools::Rectangle aRect(Point(0, 0), aImageSize); - if (m_bShowNoneButton && rColor.first == COL_NONE_COLOR) - { - const Color aW(COL_WHITE); - const Color aG(0xef, 0xef, 0xef); - xDevice->DrawCheckered(aRect.TopLeft(), aRect.GetSize(), 8, aW, aG); - xDevice->SetFillColor(); - } + if (rColor.first == COL_AUTO) + xDevice->SetFillColor(m_aAutoDisplayColor); else - { - if (rColor.first == COL_AUTO) - xDevice->SetFillColor(m_aAutoDisplayColor); - else - xDevice->SetFillColor(rColor.first); - } + xDevice->SetFillColor(rColor.first); xDevice->SetLineColor(rStyleSettings.GetDisableColor()); xDevice->DrawRect(aRect); diff --git a/vcl/inc/window.h b/vcl/inc/window.h index da11f0bc514e..0e2ee4a7cb6f 100644 --- a/vcl/inc/window.h +++ b/vcl/inc/window.h @@ -290,7 +290,6 @@ public: ParentClipMode mnParentClipMode; ActivateModeFlags mnActivateMode; DialogControlFlags mnDlgCtrlFlags; - sal_uInt16 mnLockCount; AlwaysInputMode meAlwaysInputMode; VclAlign meHalign; VclAlign meValign; diff --git a/vcl/source/window/dndeventdispatcher.cxx b/vcl/source/window/dndeventdispatcher.cxx index a5f6cdf515a2..9a0814052b24 100644 --- a/vcl/source/window/dndeventdispatcher.cxx +++ b/vcl/source/window/dndeventdispatcher.cxx @@ -260,9 +260,6 @@ sal_Int32 DNDEventDispatcher::fireDragEnterEvent( vcl::Window *pWindow, { SolarMutexClearableGuard aSolarGuard; - // set an UI lock - pWindow->IncrementLockCount(); - // query DropTarget from window Reference< XDropTarget > xDropTarget = pWindow->GetDropTarget(); @@ -323,9 +320,6 @@ sal_Int32 DNDEventDispatcher::fireDragExitEvent( vcl::Window *pWindow ) if( xDropTarget.is() ) n = static_cast < DNDListenerContainer * > ( xDropTarget.get() )->fireDragExitEvent(); - - // release UI lock - pWindow->DecrementLockCount(); } return n; @@ -385,13 +379,6 @@ sal_Int32 DNDEventDispatcher::fireDropEvent( vcl::Window *pWindow, n = static_cast < DNDListenerContainer * > ( xDropTarget.get() )->fireDropEvent( xContext, nDropAction, relLoc.X(), relLoc.Y(), nSourceActions, xTransferable ); } - - if ( !xWindow->IsDisposed() ) - { - // release UI lock - pWindow->DecrementLockCount(); - } - } return n; @@ -420,9 +407,6 @@ sal_Int32 DNDEventDispatcher::fireDragGestureEvent( vcl::Window *pWindow, n = static_cast < DNDListenerContainer * > ( xDragGestureRecognizer.get() )->fireDragGestureEvent( nDragAction, relLoc.X(), relLoc.Y(), xSource, event ); } - - // release UI lock - pWindow->DecrementLockCount(); } return n; diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx index 8383e58a6e07..02f856cf4eb6 100644 --- a/vcl/source/window/window.cxx +++ b/vcl/source/window/window.cxx @@ -647,7 +647,6 @@ WindowImpl::WindowImpl( WindowType nType ) mnParentClipMode = ParentClipMode::NONE; // Flags for Parent-ClipChildren-Mode mnActivateMode = ActivateModeFlags::NONE; // Will be converted in System/Overlap-Windows mnDlgCtrlFlags = DialogControlFlags::NONE; // DialogControl-Flags - mnLockCount = 0; // LockCount meAlwaysInputMode = AlwaysInputNone; // neither AlwaysEnableInput nor AlwaysDisableInput called meHalign = VclAlign::Fill; meValign = VclAlign::Fill; diff --git a/vcl/source/window/window2.cxx b/vcl/source/window/window2.cxx index 426eb01636ec..459557d438b8 100644 --- a/vcl/source/window/window2.cxx +++ b/vcl/source/window/window2.cxx @@ -979,19 +979,6 @@ void Window::SetCompoundControl( bool bCompound ) mpWindowImpl->mbCompoundControl = bCompound; } -void Window::IncrementLockCount() -{ - assert( mpWindowImpl != nullptr ); - mpWindowImpl->mnLockCount++; -} - -void Window::DecrementLockCount() -{ - assert( mpWindowImpl != nullptr ); - if (mpWindowImpl) - mpWindowImpl->mnLockCount--; -} - WinBits Window::GetStyle() const { return mpWindowImpl ? mpWindowImpl->mnStyle : 0; |