cppunit Patch feature-modernize
Florian Becker
fb at vxapps.com
Fri Oct 1 10:52:21 UTC 2021
Hi,
> Am 01.10.2021 um 12:47 schrieb Markus Mohrhard <markus.mohrhard at googlemail.com>:
>
> Hey,
>
> On Fri, Oct 1, 2021 at 4:02 AM Florian Becker <fb at vxapps.com <mailto:fb at vxapps.com>> wrote:
> Hi Markus,
>
> thanks a lot.
>
> Is there a list of compiler, which are supported? Also, is there any reason, why cppunit cannot upgrade to c++17?
>
> 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.
Ok, understand. Mostly this can be done by compiler checks and flag the functionality. But i think you already know that...
>
>
> I would like to create a cmake definition - do we handle that the same way?
>
> Have you had a look at the feature/cmake branch?
Not yet, never thought there would be such a branch. I’ll check it. Thanks for the information.
Regards,
Florian
>
> Cheers,
> Markus
>
>
> Regards,
> Florian
>
>> Am 30.09.2021 um 21:56 schrieb Markus Mohrhard <markus.mohrhard at googlemail.com <mailto:markus.mohrhard at googlemail.com>>:
>>
>>
>> Hey Florian,
>>
>> On Fri, Oct 1, 2021 at 1:52 AM Markus Mohrhard <markus.mohrhard at googlemail.com <mailto:markus.mohrhard at googlemail.com>> wrote:
>> Hey Florian,
>>
>> On Tue, Sep 28, 2021 at 5:59 PM Florian Becker <fb at vxapps.com <mailto:fb at vxapps.com>> wrote:
>> Hello,
>>
>> i have patched the libcppunit 1.15.1 to modernize and use clang flags
>> with highest flag settings. Hopefully you can integrate this patch
>> inside master. https://github.com/VXAPPS/cppunit/tree/feature-modernize <https://github.com/VXAPPS/cppunit/tree/feature-modernize>
>>
>> If you have suggestions, please let me know.
>>
>> I'll have a look during the next few days and will let you know if I have any questions or comments
>>
>> Thanks for your patches. The summary is:
>>
>> * first patch was used without modifications
>> * second patch was split up into smaller commits:
>> * add [[noreturn]]
>> * add override annotations
>> * switch from throw to noexcept
>> * remove semicolons
>> * a complete removal of the old casting macros by me instead of your addition to the portability header
>> * replace 0 with nullptr
>> * disable a few simple compiler warnings
>>
>> 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.
>> 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.
>>
>> * I'll have to think a bit about the third and fourth patch. If I merge them I'll let you know.
>>
>> You can find the full code that I have pushed at:
>>
>> https://git.libreoffice.org/cppunit/+log/refs/heads/master <https://git.libreoffice.org/cppunit/+log/refs/heads/master>
>>
>> Cheers,
>> Markus
>>
>>
>>
>> Cheers,
>> Markus
>>
>>
>> Regards,
>> Florian
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20211001/dd8c80dc/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4946 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20211001/dd8c80dc/attachment.bin>
More information about the LibreOffice
mailing list