diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2018-01-31 08:58:32 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2018-01-31 12:05:05 +0100 |
| commit | f941cd78a0f805b968bac65fc7c3ef4efffe10fc (patch) | |
| tree | ecb96645538b863b45707360d89bb883ace2a063 | |
| parent | 2192f72e78ce0afeecd40c03f3ad915c7199bf39 (diff) | |
SvxNumberFormat::GetAbsLSpace returns short...
...(and both calls of CorrectTabStopInSet pass in the result of a GetAbsLSpace
call), and SvxTabStop::GetTabPos() is sal_Int32& (and that's what nAbsLeft is
combined with inside CorrectTabStopInSet), so why use an intermediary parameter
of type sal_uInt16?
Change-Id: I1cd1869ead58c419eecb3e2d1fbfb87f96bca86f
Reviewed-on: https://gerrit.libreoffice.org/48956
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
| -rw-r--r-- | sw/source/filter/ww8/wrtww8.hxx | 2 | ||||
| -rw-r--r-- | sw/source/filter/ww8/ww8atr.cxx | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sw/source/filter/ww8/wrtww8.hxx b/sw/source/filter/ww8/wrtww8.hxx index 1336fc9e2dc3..2b39a7d887d5 100644 --- a/sw/source/filter/ww8/wrtww8.hxx +++ b/sw/source/filter/ww8/wrtww8.hxx @@ -883,7 +883,7 @@ public: // TODO move as much as possible here from WW8Export! ;-) - static void CorrectTabStopInSet( SfxItemSet& rSet, sal_uInt16 nAbsLeft ); + static void CorrectTabStopInSet( SfxItemSet& rSet, short nAbsLeft ); private: MSWordExportBase( const MSWordExportBase& ) = delete; diff --git a/sw/source/filter/ww8/ww8atr.cxx b/sw/source/filter/ww8/ww8atr.cxx index 3ccd047b620b..e569ec0a0904 100644 --- a/sw/source/filter/ww8/ww8atr.cxx +++ b/sw/source/filter/ww8/ww8atr.cxx @@ -642,7 +642,7 @@ void WW8Export::PrepareNewPageDesc( const SfxItemSet*pSet, } } -void MSWordExportBase::CorrectTabStopInSet( SfxItemSet& rSet, sal_uInt16 nAbsLeft ) +void MSWordExportBase::CorrectTabStopInSet( SfxItemSet& rSet, short nAbsLeft ) { if (const SvxTabStopItem *pItem = rSet.GetItem<SvxTabStopItem>(RES_PARATR_TABSTOP)) { |