Kaydet (Commit) bb702103 authored tarafından Mike Kaganski's avatar Mike Kaganski

tdf#120703 PVS: V519 variable is assigned values twice successively

V519 The 'aNewTextBoundingBox.Y2' variable is assigned values twice
     successively. Perhaps this is a mistake. Check lines: 409, 411.

I have tried to check if that could possibly be a typo, and X1 was meant,
but it seems to be not the case here (doing that, or putting anything to
X1 instead of 0, just displaces the notes in the presenter console wrong
vertically).

Change-Id: Ie686b69bddc193568c1b5c0d18b9b7bf547313d1
Reviewed-on: https://gerrit.libreoffice.org/62410
Tested-by: Jenkins
Reviewed-by: 's avatarMike Kaganski <mike.kaganski@collabora.com>
üst 6ef1eac7
...@@ -406,7 +406,6 @@ void PresenterNotesView::Layout() ...@@ -406,7 +406,6 @@ void PresenterNotesView::Layout()
mxToolBarWindow->setPosSize(0, aWindowBox.Height - nToolBarHeight, mxToolBarWindow->setPosSize(0, aWindowBox.Height - nToolBarHeight,
sal_Int32(aToolBarSize.Width + 0.5), nToolBarHeight, sal_Int32(aToolBarSize.Width + 0.5), nToolBarHeight,
awt::PosSize::POSSIZE); awt::PosSize::POSSIZE);
aNewTextBoundingBox.Y2 -= nToolBarHeight;
mnSeparatorYLocation = aWindowBox.Height - nToolBarHeight - gnSpaceBelowSeparator; mnSeparatorYLocation = aWindowBox.Height - nToolBarHeight - gnSpaceBelowSeparator;
aNewTextBoundingBox.Y2 = mnSeparatorYLocation - gnSpaceAboveSeparator; aNewTextBoundingBox.Y2 = mnSeparatorYLocation - gnSpaceAboveSeparator;
// Place the close button. // Place the close button.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment