framework/source/services/desktop.cxx |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 13b7905ba142d78b6061c52a8cc191125b0d7d5f
Author:     Stephan Bergmann <stephan.bergm...@allotropia.de>
AuthorDate: Wed Dec 13 10:18:00 2023 +0100
Commit:     Stephan Bergmann <stephan.bergm...@allotropia.de>
CommitDate: Wed Dec 13 20:48:27 2023 +0100

    Remove misleading comment
    
    ...that had been added with 33fa50e1191c75fd2fcbf6f852ec1d76af58a577 "Drop 
some
    pointless assertions" back in 2011, but is no longer meaningful now that 
this is
    a proper assert since 6e35794cad555485955c3b43593497dcdbf29840 "terminate
    XDesktop properly in unit tests"
    
    Change-Id: Id8592df03f2a4c070af3cc691a0aa65ba8c2c061
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/160675
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de>

diff --git a/framework/source/services/desktop.cxx 
b/framework/source/services/desktop.cxx
index 237d35afc6a5..12a45fa93117 100644
--- a/framework/source/services/desktop.cxx
+++ b/framework/source/services/desktop.cxx
@@ -1038,8 +1038,6 @@ void SAL_CALL Desktop::disposing()
     // Safe impossible cases
     // It's a programming error if dispose is called before terminate!
 
-    // But if you just ignore the assertion (which happens in unit
-    // tests for instance in sc/qa/unit) nothing bad happens.
     assert(m_bIsShutdown && "Desktop disposed before terminating it");
 
     {

Reply via email to