← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/ai_coverity into lp:widelands

 

I am not familiar with the AI code, but I think there is no need for heap allocation (i.e. using the operator new). It should be enough to allocate 'path' on the stack like so:

Path path;

That way it will be destroyed automatically once it goes out of scope.
-- 
https://code.launchpad.net/~widelands-dev/widelands/ai_coverity/+merge/294132
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/ai_coverity into lp:widelands.


References