[Kmymoney-devel] Review Request 122441: Fix build with gcc-4.9 and gmp <= 5.1.3

[Christian David]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122441/#review75559
-----------------------------------------------------------

Ship it!


Builds with gcc 4.8.3 and gmp 5.1.3 on openSuse 13.2.


libalkimia/alkvalue.h
<https://git.reviewboard.kde.org/r/122441/#comment52270>

    Can you add
    
    /** @todo When gmp version is higer than 5.1.3, remove cstddef include */
    
    for me.


- Christian David


On Feb. 5, 2015, 3:28 nachm., Heiko Becker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122441/
> -----------------------------------------------------------
> 
> (Updated Feb. 5, 2015, 3:28 nachm.)
> 
> 
> Review request for KMymoney and Cristian Oneț.
> 
> 
> Repository: alkimia
> 
> 
> Description
> -------
> 
> From https://gcc.gnu.org/gcc-4.9/porting_to.html:
> "The <cstddef> header was updated for C++11 support and this breaks
> some libraries which misuse macros meant for internal use by GCC
> only. For instance with GMP versions up to 5.1.3, you may see:
> 
> /usr/include/c++/4.9.0/cstddef:51:11: error: ‘::max_align_t’ has not
> been declared using ::max_align_t;
> 
> A workaround until libraries get updated is to include <cstddef> or
> <stddef.h> before any headers from that library."
> 
> 
> Diffs
> -----
> 
>   libalkimia/alkvalue.h 7c02403 
> 
> Diff: https://git.reviewboard.kde.org/r/122441/diff/
> 
> 
> Testing
> -------
> 
> Successfully built with gcc-4.9.2 and gmp-5.1.3.
> 
> 
> Thanks,
> 
> Heiko Becker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20150207/ed99d97b/attachment.html>