diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2014-06-10 17:23:12 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2014-06-10 17:25:23 +0200 |
commit | 4dbeadb9c1e46ac0008f076cd6f9c5d0a38a4d40 (patch) | |
tree | a71acd0d466e02c18bf78ffdcfea748135805d21 /sc/source/ui/optdlg/tpusrlst.cxx | |
parent | 2f8fd888b42dc41662b54a16d62575c2b15e844a (diff) |
Change SfxTabPage::FillItemSet param from ref to pointer
...there were a number of call sites that passed undefined "null pointer
references" (apparently in cases where the passed argument was actually unused)
Change-Id: I19799e90f0cd8e98367782441a5ea9df27b59830
Diffstat (limited to 'sc/source/ui/optdlg/tpusrlst.cxx')
-rw-r--r-- | sc/source/ui/optdlg/tpusrlst.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sc/source/ui/optdlg/tpusrlst.cxx b/sc/source/ui/optdlg/tpusrlst.cxx index 19c432f81f7f..06abb7ae9d53 100644 --- a/sc/source/ui/optdlg/tpusrlst.cxx +++ b/sc/source/ui/optdlg/tpusrlst.cxx @@ -192,7 +192,7 @@ void ScTpUserLists::Reset( const SfxItemSet& rCoreAttrs ) } } -bool ScTpUserLists::FillItemSet( SfxItemSet& rCoreAttrs ) +bool ScTpUserLists::FillItemSet( SfxItemSet* rCoreAttrs ) { // Modifikationen noch nicht uebernommen? // -> Click auf Add-Button simulieren @@ -225,7 +225,7 @@ bool ScTpUserLists::FillItemSet( SfxItemSet& rCoreAttrs ) if ( pUserLists ) aULItem.SetUserList( *pUserLists ); - rCoreAttrs.Put( aULItem ); + rCoreAttrs->Put( aULItem ); } return bDataModified; @@ -234,7 +234,7 @@ bool ScTpUserLists::FillItemSet( SfxItemSet& rCoreAttrs ) int ScTpUserLists::DeactivatePage( SfxItemSet* pSetP ) { if ( pSetP ) - FillItemSet( *pSetP ); + FillItemSet( pSetP ); return LEAVE_PAGE; } |