Modify

Opened 2 years ago

Closed 2 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.

Attachments (0)

Change History (2)

comment:1 Changed 2 years ago by Ion Gaztañaga

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 Changed 2 years ago by Ion Gaztañaga

Resolution: fixed
Status: newclosed

Added support for constructors and assignments from small_vector_base in commit:

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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ion Gaztañaga.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.