<html> <head> <base href="https://bugs.documentfoundation.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - Drop redundant asserts after MacrosTest::loadFromDesktop" href="https://bugs.documentfoundation.org/show_bug.cgi?id=139734#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - Drop redundant asserts after MacrosTest::loadFromDesktop" href="https://bugs.documentfoundation.org/show_bug.cgi?id=139734">bug 139734</a> from <a class="email" href="mailto:mikekaganski@hotmail.com" title="Mike Kaganski <mikekaganski@hotmail.com>"> Mike Kaganski</a> <pre>(In reply to Moaz El-defrawy from <a href="show_bug.cgi?id=139734#c4">comment #4</a>) > Update: I am removed 99% of the redundant asserts and I will submit a patch > very soon. Very good! > 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? This needs discussing case-by-case. In this form, I can't answer this. Which specific change(s) you propose? > 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. Are you talking about commit message, or about some comments in code? It's best to discuss in Gerrit change, where you post some proposed change, and then comment on parts that you are unsure with, and get responses from reviewers. Looking forward to see the change on Gerrit. Thanks for working on this!</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>