# Remove use of Boost smart pointers in meas extensions

XMLWordPrintable

#### Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s: None
• Labels:
None
• Story Points:
1
• Sprint:
DRP X16-3
• Team:
Data Release Production

#### Description

Removal of boost smart pointers in DM-5879 missed some meas extensions which are not built as part of lsst_distrib. Namely: meas_extensions_shapeHSM, meas_extensions_simpleShape and meas_extensions_photometryKron.
Update these too.

#### Activity

Hide
Paul Price added a comment -

Changes are two lines with s/boost/std/ — fine.

Show
Paul Price added a comment - Changes are two lines with s/boost/std/ — fine.
Hide
John Swinbank added a comment -

Sorry for the dumb question, but – I don't see any outstanding pull requests against _shapeHSM, I don't see any tickets that refer to it and to Boost smart pointers, and the last merge to master was nearly a month ago. In what sense has it been taken care of?

Show
John Swinbank added a comment - Sorry for the dumb question, but – I don't see any outstanding pull requests against _shapeHSM , I don't see any tickets that refer to it and to Boost smart pointers, and the last merge to master was nearly a month ago. In what sense has it been taken care of?
Hide
Pim Schellart [X] (Inactive) added a comment - - edited

In the sense that I see:

 commit e378129462947a18d3da95100c878173f3f72207 Author: Paul Price  Date: Wed May 4 20:08:21 2016 -0400    remove use of boost:: in favour of std::    The one exception is ImageConverter._owner, because that's passed  to GalSim, which uses boost::shared_ptr. 

which I think came in with the rebase on tickets/DM-2141. I don't know which ticket this belongs to, or if it has been merged to master. Just assumed that it was on its way in.

Show
Pim Schellart [X] (Inactive) added a comment - - edited In the sense that I see: commit e378129462947a18d3da95100c878173f3f72207 Author: Paul Price <price@astro.princeton.edu> Date: Wed May 4 20:08:21 2016 -0400   remove use of boost:: in favour of std:: The one exception is ImageConverter._owner, because that's passed to GalSim, which uses boost::shared_ptr. which I think came in with the rebase on tickets/ DM-2141 . I don't know which ticket this belongs to, or if it has been merged to master. Just assumed that it was on its way in.
Hide
John Swinbank added a comment -

Ah, got it – looks like it'll get merged with DM-2141. Fine. Thanks.

Show
John Swinbank added a comment - Ah, got it – looks like it'll get merged with DM-2141 . Fine. Thanks.
Hide
Pim Schellart [X] (Inactive) added a comment -

The standard library on older OSX releases seems to be broken which causes various issues.

Show
Pim Schellart [X] (Inactive) added a comment - The standard library on older OSX releases seems to be broken which causes various issues.

#### People

Assignee:
Pim Schellart [X] (Inactive)
Reporter:
Pim Schellart [X] (Inactive)
Reviewers:
Paul Price
Watchers:
John Swinbank, Paul Price, Pim Schellart [X] (Inactive)