Ticket #7176 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Remove EventWorkspace::doneAddingEventLists method

Reported by: Russell Taylor Owned by: Nick Draper
Priority: minor Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By: #6191
Blocking: Tester: Mathieu Doucet

Description

It used to be needed to fix up the axis and spectraDetectorMap. It doesn't do anything important any more.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to accepted
  • Owner set to Nick Draper

branch: feature/7176_remove_doneAddingEventLists

to test:

  • this is a code removal exercise, there are no specific tests, other than this must be checked into develop and all the automated tests pass.

comment:2 Changed 7 years ago by Nick Draper

re #7176 removing the method doneAddingEventLists()

Changeset: 920ffe81a54cbf03cb9ef47917f056113c616d36

comment:3 Changed 7 years ago by Nick Draper

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:4 Changed 7 years ago by Nick Draper

  • type changed from task to enhancement

comment:5 Changed 7 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Tester set to Mathieu Doucet

comment:6 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

All tests pass, and there's no trace of that method anywhere in the code.

comment:7 Changed 7 years ago by Anders Markvardsen

Remove reference to doneAddingEventLists. re #7393

No longer needed see #7176

Changeset: a7d604ab9069cb28817048db41e8fff33267f2d3

comment:8 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8022

Note: See TracTickets for help on using tickets.