The "new" operator definition collides with the standard (?) one about arduinojson HOT 6 CLOSED

Hello Mario,

The "placement new" that I'm using is part of the standard library and in most environments you just need to add the following line:

#include <new>

What you see in my PlacementNew.hpp is a workaround for the Arduino environment that lacks many standard features.

Please try to replace the following line in PlacementNew.hpp:

#ifdef ARDUINO

by

#if 0

so as to disable the workaround for Arduino.

from arduinojson.

Hi again,

I'd like to reproduce the bug on my computer.
What IDE are you using?
Is it the "LinkIt SDK for Arduino"?

from arduinojson.

Yes, that's the SDK I'm using (on VS 2013, but I don't think matters).

Benoit, I think I got it, although I wouldn't deserve any merit, because my 99% of luck and 1% of ability.
Simply I commented out the "inline" declaration in your PlacementNew, as well as the "#include " at the bottom. Doing that I guess the module is completely useless.
However, simply "removing" the PlacementNew content, the compilation raises several errors like:

JsonArray.hpp:In file included from
JsonArray.cpp:from
List.hpp:In member function 'ArduinoJson::Internals::ListNode* ArduinoJson::Internals::List::createNode() [with T = ArduinoJson::JsonVariant]'
JsonArray.cpp:instantiated from here
List.hpp:56: error: no matching function for call to 'operator new(unsigned int, void_&)'
:void_ operator new(unsigned int)

So, my attempt was to place an "#include " in the List.hpp module, and nowhere else. That's working fine.

P.S. Today I'll leave for a short holiday, so I won't able to put my hands on in a brief...

from arduinojson.

Hi Mario,
Any update on that issue?

from arduinojson.

Huh?...There's no "update", because was already solved as above descripted: the library works fine.
The compilation (and thus the runtime) are working fine by simply:

• commenting out the "inline" directive within the PlacementNew module;
• adding (substituting the above one) the "#include " in the List.hpp module.

Should I perform any other test?

from arduinojson.

OK, thanks for the clarification.

You're not the only one having issues with the placement new definitions.
I think I'll just remove it from the next release.

from arduinojson.