<div dir="ltr"><div>Hey,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Oct 1, 2021 at 4:02 AM Florian Becker <<a href="mailto:fb@vxapps.com">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"><div dir="auto">Hi Markus,<div><br></div><div>thanks a lot.</div><div><br></div><div>Is there a list of compiler, which are supported? Also, is there any reason, why cppunit cannot upgrade to c++17?</div></div></blockquote><div><br></div><div>This would force all dependent projects to use C++17 as well. Already the move to C++11 caused some issues for some older projects using cppunit. Currently I don't see a compelling reason to update for cppunit. The decision to move to C++11 was easier and provided a clear benefit.</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="auto"><div><br></div><div>I would like to create a cmake definition - do we handle that the same way?</div></div></blockquote><div><br></div><div>Have you had a look at the feature/cmake branch?</div><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"><div dir="auto"><div><br></div><div>Regards,</div><div>Florian</div><div><div dir="ltr"><br><blockquote type="cite">Am 30.09.2021 um 21:56 schrieb Markus Mohrhard <<a href="mailto:markus.mohrhard@googlemail.com" target="_blank">markus.mohrhard@googlemail.com</a>>:<br><br></blockquote></div><blockquote type="cite"><div dir="ltr"><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" target="_blank">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>
</div></blockquote></div></div></blockquote></div></div>