Ticket #558 (closed: fixed)
Remove the Workspace2D setX methods that copy a vector internally
Reported by: | Russell Taylor | Owned by: | Russell Taylor |
---|---|---|---|
Priority: | major | Milestone: | Iteration 17 |
Component: | Keywords: | ||
Cc: | Blocked By: | ||
Blocking: | Tester: |
Description
This is never necessary, when a workspace is initialised all the vectors of its storage are created even if their just filled with zeroes. So the correct access approach is to take a reference to the vector and modify that.
Change History
comment:2 Changed 11 years ago by Russell Taylor
(In [2395]) Eliminated the variant of Workspace2D::setX/setData that takes a const reference to a vector and then copies it inside. This is inefficient since the vectors already exist once the workspace is initialized. The correct way is to take a reference to the MantidVec and assign directly to that. Re #558.
Note: See
TracTickets for help on using
tickets.