Opened on 01/16/2018 at 03:58:58 PM

Closed on 01/25/2018 at 04:19:54 PM

#6279 closed change (fixed)

[emscripten] Make Map<> movable

Reported by: hfiguiere Assignee: hfiguiere
Priority: Unknown Milestone:
Module: Core Keywords:
Cc: sergz Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

​https://codereview.adblockplus.org/29677755/

Description

Background

Map<> should be movable.

What to change

Map<> and all the associated type should allow to be movable. This to have a Map<> as value type.

Attachments (0)

Change History (4)

comment:1 Changed on 01/20/2018 at 05:28:38 PM by hfiguiere

  • Owner set to hfiguiere

comment:2 Changed on 01/23/2018 at 09:22:20 PM by hfiguiere

  • Cc sergz added
  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 01/25/2018 at 04:19:15 PM by abpbot

A commit referencing this issue has landed:
​Issue 6279 - Make HashContainer movable and optimize resizing by moving entries.

comment:4 Changed on 01/25/2018 at 04:19:54 PM by hfiguiere

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

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from hfiguiere.
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.