[Libreoffice-commits] core.git: vcl/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Thu Jan 3 13:39:08 UTC 2019


 vcl/source/window/window2.cxx |    5 -----
 1 file changed, 5 deletions(-)

New commits:
commit f90ec6941c745cd4478aff825e175fde0b873a12
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu Jan 3 13:38:52 2019 +0100
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Thu Jan 3 14:38:42 2019 +0100

    Remove obsolete scope
    
    The comment here states that the braces are for DbgChkThis; and it was so
    when the code arrived in commit 8ab086b6cc054501bfbf7ef6fa509c393691e860.
    
    Change-Id: Iba96822cf3c92217d0da2a15ce6d0976e8397793
    Reviewed-on: https://gerrit.libreoffice.org/65835
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>

diff --git a/vcl/source/window/window2.cxx b/vcl/source/window/window2.cxx
index 877c66bdee05..3d5b1b16368c 100644
--- a/vcl/source/window/window2.cxx
+++ b/vcl/source/window/window2.cxx
@@ -280,10 +280,6 @@ void Window::EndTracking( TrackingEventFlags nFlags )
 
     if ( pSVData->maWinData.mpTrackWin.get() == this )
     {
-        // due to DbgChkThis in brackets, as the window could be destroyed
-        // in the handler
-        {
-
         if ( pSVData->maWinData.mpTrackTimer )
         {
             delete pSVData->maWinData.mpTrackTimer;
@@ -293,7 +289,6 @@ void Window::EndTracking( TrackingEventFlags nFlags )
         pSVData->maWinData.mpTrackWin    = nullptr;
         pSVData->maWinData.mnTrackFlags  = StartTrackingFlags::NONE;
         ReleaseMouse();
-        }
 
         // call EndTracking if required
         {


More information about the Libreoffice-commits mailing list