304982 - CPP_THROW_NEW should be defined to throw() even if exceptions are enabled

Status: RESOLVED FIXED

(Firefox Build System :: General, defect, P2)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

Our operator new implementations are still returning null instead of throwing an
exception when exceptions are enabled. So the macro should be defined as throw()
even with --enable-cpp-exceptions.

Comment 1

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: patch

Comment 2

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in configure.in;
/cvsroot/mozilla/configure.in,v  <--  configure.in
new revision: 1.1508; previous revision: 1.1507
done

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

Comment on attachment 192954 [details] [diff] [review]
patch

OK, this annoyed me on the 1.8 branch today, would be nice to have it there too.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in configure.in;
/cvsroot/mozilla/configure.in,v  <--  configure.in
new revision: 1.1503.2.81; previous revision: 1.1503.2.80
done