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>
This commit is contained in:
Stephan Bergmann
2018-01-31 08:58:32 +01:00
parent 2192f72e78
commit f941cd78a0
2 changed files with 2 additions and 2 deletions

View File

@@ -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;

View File

@@ -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))
{