[Libreoffice-bugs] [Bug 139734] Drop redundant asserts after MacrosTest::loadFromDesktop

bugzilla-daemon at bugs.documentfoundation.org bugzilla-daemon at bugs.documentfoundation.org
Fri Feb 19 21:28:00 UTC 2021


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

--- Comment #4 from Moaz El-defrawy <moaz.eldefrawy at gmail.com> ---
Update: I am removed 99% of the redundant asserts and I will submit a patch
very soon.

I had a question though:
There is a pattern of asserting the return values of many functions not just
"loadFromDesktop". Why don't we assert their return values instead just like
"loadFromDesktop" and remove the extra ones?

another question:
I am not sure how style the comments to make them more readable. I tried html
and readme files styling but neither worked.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20210219/7d2b1474/attachment.htm>


More information about the Libreoffice-bugs mailing list