Refactored loadDose
Description
Description
Details
Details
- Provenance
• strubelp Authored on Jul 27 2017, 12:41 PM • strubelp Pushed on Aug 3 2017, 4:03 PM - Parents
- rRTTB5a5eb0f32d5b: Removed unneeded functions during improvement for core code coverage.
- Branches
- Unknown
- Tags
Event Timeline
Comment Actions
Danke für die gute Arbeit. Weniger code ist manchmal mehr 😄
Nur drei kleine Anmerkungen. Kannst danach in den master mergen.
| /apps/DoseAcc/CMakeLists.txt | ||
|---|---|---|
| 3 | können evtl RTTBITKIO RTTBDicomIO RTTBHelaxIO rausfliegen als dependencies bei den ganzen Apps? Das müsste doch jetzt durch RTTBUtilsIO abgedeckt werden. Oder fehlt dann was? | |
| /code/io/utils/rttbDoseLoader.cpp | ||
| 77 | hier bitte noch args in der doku erklären | |
| 81 | ich würde hier die ausgaben rausschmeißen. Evtl in den Apps einfügen (wenn du denkst, es sei sinnvoll) | |