Re: C++ Extensions

(sorry for top posting) Hi Sara, I think such will be just fine for RC2, unless Anatol got any object, go ahead. -K On Aug 20, 2015 6:49 PM, "Sara Golemon" <pollita@php.net> wrote: > On Wed, Aug 19, 2015 at 1:10 PM, Stanislav Malyshev <smalyshev@gmail.com> > wrote: > >> A) Adding those defines prior to including stdint.h > >> B) Expecting C++ ext devs to define those before including php.h > >> C) Modifying zend_long.h to use things like > >> std::numeric_limits<int64_t>() when __cplusplus is defined > >> > >> A may (potentially) have unexpected side-effects > >> B has precedent (see ext/intl/config.m4) > >> C makes the ifdef block that's already there notably more complex > > > > I'd rather have everything solved by php.h than having magic > > incantations that one has to do prior to it. I would prefer A unless > > there's really something wrong with it that. Given that these macros are > > on their way out anyway: > > https://sourceware.org/bugzilla/show_bug.cgi?id=15366 > > I'm not sure it's that dangerous to define them in the meantime. > > > I agree! :) Though I didn't realize they were deprecated anyway. > > Second question, is it too late to put it in 7.0.0? > Happy enough to put it behind #ifdef __cplusplus to limit the exposed > surface area. > > -Sara > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > >