robotology / yarp

YARP - Yet Another Robot Platform
http://www.yarp.it
Other
524 stars 195 forks source link

Remove dynamic allocations from "impl" layer of motor control interfaces #1684

Open valegagge opened 6 years ago

valegagge commented 6 years ago

In order to fix the issue #1660 , I allocated temporary buffers dinamiccally.

As suggest by @lornat75 , It is better remove dynamic allocations from "impl" layer of yarp despite of performance test results, for two reasons:

Therefore I'll develop a simple memory manager to solve this issue; it could be used in other contex also.

see PR https://github.com/robotology/yarp/pull/1668

drdanz commented 6 years ago

If the issue is related to multi-threading, perhaps declaring the buffers thread_local can help? Maybe not, I don't know the details of the issue... :stuck_out_tongue_closed_eyes: