Memory leaks about sketchup-api-c-wrapper HOT 6 CLOSED

Yes, I agree. There was a switch towards new and delete as @jiminy-billy-bob reported compiler issues on Visual Studio. @jiminy-billy-bob - perhaps you could change your compiler to C++11 compatible version and we can fix this?

The commit that made this change was 903f09e

Will fix throughout the wrapper.

from sketchup-api-c-wrapper.

There was a switch towards new and delete as @jiminy-billy-bob reported compiler issues on Visual Studio.

What issues was that? Is that conversation logged in the issue tracker?

The only thing that is C++11 in my previous examples are std::transform. Avoiding new and delete is doable without C++11. However, C++ is so much better and easier to work with once you utilize C++11. I would not start any new projects these days without at least C++11 support.

from sketchup-api-c-wrapper.

What issues was that? Is that conversation logged in the issue tracker?
No, not logged anywhere where I can see. I was still getting used to Github at that point. In hindsight, I can see that my original implementation was not correct either. Will be my next commit.

from sketchup-api-c-wrapper.

The conversation is here : #2

from sketchup-api-c-wrapper.

I see - the message doesn't relate to C++11 though. It's simply about non-standard C++ extensions that allowed variable C-type arrays.

With modern C++ you would typically use std::vector for variable length continuous data, and std::array for fixed length arrays. The benefit of std::array for fixed length arrays over C style arrays is that they are compatible with the STL containers.

from sketchup-api-c-wrapper.

Issue resolved with ffc704c

from sketchup-api-c-wrapper.