The assertion/logging functionality from osl/diagnose.h (OSL_TRACE, OSL_ASSERT, OSL_ENSURE, OSL_FAIL) and tools/debug.hxx (DBG_ASSERTWARNING, DBG_ASSERT, DBG_BF_ASSERT, DBG_WARNING, DBG_WARNING1--5, DBG_WARNINGFILE, DBG_ERRORFILE) is obsolete and needs to be cleaned up: * To assert invariants of the code (that can only be violated if there are programming errors) use standard C/C++ assert. * To log warnings about unusual events (that the code nevertheless needs to handle in some way, like malformed input or I/O failures), use the SAL_WARN... macros from sal/log.h. * To log other information useful for debugging, use the SAL_INFO... macros. See <https://wiki.documentfoundation.org/Development#Assertions_and_Logging>, the mail thread at <http://lists.freedesktop.org/archives/libreoffice/2011-November/020864.html>, and the documentation in the sal/log.h header for further information.
Further, sal_detail_logFormat in sal/log.h has a commented-out GCC __attribute__((format)), to catch misuses of its printf-style message composition. It is used by the SAL_WARN and SAL_INFO macros, which in turn is used by the obsoleted OSL_TRACE, for which many calls currently use ill-formed format strings. Once those uses are cleaned up (preferably by replacing them with SAL_INFO_S), the __attribute__((format)) should be enabled.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e4429d43f55c52f16130a745d428b89cbf2f0394 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d4bca924dd7ebe1b57f15de3e741f0fffcb67225 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
How does this bug relate to bug 39425?
*** Bug 39425 has been marked as a duplicate of this bug. ***
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f9aabc714af41d7c3dd5cd08b6dc0761db904c4d fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f4f727f7bfc848deb82e4df4c6e3b9affaf5f613 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ef1d096ddc311a2fd821d4076421c4a8f8d63abf fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a3d299e15526fdcbcaae269e33a83a9c0b187a5a fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=b20e7e1d11c8765116c415d7dabd3602d88d1103 fdo#43157 - Clean up OSL_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d461088eb4dd3b3bf7c2caaab082890ebcf80fea fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=671b73789b2b09384e278efa0b989560b6606e4f fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Radu Ioan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ac937d8be69953abcaf1365644c6607c279f37a6 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Honza HavliÄek committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d7b48d03e19d2c05ecf2143b1547ceab5ab89e5c fdo#43157 - Clean up OSL_ENSURE The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jacobo Aragunde Perez committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=723afc607ec3e947afa0f2122751ea537bca62bd fdo#43157 - Clean up OSL_ENSURE The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
adding LibreOffice developer list as CC to unresolved EasyHacks for better visibility. see e.g. http://nabble.documentfoundation.org/minutes-of-ESC-call-td4076214.html for details
Timothy Markle committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=995073b08e69f6616479f5dd4d4094d96f893207 fdo#43157 - Clean up OSL_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to comment #0) > See > <https://wiki.documentfoundation.org/Development#Assertions_and_Logging>, > the mail thread at > <http://lists.freedesktop.org/archives/libreoffice/2011-November/020864. > html>, and the documentation in the sal/log.h header for further information. The first of those links got stale, use <https://wiki.documentfoundation.org/Development/GeneralProgrammingGuidelines#Assertions_and_Logging> now (as reflected in this issue's URL field now). Thanks to Caolán for spotting.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=c03c9da8c249c2e2a61bab3d6fad325d5934f5e0 fdo#43157 : clean up OSL_POSTCOND The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=453ee351f32494b1f30b477069cc7a1834352377 fdo#43157 : clean up more OSL_POSTCOND The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=22ab56f5e35dff58e1923b693dc33ff252293557 fdo#43157 : clean up OSL_DEBUG_ONLY The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=df2c663fef3435a8c3d541b364780876229ccd29 fdo#43157 [API CHANGE] Remove deprecated _OSL_GLOBAL The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=fe075edc27a1fc301c31cfdff5ddbc83e52d929f fdo#43157 : [API CHANGE] Remove OSL_THIS_FILE The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Michaël Lefèvre committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=70e744b2157ebe48f3a5c839560fedf6521d0464 fdo#43157 Remove OSL_POSTCOND, prefer SAL_WARN_IF The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
(In reply to comment #0) > The assertion/logging functionality from osl/diagnose.h (OSL_TRACE, > OSL_ASSERT, OSL_ENSURE, OSL_FAIL) and tools/debug.hxx (DBG_ASSERTWARNING, > DBG_ASSERT, DBG_BF_ASSERT, DBG_WARNING, DBG_WARNING1--5, DBG_WARNINGFILE, > DBG_ERRORFILE) is obsolete and needs to be cleaned up: > > * To assert invariants of the code (that can only be violated if there are > programming errors) use standard C/C++ assert. > > * To log warnings about unusual events (that the code nevertheless needs to > handle in some way, like malformed input or I/O failures), use the > SAL_WARN... macros from sal/log.h. > > * To log other information useful for debugging, use the SAL_INFO... macros. > > See > <https://wiki.documentfoundation.org/Development#Assertions_and_Logging>, > the mail thread at > <http://lists.freedesktop.org/archives/libreoffice/2011-November/020864. > html>, and the documentation in the sal/log.h header for further information. In spite of the fact that many people uploaded commits I found that in some files these obsolete assertions/logging. I am going to change as many of these files as I can.
Mukhiddin Yusupov committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=0562a17c64f4016f2d3ba01e1c19c27feadb2543 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Stephan Bergmann committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=92621df9e72b9d4e44e448f92df14c4ba731d5e4 Revert "fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT" The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Mukhiddin Yusupov committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4c05911b7ef25505acacbb81053a56cf428e9edc fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
NEEDINFO->NEW: The NEEDINFO state seems to have happened accidentally.
Naruhiko Ogasawara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=922f2005f34589e60969be3f2bf74e4af58e2e69 fdo#43157 - Clean up OSL_ASSERT, DBG_ASSERT It will be available in 4.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
*** Bug 88309 has been marked as a duplicate of this bug. ***
(In reply to Stephan Bergmann from comment #0) > See > <https://wiki.documentfoundation.org/Development#Assertions_and_Logging> ...which has moved to <https://wiki.documentfoundation.org/Development/GeneralProgrammingGuidelines#Assertions_and_Logging>
Hi! This is my first bug, and I want to solve it. Please guide me. Thank you.
(In reply to Jitesh from comment #33) > Hi! This is my first bug, and I want to solve it. Please guide me. Thank you. Jitesh, please ask specific questions. (And it might often be easier to ask them in the IRC channel or on the mailing list.) (Also, with a large clean-up task like this, there is no real need to assign it to yourself if you want to work on it. There is so many places in the codebase that need clean-up that multiple people could work on it in parallel easily.)
Stephan Bergmann committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ee11e221d2108212619e1bbe7f029e7d9afdba32 tdf#43157: Fix format string violations in OSL_TRACE etc. It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Stephan Bergmann committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d1c9e5ee53f8609092ea026639c8f28afec1d36a tdf#43157: Fix format string violations in OSL_TRACE etc. It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Stephan Bergmann committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=36aafefc064cf5d5df51b7ca7bcc7e2256cefd69 tdf#43157: Fix format string violations in OSL_TRACE etc. (Mac OS X) It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Stephan Bergmann committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7196df7ac616be39689f21d8784fd78030868586 tdf#43157: Enable format check in sal_detail_logFormat It will be available in 4.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jorge Cunha Mendes committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=c2254a12df715f3220f8c070be02b754d6375b40 tdf#43157: remove osl/diagnose.h dependency from sal/osl/unx/socket.cxx It will be available in 5.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Migrating Whiteboard tags to Keywords: (EasyHack DifficultyBeginner SkillCpp TopicCleanup ) [NinjaEdit]
aybuke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=91c072b473beadda01a38dbc26086207c7b4d145 tdf#43157: Fix format string violations in OSL_TRACE etc. It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Berk Gureken committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=9108dc39525fb1d881d5b4a634039a25ef2d2d25 tdf#43157 Cleanup DBG_ASSERT for fpicker and framework It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Hi everyone, I sent this patch and it compiled succesfully: https://gerrit.libreoffice.org/#/c/21958/
JanI is default CC for Easy Hacks (Add Jan; remove LibreOffice Dev List from CC) [NinjaEdit]
Arnold Dumas committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8c97225d311af6711ac49755079acd29842afeba tdf#43157: Clean up OSL_ASSERT, DBG_ASSERT It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
aybuke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=05d2684840d3956ff76ed6db00a43cf2d49c8454 tdf#43157 Cleanup DBG_ASSERT for formula module. It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Sedat Ak committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e959fd9e2389727c74631b8b29c5d39b404b390a tdf#43157 Clean up DBG_ASSERT for svx It will be available in 5.2.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
This seems to be fixed.
It is not--there are still loads of DBG_ASSERT, OSL_ASSERT, OSL_ENSURE, OSL_FAIL, etc.
Arnold Dumas committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e1ca24369142af3e721e789bf757be671e1905b7 tdf#43157: Clean up OSL_TRACE It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
krishna keshav committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=46e372a6015033fabf1b2ac1bb46dec993b25178 tdf#43157 Clean up DBG_ASSERT It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Controlled still open
Jochen Nitschke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3cc30343bef48c7f19a3b6774cfd500e00258f76 tdf#43157 clean up OSL_ASSERT in connectivity It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Please be aware, that this easyhack is considered an important but large scale cosmetic change as described in https://wiki.documentfoundation.org/Development/LargeScaleChanges It was in decided by the ESC to close this kind of easyhacks, and send them directly as mail, to new contributors. https://lists.freedesktop.org/archives/libreoffice/2016-August/074920.html Please do not submit patches with many files !! This particular easyhack is kept open as an exception to the rule, due to a high wish in the ESC
drazil committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=41ce5e234c5b884b2efd426f46caec2831545b45 tdf#43157 Clean up OSL_ASSERT It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
[updated to difficultyInteresting, as it can be very difficult to decide what the right replacement shall be in each case; you typically need a deep understanding of the relevant code]
Gökhan Gurbetoğlu committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f62d22286972d22809fafbd9cb44263d12a6aa85 tdf#43157 - Clean up OSL_ASSERT, DBG_ASSERT, etc. It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
drazil committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=12c11e65780e48bc569c1028b0f5efec3373c5ee tdf#43157 Clean up OSL_ENSURE in FontTable.cxx It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Sophie Su committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=5059efab2f982daf041701165d15587794ffbfc7 tdf#43157 clean up OSL_ASSERT in /core/sw/source/core/undo/docundo.cxx It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bd97bca534b61ef302c606ae7452454d5e87910e tdf#43157 Cleanup OSL_ASSERT in scenwnd.cxx It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Muhammet Kara committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=80e6671cacbe1785948de0311be972f897e962d9 tdf#43157 Cleanup DBG_ASSERT in misc.cxx It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
dilekuzulmez committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=1cf15a4cdde09972e78baee704014099433ac43e tdf#43157 Clean up OSL_ASSERT, DBG_ASSERT, etc. It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I think this is too difficult for an EasyHack after all (thus removing the keyword). For somebody not intimately familiar with a piece of code, it can be hard to tell what that code's invariants are (i.e., where to use 'assert' vs. 'SAL_WARN' etc.). Commits for this issue therefore typically needed multiple iterations (or even got abandoned mid-way), which can cause more frustration for newcomers than help.
I am working on core/tools/ directory for cleanup. In order to avoid merge conflict or any repeated work done, if anyone else want to work on the same, please do comment.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=2b1737f648024328390bf44c4f2c614e748a92fd tdf#43157: convert svx module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=fbea669c74f686f5a75aa80384bbfeff98680f8d tdf#43157: convert desktop module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=48de83aa42166f0107ad5d98fce491d21173741f tdf#43157: convert filter module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a85694c4c7aae17aae3f419ef935ae5cab39b174 tdf#43157: convert xmlsecurity module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=825db3204260c8dc9502b997250c42193376322b tdf#43157: convert cui module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d8c2e2cdf267d9222629739261519b89a4025ea2 tdf#43157: convert basctl module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f1ac23866db030bd4489ab79a7e2e669070b9dab tdf#43157: convert dbaccess module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=fbf171d1adf3e260f6a93217e82a1b74cc4f670c tdf#43157: convert extensions module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=fb684bbd9d376ae5e3081c236f5d0775b5a93c03 tdf#43157: convert oox module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=9d4d4d8434e5333ecb9464c76aea795dfbd6da2c tdf#43157: convert scripting module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=83017d884d58a2fe0ccaf23271ab3e232118ec79 tdf#43157: convert sc module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=392dbe6fe982e11009e04229e7e5200f838ea4f9 tdf#43157: convert sw module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=381c43eb5d5455aeb5ed3335e89bb6b56c843080 tdf#43157: convert writerfilter module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3965f4cb28676133dc37a926e56b4d612e2a57ba tdf#43157: convert starmath module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d0a99cc2ed76be220f7e868e332ba19f6e48a440 tdf#43157: convert sd module away from OSL_ASSERT to assert It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Fakabbir Amin committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=f9f0b6ac12c5d6c73d9aa415a9de5e37105085d8 tdf#43157 Cleanup DBG_ASSERT in pstm.cxx It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8c0be54a7da6262dffe04357121814dd22b5d7fe tdf#43157: convert cppcanvas module away from OSL_ASSERT It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3a6eb7d7e5aaa455d892d63f95cced03d3e32a95 tdf#43157 - registry: convert from OSL_ASSERT to assert It will be available in 5.5.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3c354ea9ed8158143110fda56af22d12e750fc2f tdf#43157 - salhelper: convert from OSL_ASSERT to assert It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Thomas Beck committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=778a396adef3e62a87f6e9beec0964323ad57f44 tdf#43157 Clean up OSL_ASSERT, DBG_ASSERT, etc. It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3b0f8ece47f70f53b1462c824463c453b47c3020 tdf#43157 - salhelper: timer.cxx change OSL_ASSERT to assert It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=bca760c294c256fcb1fd3649dbf50f61a4bda474 tdf#43157 - osl: convert OSL_ASSERTs to assert in signalshared.cxx It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7c6ccc424eec53cac72a9fc81913f6ad780d91aa tdf#43157 - osl: convert OSL_PRECONDs in w32 & unx file.cxx It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=ef151d297c4c490d8694e14f8f556b56cad3325a fdo#43157 convert OSL_ASSERT to assert in sw/source/core It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=37e7bb9c7803a3d4a3b4c44562487403e274203f fdo#43157 convert OSL_ASSERT to assert in sw/source It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=25d850cd5b71948b35ca94ccef37653fe88d0829 fdo#43157 convert OSL_ASSERT to assert in sc/source/core It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=0ab214536efce0fdecf9a79b58467c5d464c1929 fdo#43157 convert OSL_ASSERT to assert in sd/source/ui/slidesorter/cache It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=e1fc7ea54a6dc76378fee8880530b24ca33039b6 fdo#43157 convert OSL_ASSERT to assert in sd/source/ui/slidesorter It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=4b7b93be530dd5b7924588fbc5855c6fd057984c tdf#43157 - osl: convert OSL_ASSERT in unx & w32 tempfile.cxx It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6c4224b7ac9d8b016759ae8b5c0afdbef473a16f tdf#43157 - osl: convert OSL_ASSERT in unx/file_misc.cxx It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Bernhard Widl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=b0210c34dcaa5fc44956078ffa1dfe734d379224 fdo#43157 convert OSL_ASSERT to assert in sc/source/filter It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Eike Rathke committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=6959aff40692cc02c99efb6781091263f8a25a84 Address b0210c34dcaa5fc44956078ffa1dfe734d379224 nitpicks, tdf#43157 follow-up It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Albert Thuswaldner committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=91325884de5d3c3f838cfdcb1c70d42e5edab149 tdf#43157 - Clean up OSL_ENSURE and OSL_FAIL in sc/filter/lotus It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Grzegorz Araminowicz committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=8ff306d81be73cedec940ae996d73f4e3a4a6422 tdf#43157 Clean up OSL_ENSURE in oox/drawingml/diagram It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Ulrich Gemkow committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=c686dc4f33c7461a28df95603d06591a3a0171cd tdf#43157 Replace DBG_ASSERT() with assert() in tabbar.cxx It will be available in 6.0.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
** Please read this message in its entirety before responding ** To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year. There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present. If you have time, please do the following: Test to see if the bug is still present with the latest version of LibreOffice from https://www.libreoffice.org/download/ If the bug is present, please leave a comment that includes the information from Help - About LibreOffice. If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a comment that includes the information from Help - About LibreOffice. Please DO NOT Update the version field Reply via email (please reply directly on the bug tracker) Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not appropriate in this case) If you want to do more to help you can test to see if your issue is a REGRESSION. To do so: 1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) from http://downloadarchive.documentfoundation.org/libreoffice/old/ 2. Test your bug 3. Leave a comment with your results. 4a. If the bug was present with 3.3 - set version to 'inherited from OOo'; 4b. If the bug was not present in 3.3 - add 'regression' to keyword Feel free to come ask questions or to say hello in our QA chat: https://kiwiirc.com/nextclient/irc.freenode.net/#libreoffice-qa Thank you for helping us make LibreOffice even better for everyone! Warm Regards, QA Team MassPing-UntouchedBug
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/2a8f2fd072c9de7b5dc538419565e3351bd0936a%5E%21 tdf#43157 Clean up OSL_TRACE (replace with SAL_INFO) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/516983b7798a3ecc4d9e443ef5d8e573e01f5e53%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/187174d5c18f80f2e6fe03355379e5e14117791e%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/bfdb5a2d96483907cd4b757398b3df159a9e7af2%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/c338638eeabbc959a4d71e04774365e628a093c2%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/b8e002f4d4426183c7ea35cb7af6917ed800256c%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/19573d94074a3ce296bacf35a21242ff8b75cad4%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/48ee3cce67d028d6e6e97405f6add22a80a3fc98%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Shubham Goyal committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/891db26079e0983f2afb38d32aa57eec9dbefc98%5E%21 tdf#43157 - Clean up OSL_ASSERT, DBG_ASSERT It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/d376c09f0100e6545b4804a6698bcc746ea9427c%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/af36ccc2143a7181964cc87077303ac5a5be205e%5E%21 tdf#43157 Clean up OSL_VERIFY (replace with SAL_WARN) It will be available in 6.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/5291c3ef56948a91166a597033f3e48e00d57403 tdf#43157 basegfx: remove OSL_ENSURE preconditions It will be available in 6.5.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Shubham Goyal committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/f61e5e85c6bc3337eabfe3b9b69c4c1a3abf08e7 tdf#43157 Clean up DBG_ASSERT It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Pelin Kuran committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/d61cd023f549c41a48f2b7fbf821ad547897ae8a tdf#43157: Clean up OSL_ASSERT, DBG_ASSERT, etc. It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Pelin Kuran committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/bdb1c72198f60fdd91460e26282134d43bc0e2df tdf43157:Clean up OSL_ASSERT, DBG_ASSERT, etc.. It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Jens Carl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/a6496d03f69ae4a0140f78eff2d6f29a6fdb74da tdf#43157 Clean up OSL_ASSERT It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Tomaž Vajngerl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/38c5cfa1da8f0a615e479a68f3bcb238395109a8 fdo#43157: VclPixelProcessor2D replace OSL_ENSURE with SAL_WARN_IF It will be available in 7.0.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Dear Stephan Bergmann, To make sure we're focusing on the bugs that affect our users today, LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed bugs which have not been touched for over a year. There have been thousands of bug fixes and commits since anyone checked on this bug report. During that time, it's possible that the bug has been fixed, or the details of the problem have changed. We'd really appreciate your help in getting confirmation that the bug is still present. If you have time, please do the following: Test to see if the bug is still present with the latest version of LibreOffice from https://www.libreoffice.org/download/ If the bug is present, please leave a comment that includes the information from Help - About LibreOffice. If the bug is NOT present, please set the bug's Status field to RESOLVED-WORKSFORME and leave a comment that includes the information from Help - About LibreOffice. Please DO NOT Update the version field Reply via email (please reply directly on the bug tracker) Set the bug's Status field to RESOLVED - FIXED (this status has a particular meaning that is not appropriate in this case) If you want to do more to help you can test to see if your issue is a REGRESSION. To do so: 1. Download and install oldest version of LibreOffice (usually 3.3 unless your bug pertains to a feature added after 3.3) from https://downloadarchive.documentfoundation.org/libreoffice/old/ 2. Test your bug 3. Leave a comment with your results. 4a. If the bug was present with 3.3 - set version to 'inherited from OOo'; 4b. If the bug was not present in 3.3 - add 'regression' to keyword Feel free to come ask questions or to say hello in our QA chat: https://web.libera.chat/?settings=#libreoffice-qa Thank you for helping us make LibreOffice even better for everyone! Warm Regards, QA Team MassPing-UntouchedBug
(In reply to QA Administrators from comment #118) > Test to see if the bug is still present with the latest version of > LibreOffice from https://www.libreoffice.org/download/ > > If the bug is present, please leave a comment that includes the information > from Help - About LibreOffice. There is still lots of occurrences of the problematic macros on current master branch (towards LO 7.4).
(In reply to Stephan Bergmann from comment #63) > I think this is too difficult for an EasyHack after all (thus removing the > keyword). For somebody not intimately familiar with a piece of code, it can > be hard to tell what that code's invariants are (i.e., where to use 'assert' > vs. 'SAL_WARN' etc.). Commits for this issue therefore typically needed > multiple iterations (or even got abandoned mid-way), which can cause more > frustration for newcomers than help. > From: bugzilla-daemon@bugs.documentfoundation.org > To: mentoring@documentfoundation.org > Subject: [Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc. > Date: Thu, 11 Aug 2022 14:46:26 +0000 > > Buovjaga <ilmari.lauhakangas@libreoffice.org> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > CC| |mentoring@documentfoundatio > | |n.org > Keywords| |easyHack > From: bugzilla-daemon@bugs.documentfoundation.org > To: mentoring@documentfoundation.org > Subject: [Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc. > Date: Thu, 11 Aug 2022 14:56:45 +0000 > > Mike Kaganski <mikekaganski@hotmail.com> changed: > > What |Removed |Added > ---------------------------------------------------------------------------- > Keywords|difficultyInteresting |difficultyMedium @Ilmari, Mike, can you clarify your recent changes here vs. my comment 63?
(In reply to Stephan Bergmann from comment #120) I believe it's still easyhack, with rather trivial difficulty - just needs some guidelines. E.g., there's lots of 'OSL_ENSURE(false, ...)' stuff in things like catch clauses; I don't see how replacing those could be non-trivial. Thus, I suggested to make it easyhack again, with rather low difficulty - and if needed, new, more focused easyhacks could be created instead.
(In reply to Mike Kaganski from comment #121) > E.g., there's lots of 'OSL_ENSURE(false, ...)' stuff in things like catch > clauses; I don't see how replacing those could be non-trivial. What's non-trivial there per se? How to tell, without intimate knowledge of the code, whether that catch clause can legitimately be reached (and the OSL_ENSURE should be replaced with SAL_WARN or SAL_INFO) or not (and the OSL_ENSURE should be replace with assert or O3TL_UNREACHABLE)?
(In reply to Stephan Bergmann from comment #122) Hmm, put that way, I agree that it is far from trivial ... And yes, replacement without taking that in account would loose "this needs to be checked" information. Sorry for initiating this change (it was me who started it). One nitpick (that doesn't change much - I'd still ask why isn't this suggested to newcomers) is that when you remove the easyhack keyword, difficulty* should be removed as well (likely, that is also true for skill*).
Baole Fang committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/69a3bcdaa54700a372151af52ccfbb0deed9db45 tdf#43157: Clean up OSL_FAIL It will be available in 7.6.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/383f666bcad27b2fbea6ac13a42cafd6f035fc5c tdf#43157 vcl: remove DBG_ASSERT() from BreakLinesWithIterator() It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/29df1afa1f439bf44a6be1b04ec0190e8f40abe2 tdf#43157 vcl: remove DBG_ASSERT() from DrawDeviceAlphaBitmapSlowPath() It will be available in 24.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Gabor Kelemen committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/57c228803e55ed343c6693de7d0857ad7d3cd9e3 tdf#43157 Use SAL_WARN_IF instead of OSL_PRECOND It will be available in 24.8.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Gabor Kelemen committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/722d0c8b9df2cddd2f783076ca626329909218f4 tdf#43157 Use SAL_WARN instead of OSL_FAIL It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Gabor Kelemen committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/6c0db491f000c55e6bf6061d33ba2312b0ec303f tdf#43157 Use SAL_WARN_IF instead of OSL_ENSURE It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Chris Sherlock committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/8f6f57e126c8e44f23fde6d35f027214a3fd557b tdf#43157 vcl: remove OSL_ENSURE from BitmapEx functions It will be available in 25.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.