Kaydet (Commit) 3cb3a809 authored tarafından Noel Grandin's avatar Noel Grandin

loplugin:constparams in fpicker

Change-Id: I1ef422ccd902a3bc5e4fce5ea373cfef24f8edef
Reviewed-on: https://gerrit.libreoffice.org/40204Tested-by: 's avatarJenkins <ci@libreoffice.org>
Reviewed-by: 's avatarNoel Grandin <noel.grandin@collabora.co.uk>
üst 3b10ec87
...@@ -186,12 +186,12 @@ namespace svt ...@@ -186,12 +186,12 @@ namespace svt
} }
OUString OControlAccess::getHelpURL( vcl::Window* _pControl, bool _bFileView ) OUString OControlAccess::getHelpURL( vcl::Window const * _pControl, bool _bFileView )
{ {
OString aHelpId = _pControl->GetHelpId(); OString aHelpId = _pControl->GetHelpId();
if ( _bFileView ) if ( _bFileView )
// the file view "overrides" the SetHelpId // the file view "overrides" the SetHelpId
aHelpId = static_cast< SvtFileView* >( _pControl )->GetHelpId( ); aHelpId = static_cast< SvtFileView const * >( _pControl )->GetHelpId( );
OUString sHelpURL; OUString sHelpURL;
OUString aTmp( OStringToOUString( aHelpId, RTL_TEXTENCODING_UTF8 ) ); OUString aTmp( OStringToOUString( aHelpId, RTL_TEXTENCODING_UTF8 ) );
...@@ -677,7 +677,7 @@ namespace svt ...@@ -677,7 +677,7 @@ namespace svt
} }
Any OControlAccess::implGetControlProperty( Control* _pControl, PropFlags _nProperty ) const Any OControlAccess::implGetControlProperty( Control const * _pControl, PropFlags _nProperty ) const
{ {
DBG_ASSERT( _pControl, "OControlAccess::implGetControlProperty: invalid argument, this will crash!" ); DBG_ASSERT( _pControl, "OControlAccess::implGetControlProperty: invalid argument, this will crash!" );
...@@ -705,10 +705,10 @@ namespace svt ...@@ -705,10 +705,10 @@ namespace svt
DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(), DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(),
"OControlAccess::implGetControlProperty: invalid control/property combination!" ); "OControlAccess::implGetControlProperty: invalid control/property combination!" );
Sequence< OUString > aItems( static_cast< ListBox* >( _pControl )->GetEntryCount() ); Sequence< OUString > aItems( static_cast< ListBox const * >( _pControl )->GetEntryCount() );
OUString* pItems = aItems.getArray(); OUString* pItems = aItems.getArray();
for ( sal_Int32 i=0; i<static_cast< ListBox* >( _pControl )->GetEntryCount(); ++i ) for ( sal_Int32 i=0; i<static_cast< ListBox const * >( _pControl )->GetEntryCount(); ++i )
*pItems++ = static_cast< ListBox* >( _pControl )->GetEntry( i ); *pItems++ = static_cast< ListBox const * >( _pControl )->GetEntry( i );
aReturn <<= aItems; aReturn <<= aItems;
} }
...@@ -719,10 +719,10 @@ namespace svt ...@@ -719,10 +719,10 @@ namespace svt
DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(), DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(),
"OControlAccess::implGetControlProperty: invalid control/property combination!" ); "OControlAccess::implGetControlProperty: invalid control/property combination!" );
sal_Int32 nSelected = static_cast< ListBox* >( _pControl )->GetSelectEntryPos(); sal_Int32 nSelected = static_cast< ListBox const * >( _pControl )->GetSelectEntryPos();
OUString sSelected; OUString sSelected;
if ( LISTBOX_ENTRY_NOTFOUND != nSelected ) if ( LISTBOX_ENTRY_NOTFOUND != nSelected )
sSelected = static_cast< ListBox* >( _pControl )->GetSelectEntry(); sSelected = static_cast< ListBox const * >( _pControl )->GetSelectEntry();
aReturn <<= sSelected; aReturn <<= sSelected;
} }
break; break;
...@@ -732,9 +732,9 @@ namespace svt ...@@ -732,9 +732,9 @@ namespace svt
DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(), DBG_ASSERT( WindowType::LISTBOX == _pControl->GetType(),
"OControlAccess::implGetControlProperty: invalid control/property combination!" ); "OControlAccess::implGetControlProperty: invalid control/property combination!" );
sal_Int32 nSelected = static_cast< ListBox* >( _pControl )->GetSelectEntryPos(); sal_Int32 nSelected = static_cast< ListBox const * >( _pControl )->GetSelectEntryPos();
if ( LISTBOX_ENTRY_NOTFOUND != nSelected ) if ( LISTBOX_ENTRY_NOTFOUND != nSelected )
aReturn <<= static_cast< ListBox* >( _pControl )->GetSelectEntryPos(); aReturn <<= static_cast< ListBox const * >( _pControl )->GetSelectEntryPos();
else else
aReturn <<= (sal_Int32)-1; aReturn <<= (sal_Int32)-1;
} }
...@@ -744,7 +744,7 @@ namespace svt ...@@ -744,7 +744,7 @@ namespace svt
DBG_ASSERT( WindowType::CHECKBOX == _pControl->GetType(), DBG_ASSERT( WindowType::CHECKBOX == _pControl->GetType(),
"OControlAccess::implGetControlProperty: invalid control/property combination!" ); "OControlAccess::implGetControlProperty: invalid control/property combination!" );
aReturn <<= static_cast< CheckBox* >( _pControl )->IsChecked( ); aReturn <<= static_cast< CheckBox const * >( _pControl )->IsChecked( );
break; break;
default: default:
......
...@@ -86,7 +86,7 @@ namespace svt ...@@ -86,7 +86,7 @@ namespace svt
void enableControl( sal_Int16 _nId, bool _bEnable ); void enableControl( sal_Int16 _nId, bool _bEnable );
static void setHelpURL( vcl::Window* _pControl, const OUString& _rURL, bool _bFileView ); static void setHelpURL( vcl::Window* _pControl, const OUString& _rURL, bool _bFileView );
static OUString getHelpURL( vcl::Window* _pControl, bool _bFileView ); static OUString getHelpURL( vcl::Window const * _pControl, bool _bFileView );
private: private:
/** implements the various methods for setting properties on controls /** implements the various methods for setting properties on controls
...@@ -120,7 +120,7 @@ namespace svt ...@@ -120,7 +120,7 @@ namespace svt
See PropFlags::* See PropFlags::*
@return @return
*/ */
css::uno::Any implGetControlProperty( Control* _pControl, PropFlags _nProperty ) const; css::uno::Any implGetControlProperty( Control const * _pControl, PropFlags _nProperty ) const;
static void implDoListboxAction( ListBox* _pListbox, sal_Int16 _nCtrlAction, const css::uno::Any& _rValue ); static void implDoListboxAction( ListBox* _pListbox, sal_Int16 _nCtrlAction, const css::uno::Any& _rValue );
......
...@@ -116,7 +116,7 @@ namespace ...@@ -116,7 +116,7 @@ namespace
{ {
OUString getMostCurrentFilter( std::unique_ptr<SvtExpFileDlg_Impl>& pImpl ) OUString getMostCurrentFilter( std::unique_ptr<SvtExpFileDlg_Impl> const & pImpl )
{ {
assert( pImpl && "invalid impl pointer" ); assert( pImpl && "invalid impl pointer" );
const SvtFileDialogFilter_Impl* pFilter = pImpl->_pUserFilter; const SvtFileDialogFilter_Impl* pFilter = pImpl->_pUserFilter;
...@@ -131,7 +131,7 @@ namespace ...@@ -131,7 +131,7 @@ namespace
} }
bool restoreCurrentFilter( std::unique_ptr<SvtExpFileDlg_Impl>& pImpl ) bool restoreCurrentFilter( std::unique_ptr<SvtExpFileDlg_Impl> const & pImpl )
{ {
SAL_WARN_IF( !pImpl->GetCurFilter(), "fpicker.office", "restoreCurrentFilter: no current filter!" ); SAL_WARN_IF( !pImpl->GetCurFilter(), "fpicker.office", "restoreCurrentFilter: no current filter!" );
SAL_WARN_IF( pImpl->GetCurFilterDisplayName().isEmpty(), "fpicker.office", "restoreCurrentFilter: no current filter (no display name)!" ); SAL_WARN_IF( pImpl->GetCurFilterDisplayName().isEmpty(), "fpicker.office", "restoreCurrentFilter: no current filter (no display name)!" );
...@@ -839,7 +839,7 @@ IMPL_LINK( SvtFileDialog, OpenUrlHdl_Impl, SvtURLBox*, pVoid, void ) ...@@ -839,7 +839,7 @@ IMPL_LINK( SvtFileDialog, OpenUrlHdl_Impl, SvtURLBox*, pVoid, void )
{ {
OpenHdl_Impl(pVoid); OpenHdl_Impl(pVoid);
} }
void SvtFileDialog::OpenHdl_Impl(void* pVoid) void SvtFileDialog::OpenHdl_Impl(void const * pVoid)
{ {
if ( pImpl->_bMultiSelection && _pFileView->GetSelectionCount() > 1 ) if ( pImpl->_bMultiSelection && _pFileView->GetSelectionCount() > 1 )
{ {
......
...@@ -111,7 +111,7 @@ private: ...@@ -111,7 +111,7 @@ private:
DECL_LINK( RemovePlacePressed_Hdl, Button*, void ); DECL_LINK( RemovePlacePressed_Hdl, Button*, void );
DECL_LINK( Split_Hdl, Splitter*, void ); DECL_LINK( Split_Hdl, Splitter*, void );
void OpenHdl_Impl(void* pVoid); void OpenHdl_Impl(void const * pVoid);
void Init_Impl( PickerFlags nBits ); void Init_Impl( PickerFlags nBits );
/** find a filter with the given wildcard /** find a filter with the given wildcard
@param _rFilter @param _rFilter
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment