search

Geode-solutions / OpenGeode

Open source framework for representing and manipulating geometric models
https://geode-solutions.com/opengeode
MIT License
149 stars 11 forks source link

fix(Basic): add container concatenation in algorithms #1009

Closed panquez closed 2 weeks ago

BotellaA commented 2 weeks ago

Tu peux aussi faire absl::c_insert( Values, std::backinserter(container))

Tu pourrais faire une version en c_insert et une en c_move si on veut éviter les copies et voler la propriété des values

C'est pt pas c_insert mais c_copy, je ne sais plus

github-actions[bot] commented 2 weeks ago

clang-tidy review says "All clean, LGTM! :+1:"

panquez commented 2 weeks ago

Tu peux aussi faire absl::c_insert( Values, std::backinserter(container)) Tu pourrais faire une version en c_insert et une en c_move si on veut éviter les copies et voler la propriété des values

C'est pt pas c_insert mais c_copy, je ne sais plus

Yes it is c_copy. Update done!

BotellaA commented 2 weeks ago

:tada: This PR is included in version 15.2.3-rc.3 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

BotellaA commented 1 week ago

:tada: This PR is included in version 15.2.3 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: