diff options
| author | Maxim Monastirsky <momonasmon@gmail.com> | 2020-08-02 18:45:21 +0300 |
|---|---|---|
| committer | Maxim Monastirsky <momonasmon@gmail.com> | 2020-08-03 09:58:50 +0200 |
| commit | da09a3cf14ea2c7690b98ec32639125087bcf18f (patch) | |
| tree | ecc0dccfaff1ebc7660cb4e0e1e4ad529ec821ac /sfx2/source/dialog | |
| parent | e88cf047c70cedd6ac3553258d4d33bee878a1e6 (diff) | |
Remove pointless check
This check was originally introduced by me for a few toolbar
controllers (save, underline, currency), as an attempt to reuse
the same controllers also for non-split buttons (see tdf#98745
and tdf#101672). But since then it was copy-pasted to other
places where it probably makes no sense.
Change-Id: I7af2a3f5864ced88c91f330e87f32046d0c21383
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99960
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
Diffstat (limited to 'sfx2/source/dialog')
| -rw-r--r-- | sfx2/source/dialog/charmappopup.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sfx2/source/dialog/charmappopup.cxx b/sfx2/source/dialog/charmappopup.cxx index 7788a45c73c0..f8a791889c37 100644 --- a/sfx2/source/dialog/charmappopup.cxx +++ b/sfx2/source/dialog/charmappopup.cxx @@ -35,7 +35,7 @@ void CharmapPopup::initialize( const css::uno::Sequence< css::uno::Any >& rArgum ToolBox* pToolBox = nullptr; sal_uInt16 nId = 0; - if (getToolboxId(nId, &pToolBox) && pToolBox->GetItemCommand(nId) == m_aCommandURL) + if (getToolboxId(nId, &pToolBox)) pToolBox->SetItemBits(nId, ToolBoxItemBits::DROPDOWNONLY | pToolBox->GetItemBits(nId)); } |