Modify

Ticket #11867 (closed Feature Requests: fixed)

Opened 15 months ago

Last modified 15 months ago

small_vector should have constructor and assignment operator taking other small_vector

Reported by: cornedbee Owned by: igaztanaga
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

Change History

comment:1 Changed 15 months ago by igaztanaga

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 15 months ago by igaztanaga

  • Status changed from new to closed
  • Resolution set to fixed

Added support for constructors and assignments from small_vector_base in commit:

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

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


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

 
Note: See TracTickets for help on using tickets.