diff options
author | Tor Lillqvist <tml@collabora.com> | 2017-05-16 21:48:02 +0300 |
---|---|---|
committer | Tor Lillqvist <tml@collabora.com> | 2017-05-18 08:06:38 +0300 |
commit | d89fa2bd4944625e2dbe56d5709a3f126db24f21 (patch) | |
tree | af8e22727ecb7f7d69acae2fa9c2430a6db28d43 /vcl/source/window/floatwin.cxx | |
parent | e5736b9e4ded62cdeaa49649763392779d312e0a (diff) |
Clean up a set of RTL-related comments
It is fairly pointless to have a comment like "--- RTL --- (mirror
mouse pos)" in front of code that obviously is related to RTL anyway,
like "if( AllSettings::GetLayoutRTL() )". Also, the fact that many
comment were exactly the same indicates that they has been added as an
in-line version control change marker.
In other places, comments were informational, but no need for them to
include yet another metasyntax, "--- RTL ---". Just "RTL:" works fine.
Change-Id: I20feef308ed8ac9d32a8bf440fc20b442753c4ff
Diffstat (limited to 'vcl/source/window/floatwin.cxx')
-rw-r--r-- | vcl/source/window/floatwin.cxx | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/vcl/source/window/floatwin.cxx b/vcl/source/window/floatwin.cxx index 656a551d2a55..71de988dfed7 100644 --- a/vcl/source/window/floatwin.cxx +++ b/vcl/source/window/floatwin.cxx @@ -307,7 +307,7 @@ Point FloatingWindow::ImplCalcPos( vcl::Window* pWindow, aPos.X() = devRect.Left()-aSize.Width()+1; aPos.Y() = devRect.Top(); aPos.Y() -= pWindow->mpWindowImpl->mnTopBorder; - if( bRTL ) // --- RTL --- we're comparing screen coordinates here + if( bRTL ) { if( (devRectRTL.Right()+aSize.Width()) > aScreenRect.Right() ) bBreak = false; @@ -331,7 +331,7 @@ Point FloatingWindow::ImplCalcPos( vcl::Window* pWindow, case FloatWinPopupFlags::Right: aPos = devRect.TopRight(); aPos.Y() -= pWindow->mpWindowImpl->mnTopBorder; - if( bRTL ) // --- RTL --- we're comparing screen coordinates here + if( bRTL ) { if( (devRectRTL.Left() - aSize.Width()) < aScreenRect.Left() ) bBreak = false; @@ -401,7 +401,7 @@ Point FloatingWindow::ImplCalcPos( vcl::Window* pWindow, } else { - if( bRTL ) // --- RTL --- we're comparing screen coordinates here + if( bRTL ) { if( devRectRTL.Right()-aSize.Width()+1 < aScreenRect.Left() ) aPos.X() -= aScreenRect.Left() - devRectRTL.Right() + aSize.Width() - 1; @@ -446,7 +446,6 @@ Point FloatingWindow::ImplConvertToAbsPos(vcl::Window* pReference, const Point& if( pReference->HasMirroredGraphics() ) { if(!pReference->IsRTLEnabled() ) - // --- RTL --- re-mirror back to get device coordinates pWindowOutDev->ReMirror( aAbsolute ); tools::Rectangle aRect( pReference->ScreenToOutputPixel(aAbsolute), Size(1,1) ) ; @@ -471,7 +470,6 @@ tools::Rectangle FloatingWindow::ImplConvertToAbsPos(vcl::Window* pReference, co if( pReference->HasMirroredGraphics() ) { if(!pReference->IsRTLEnabled() ) - // --- RTL --- re-mirror back to get device coordinates pParentWinOutDev->ReMirror(aFloatRect); aFloatRect.SetPos(pReference->ScreenToOutputPixel(aFloatRect.TopLeft())); |