cui/source/tabpages/tparea.cxx | 7 ++----- sc/inc/kahan.hxx | 11 +++++++++++ slideshow/source/engine/shapes/drawshape.cxx | 2 +- 3 files changed, 14 insertions(+), 6 deletions(-)
New commits: commit edb6b8d715f603af544495df06bb6011d299bbf3 Author: Julien Nabet <serval2...@yahoo.fr> AuthorDate: Thu Sep 7 16:34:53 2023 +0200 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Tue Sep 12 09:27:58 2023 +0200 tdf#157138: Can't switch from Use Background to None area fill Revert partly tdf#151260 fix Change-Id: I4c052e2cd1cb41dc8bda4b388b46e4416e351434 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156669 (cherry picked from commit b69e14038288387b5f288a06821fb5df66dcf94e) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156733 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> (cherry picked from commit 7ccaecd33a30905e3929d43c8164b16bbb101b73) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156741 Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/cui/source/tabpages/tparea.cxx b/cui/source/tabpages/tparea.cxx index bbdb9733a02f..119d6d154076 100644 --- a/cui/source/tabpages/tparea.cxx +++ b/cui/source/tabpages/tparea.cxx @@ -238,11 +238,8 @@ DeactivateRC SvxAreaTabPage::DeactivatePage( SfxItemSet* _pSet ) { XFillStyleItem aStyleItem( drawing::FillStyle_NONE ); _pSet->Put( aStyleItem ); - if (_pSet->HasItem(XATTR_FILLUSESLIDEBACKGROUND)) - { - XFillUseSlideBackgroundItem aFillBgItem( false ); - _pSet->Put( aFillBgItem ); - } + XFillUseSlideBackgroundItem aFillBgItem( false ); + _pSet->Put( aFillBgItem ); } break; } commit 655a97dd4c16c64340ddb0c3f7cf682f28f45e33 Author: Tomaž Vajngerl <tomaz.vajng...@collabora.co.uk> AuthorDate: Tue Sep 5 11:10:45 2023 +0200 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Tue Sep 12 09:27:54 2023 +0200 tdf#154138 C/P regression - used setWidth instead of setHeight Change-Id: I2433b4d32e939cadf499b61dfa9033125eaa3c76 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156541 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/slideshow/source/engine/shapes/drawshape.cxx b/slideshow/source/engine/shapes/drawshape.cxx index 8993c8edbe77..cdda31da8a66 100644 --- a/slideshow/source/engine/shapes/drawshape.cxx +++ b/slideshow/source/engine/shapes/drawshape.cxx @@ -775,7 +775,7 @@ namespace slideshow::internal aAABorder.setWidth( ::std::max( rShapeBorder.getWidth(), aAABorder.getWidth() ) ); - aAABorder.setWidth( ::std::max( + aAABorder.setHeight( ::std::max( rShapeBorder.getHeight(), aAABorder.getHeight() ) ); } commit e2397a57ca66a06950a669a3971fbc55b8f5f979 Author: Eike Rathke <er...@redhat.com> AuthorDate: Fri Sep 1 15:20:28 2023 +0200 Commit: Xisco Fauli <xiscofa...@libreoffice.org> CommitDate: Tue Sep 12 09:27:49 2023 +0200 Resolves: tdf#156985 Treat adding two KahanSum differently When summing mixed formula cells and numeric cells, cell type runs are summed using KahanSum that when switching cell types are added. Using add() to explicitly add the rhs m_fError compensation value separately may had lead to effectively cancelling out the relation of sum and error, living on with an unrelated error value. Instead, add a "final" rhs sum+compensation. Change-Id: I751d3e0eeef9cd80482895c24f05b1ab667c3020 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156253 Tested-by: Jenkins Reviewed-by: Eike Rathke <er...@redhat.com> (cherry picked from commit 1f8cc7644293e62ad6430bbeec243d3283e478d7) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156432 Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org> diff --git a/sc/inc/kahan.hxx b/sc/inc/kahan.hxx index 6c84f6eeef2e..ac97ae4394fa 100644 --- a/sc/inc/kahan.hxx +++ b/sc/inc/kahan.hxx @@ -71,8 +71,15 @@ public: */ inline void add(const KahanSum& fSum) { +#ifdef _WIN32 + // For some odd unknown reason WIN32 fails badly with the + // sum+compensation value. Continue keeping the old though slightly off + // (see tdf#156985) explicit addition of the compensation value. add(fSum.m_fSum); add(fSum.m_fError); +#else + add(fSum.m_fSum + fSum.m_fError); +#endif add(fSum.m_fMem); } @@ -82,8 +89,12 @@ public: */ inline void subtract(const KahanSum& fSum) { +#ifdef _WIN32 add(-fSum.m_fSum); add(-fSum.m_fError); +#else + add(-(fSum.m_fSum + fSum.m_fError)); +#endif add(-fSum.m_fMem); }