Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-6056

Remove PTR and CONST_PTR macros

    XMLWordPrintable

    Details

      Description

      This ticket implements the solution proposed in RFC-179, replacing PTR and CONST_PTR with std::shared_ptr<T> and std::shared_ptr<const T> respectively.

        Attachments

          Issue Links

          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-179 Remove PTR and CONST_PTR macros

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            CONST_PTR seems to be used in meas_base, meas_modelfit, meas_algorithms, meas_extensions_photometryKron, ip_diffim, meas_base, ctrl_events, and daf_persistence.

            Additionally PTR is used in meas_extensions_shapeHSM, meas_astrom, meas_deblender, obs_subaru, meas_extensions_astrometryNet, shapelet, and meas_extensions_psfex.

            Show
            tjenness Tim Jenness added a comment - CONST_PTR seems to be used in meas_base, meas_modelfit, meas_algorithms, meas_extensions_photometryKron, ip_diffim, meas_base, ctrl_events, and daf_persistence. Additionally PTR is used in meas_extensions_shapeHSM, meas_astrom, meas_deblender, obs_subaru, meas_extensions_astrometryNet, shapelet, and meas_extensions_psfex.
            Hide
            Permalink
            yusra Yusra AlSayyad added a comment -

            Looks like we never finished implementing this RFC. Matthias Wittgen might be interested

            Show
            yusra Yusra AlSayyad added a comment - Looks like we never finished implementing this RFC. Matthias Wittgen might be interested
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Sorry for the delay! I have only minor comments, mostly involving formatting.

            Show
            krzys Krzysztof Findeisen added a comment - Sorry for the delay! I have only minor comments, mostly involving formatting.

              People

              Assignee:
              wittgen Matthias Wittgen
              Reporter:
              pschella Pim Schellart [X] (Inactive)
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              Jim Bosch, Krzysztof Findeisen, Tim Jenness, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.