On 11-12-14 13:09 , Paolo Carlini wrote:
Hi,

Hi,

Revision 172595:

http://gcc.gnu.org/viewcvs?view=revision&revision=172595

added NULL to config/locale/generic/c_locale.h on
google/gcc-4_6-branch. But<cstddef>  isn't included, which lead to
build failure since NULL isn't defined.  This patch
includes<cstddef>.  OK for google/gcc-4_6-branch?

If you are interested in my personal opinion, I have of course
nothing to do with this branch, I think this code should not simply
use NULL. After all, its C++, right? ;)

Seems reasonable. Jing, 172595 was a patch from you. Would it be the same if you just used 0 instead of NULL?


Diego.

Reply via email to