Opened 7 years ago

Closed 7 years ago

#11867 closed Feature Requests (fixed)

small_vector should have constructor and assignment operator taking other small_vector

Reported by: Sebastian Redl Owned by: Ion Gaztañaga
Milestone: To Be Determined Component: container
Version: Boost 1.60.0 Severity: Problem
Keywords: Cc:

Description

small_vector<T, N> should be constructible and assignable from small_vector<T, M>. There's really no good reason not to have these functions.

Change History (2)

comment:1 by Ion Gaztañaga, 7 years ago

Maybe not from small_vector<T, M> but from small_vector_base, which might be more flexible.

LLVM's SmallVector, which is the inspiration for small_vector, supports only move construction and assignment from SmallVectorImpl, I can't figure out why copy construction is not supported:

http://llvm.org/docs/doxygen/html/classllvm_1_1SmallVector.html

Thanks for the report.

comment:2 by Ion Gaztañaga, 7 years ago

Resolution: fixed
Status: newclosed

Added support for constructors and assignments from small_vector_base in commit:

https://github.com/boostorg/container/commit/641d1a5d5aca2a60708b886afd1799c81aab80b0

Note: See TracTickets for help on using tickets.