Ticket #3137 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Promote coord_t/signal_t to Mantid namespace, integrate in MD stuff

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 29
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

coord_t is a typedef for coordinates; signal_t is a typedef for signals and errors. Currently and for the forseeable future, they will be doubles.

Having typedefs means it is a lot easier to fix issues if e.g. the size of double changes on a particular platform.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

Also, change the case of "coordinate" to "Coordinate" since it is a real class

comment:3 Changed 9 years ago by Janik Zikovsky

(In [12320]) Refs #3137: Used coord_t and signal_t in appropriate spots in MD* classes. Right now both are typedef'd to double but it builds if typedef'd so something else e.g float.

comment:4 Changed 9 years ago by Janik Zikovsky

(In [12322]) Refs #3137: Fix warnings.

comment:5 Changed 9 years ago by Janik Zikovsky

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

comment:6 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:7 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

Refactoring completed without breaking existing tests or top-level Paraview interface.

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3984

Note: See TracTickets for help on using tickets.