Details

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

      Description

      Replace by deleting copy constructor.

        Issue Links

          Activity

          Hide
          swinbank John Swinbank added a comment -

          Pim Schellart suggests I might want to weigh in here, so: my preference is for explicit deletion rather than a mixin. But it's sufficiently slight that I wouldn't have bothered mentioning it if Pim hadn't asked, so I'm certainly not going to complain if you jump in the other direction.

          Show
          swinbank John Swinbank added a comment - Pim Schellart suggests I might want to weigh in here, so: my preference is for explicit deletion rather than a mixin. But it's sufficiently slight that I wouldn't have bothered mentioning it if Pim hadn't asked, so I'm certainly not going to complain if you jump in the other direction.
          Hide
          pschella Pim Schellart added a comment -

          Ok, so given nobody (including me) has a strong preference, and assuming John Parejko doesn't object, I'm going with the explicit delete option. It seems to offer the least amount of possible limitations in the future.

          Show
          pschella Pim Schellart added a comment - Ok, so given nobody (including me) has a strong preference, and assuming John Parejko doesn't object, I'm going with the explicit delete option. It seems to offer the least amount of possible limitations in the future.
          Hide
          Parejkoj John Parejko added a comment -

          Given that my only objection was overruled (not without good reason!), this is ok to merge.

          Show
          Parejkoj John Parejko added a comment - Given that my only objection was overruled (not without good reason!), this is ok to merge.
          Hide
          pschella Pim Schellart added a comment -

          Merged after final CI run. Thank you for the review!

          Show
          pschella Pim Schellart added a comment - Merged after final CI run. Thank you for the review!
          Hide
          swinbank John Swinbank added a comment -

          SPs contained in parent issue.

          Show
          swinbank John Swinbank added a comment - SPs contained in parent issue.

            People

            • Assignee:
              pschella Pim Schellart
              Reporter:
              pschella Pim Schellart
              Reviewers:
              John Parejko
              Watchers:
              Jim Bosch, John Parejko, John Swinbank, Pim Schellart
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development

                  Agile