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 /sw/source/ui/envelp/envfmt.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 'sw/source/ui/envelp/envfmt.cxx')
-rw-r--r-- | sw/source/ui/envelp/envfmt.cxx | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sw/source/ui/envelp/envfmt.cxx b/sw/source/ui/envelp/envfmt.cxx index c816fe933972..6ec8cab83818 100644 --- a/sw/source/ui/envelp/envfmt.cxx +++ b/sw/source/ui/envelp/envfmt.cxx @@ -457,7 +457,7 @@ void SwEnvFmtPage::ActivatePage(const SfxItemSet& rSet) int SwEnvFmtPage::DeactivatePage(SfxItemSet* _pSet) { if( _pSet ) - FillItemSet(*_pSet); + FillItemSet(_pSet); return SfxTabPage::LEAVE_PAGE; } @@ -485,10 +485,10 @@ void SwEnvFmtPage::FillItem(SwEnvItem& rItem) } } -bool SwEnvFmtPage::FillItemSet(SfxItemSet& rSet) +bool SwEnvFmtPage::FillItemSet(SfxItemSet* rSet) { FillItem(GetParentSwEnvDlg()->aEnvItem); - rSet.Put(GetParentSwEnvDlg()->aEnvItem); + rSet->Put(GetParentSwEnvDlg()->aEnvItem); return true; } |