> On Feb. 7, 2015, 1:28 nachm., Christian David wrote:
> > libalkimia/alkvalue.h, line 23
> > <https://git.reviewboard.kde.org/r/122441/diff/1/?file=347156#file347156line23>
> >
> >     Can you add
> >     
> >     /** @todo When gmp version is higer than 5.1.3, remove cstddef include 
> > */
> >     
> >     for me.

Markdown error: there are two more stars of course: ```/** @todo When gmp 
version is higer than 5.1.3, remove cstddef include */```


- Christian


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


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
> 
>

_______________________________________________
KMyMoney-devel mailing list
KMyMoney-devel@kde.org
https://mail.kde.org/mailman/listinfo/kmymoney-devel

Reply via email to