This is a list of all comments for CR-1942.

Review Summary: No summary

General Comment by Tom Vermeulen on 30 January 2020, 08:50 
http://subversion02:8060/cru/CR-1942#c2961

  deze review werd aangemaakt zonder directe link naar het issue
  in kwestie


----------------------------------------
File: TECH/Svg/Basis/Rect.cls.xml
Revision Comment by Tom Vermeulen on 30 January 2020, 11:12 defect
http://subversion02:8060/cru/CR-1942#c2962

  commentaarlijnen kan je beter wegsmijten

Revision Comment by Tom Vermeulen on 30 January 2020, 11:13 
http://subversion02:8060/cru/CR-1942#c2963

  [~WimV] schrijft uw gedacht ook eens neer ivm het meegeven van
  values aan de constructor. Sam en ik hebben het er gisteren
  even over gehad. In theorie is het properder om de constructor
  enkel objecten mee te geven waarop dependencies zijn. Dan moet
  elk basisobject natuurlijk wel een "setparams" of "initialize"
  method krijgen.
  Voordeel: testen duidelijker en conform de rest van de nieuwere
  code
  Nadeel: Sam gaat wat door zijn code moeten crossen om het over
  te zetten (dacht dat het bij het DomElement wel al zo was)


---

ID: CR-1942 http://subversion02:8060/cru/CR-1942

Title: [ICT-1128]
Statement of Objectives:


State: Closed

Summary: 

Author: Sam Van Hoey
Moderator: Sam Van Hoey
Reviewers: (0 active, 2 completed*)
     Tom Vermeulen   (*)
     Wim Vermeulen   (*)