<div dir="ltr"><div dir="ltr">Hey Florian,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 1, 2021 at 1:52 AM Markus Mohrhard <<a href="mailto:markus.mohrhard@googlemail.com" target="_blank">markus.mohrhard@googlemail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hey Florian,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 28, 2021 at 5:59 PM Florian Becker <<a href="mailto:fb@vxapps.com" target="_blank">fb@vxapps.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
i have patched the libcppunit 1.15.1 to modernize and use clang flags <br>
with highest flag settings. Hopefully you can integrate this patch <br>
inside master. <a href="https://github.com/VXAPPS/cppunit/tree/feature-modernize" rel="noreferrer" target="_blank">https://github.com/VXAPPS/cppunit/tree/feature-modernize</a><br>
<br>
If you have suggestions, please let me know.<br></blockquote><div><br></div><div>I'll have a look during the next few days and will let you know if I have any questions or comments</div></div></div></blockquote><div><br></div><div>Thanks for your patches. The summary is:</div><div><br></div><div>* first patch was used without modifications</div><div>* second patch was split up into smaller commits:</div><div>   * add [[noreturn]]</div><div>   * add override annotations</div><div>   * switch from throw to noexcept</div><div>   * remove semicolons</div><div>   * a complete removal of the old casting macros by me instead of your addition to the portability header</div><div>   * replace 0 with nullptr</div><div>   * disable a few simple compiler warnings</div><div><br></div><div>I have not yet taken the size_t changes as that part contains some bugs. I have already fixed one but want to have a second look and think about whether this is how we want to solve this problem.</div><div>I also had to remove all references to [[deprecated]] in this change as [[deprecated]] is only C++14 and at least for now cppunit is still at C++11.<br></div><div><br></div><div>* I'll have to think a bit about the third and fourth patch. If I merge them I'll let you know.</div><div><br></div><div>You can find the full code that I have pushed at:</div><div><br></div><div><a href="https://git.libreoffice.org/cppunit/+log/refs/heads/master">https://git.libreoffice.org/cppunit/+log/refs/heads/master</a></div><div><br></div><div>Cheers,</div><div>Markus<br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>Cheers,</div><div>Markus</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
Florian<br>
<br>
</blockquote></div></div>
</blockquote></div></div>