Bug 141565

Summary:

Add an initializer list constructor to Deque

Product:

WebKit

Reporter:

Anders Carlsson <andersca>

Component:

New Bugs

Assignee:

Anders Carlsson <andersca>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

benjamin, cmarcelo, commit-queue

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	kling: review+

Description Anders Carlsson 2015-02-13 10:20:38 PST

Add an initializer list constructor to Deque

Comment 1 Anders Carlsson 2015-02-13 10:25:18 PST

Created attachment 246530 [details]
Patch

Comment 2 Andreas Kling 2015-02-13 10:29:25 PST

Comment on attachment 246530 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=246530&action=review

r=me

> Source/WTF/wtf/Deque.h:281
> +    for (auto& element : initializerList)
> +        append(element);

In a perfect world this would use some kind of reserveInitialCapacity() + uncheckedAppend() combo.

Comment 3 Anders Carlsson 2015-02-13 10:46:12 PST

Committed r180059: <http://trac.webkit.org/changeset/180059>