kd-tree: Deallocate Point pointer vector after build().

[nh2]

This

vector<const Point_d*> data;

is no longer needed after being .clear()ed at the end of build(). This can save a good amount of memory, a Point_d* on a 64-bit system is 8 bytes; almost as large as a typical 3-float Point_d.

There's no point keeping its capacity, since any call to build() will efficiently .reserve() it anyway.

Please use the following template to help us managing pull requests.

Summary of Changes

Describe what your pull request changes to CGAL (this can be skipped if it solves an issue already in the tracker or if it is a Feature or Small Feature submitted to the CGAL Wiki).

Release Management

• Affected package(s):
• Issue(s) solved (if any): fix #0000, fix #0000,...
• Feature/Small Feature (if any):
• Link to compiled documentation (obligatory for small feature) wrong link name to be changed
• License and copyright ownership:

[sloriot]

Successfully tested in CGAL-6.0-Ic-228

[lrineau]

Also merged. Thanks @nh2!