Project

General

Profile

Feature #10098

Expand container_algorithms and necessary maintenance

Added by Kyle Knoepfel over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Kyle Knoepfel
Target version:
art - 1.18.03
Start date:
09/10/2015
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:
5.00 h
Duration:

Description

It would be helpful to have the following algorithms:
  • cet::transform_all
  • cet::copy_all_if

both of which wrap the corresponding std containers. It might be worth creating the wrapped std objects for all algorithms in <algorithm>.

We should also consider deprecating:

  • cet::make_unique in favor of the std version
  • art::vectorTransform in favor of the cet::transform_all version above

History

#1 Updated by Christopher Green over 4 years ago

  • Status changed from New to Accepted
  • Estimated time set to 2.00 h

Accepted, excepting the general request: we will be implementing only cet::transform_all() and copy_if_all(). We will deprecate cet::make_unique() in favor of std::make_unique(), and art::vectorTransform().

One notes that cet::*_all() will be rendered obsolete by C++17's Range Library.

#2 Updated by Kyle Knoepfel over 4 years ago

  • Status changed from Accepted to Resolved
  • Assignee set to Kyle Knoepfel
  • % Done changed from 0 to 100

Implemented with cetlib:a60165f. Pushed onto the feature/Remove-cpp0x branch, which will be merged with the master branch after/during the migration to ROOT6.

#3 Updated by Kyle Knoepfel about 4 years ago

Really implemented this time, with commits cetlib:ee1d93a, cetlib:da1e546 , art:ab24c30 and art:e5319a0.

#4 Updated by Kyle Knoepfel about 4 years ago

  • Target version set to 1.18.03

#5 Updated by Kyle Knoepfel about 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF