https://bugs.documentfoundation.org/show_bug.cgi?id=43157

--- Comment #120 from Stephan Bergmann <[email protected]> ---
(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: [email protected]
> To: [email protected]
> Subject: [Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc.
> Date: Thu, 11 Aug 2022 14:46:26 +0000
> 
> Buovjaga <[email protected]> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                  CC|                            |mentoring@documentfoundatio
>                    |                            |n.org
>            Keywords|                            |easyHack

> From: [email protected]
> To: [email protected]
> Subject: [Bug 43157] Clean up OSL_ASSERT, DBG_ASSERT, etc.
> Date: Thu, 11 Aug 2022 14:56:45 +0000
> 
> Mike Kaganski <[email protected]> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>            Keywords|difficultyInteresting       |difficultyMedium

@Ilmari, Mike, can you clarify your recent changes here vs. my comment 63?

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to