Removed & from VolumeType and DoseTypeGy parameters, also changed…
Concern Raised921af23734df
Actions

Description

Removed & from VolumeType and DoseTypeGy parameters, also changed computeSpecificValue so that it returns the value instead of writing it into the measure map

Details

Auditors

hentsch

Provenance

• strubelp	Authored on Jun 12 2017, 3:34 PM
• strubelp	Pushed on Jun 13 2017, 4:13 PM

Event Timeline

• strubelp committed rRTTB921af23734df: Removed & from VolumeType and DoseTypeGy parameters, also changed… (authored by • strubelp).Jun 12 2017, 3:34 PM

hentsch raised a concern with this commit.Jun 19 2017, 10:29 AM

hentsch added a subscriber: hentsch.

hentsch added inline comments.

/code/algorithms/rttbDoseToVolumeMeasureCalculator.cpp
22	kannst du bitte ein Issue anlegen, dass Multithreading hier noch gecheckt werden muss? bitte lieber den Code so lassen und eine exception werfen, wenn _multiThreading = true. Sonst ist es später schwieriger, das wieder einzufügen
28	hier wird mir eine Leerzeile angezeigt. Falls das so ist: bitte löschen
/code/algorithms/rttbMOCxVolumeToDoseMeasureCalculator.cpp
37	lieber static_cast<DoseTypeGy>, das ist explizit: siehe http://www.cplusplus.com/doc/tutorial/typecasting/
/code/algorithms/rttbMOHxVolumeToDoseMeasureCalculator.cpp
36	s.o. (static_cast)
/code/algorithms/rttbVolumeToDoseMeasureCalculator.cpp
22	wie oben: lieber so lassen, wie es ist und exception werfen.

This commit now has outstanding concerns.Jun 19 2017, 10:29 AM

• strubelp marked 5 inline comments as done.Jun 26 2017, 11:44 AM

Commit No Longer Exists

This commit no longer exists in the repository.