search

sonible / VCTR

A powerful C++ 20 wrapper around your favorite standard library containers
https://sonible.github.io/VCTR/
Other
25 stars 1 forks source link

Vector: Add constructor that takes an allocator #107

Closed JanosGit closed 1 year ago

JanosGit commented 1 year ago

Looking at the test fails it seems like gcc tries to use the new allocator constructor when creating a Vector from a std::list in the unit cases - don't know why it does so

iromur commented 1 year ago

Hmm, cannot say much here… But happy to approve if that makes sense.

JanosGit commented 1 year ago

Still don't understand it entirely, but I found a fix for it and also added a unit test for the new constructor along the way