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

Review Summary: No summary


----------------------------------------
File: APPS/Halux/PPS/ALUFrees/Svg/Kaderdeur.cls.xml
Revision Comment by Tom Vermeulen on 25 March 2020, 12:01 defect
http://subversion02:8060/cru/CR-2022#c3261

  indenten tussen de accolades

Revision Comment by Tom Vermeulen on 25 March 2020, 12:02 defect
http://subversion02:8060/cru/CR-2022#c3262

  Dit is opeetcode. Als er een exception geraised wordt, gaat
  niemand het weten. Is dit het gewenste gedrag? Indien niet: er
  iets mee doen (en testje(s) voor schrijven)

  Reply by Wim Vermeulen on 02 April 2020, 16:19
  > Na een Catch() moet er altijd een "reporting" of een nieuwe
  > throw gedaan worden.
  > In dit geval zegt mijn gevoel dat hier geen try-catch moet
  > staan, want het is aan de gebruiker/oproepende code om zelf
  > te bepalen welke error-handling er nodig is.
  > 
  > Anderszijds lijkt dit meer op een stukje test-code, dan kan
  > je dat best ook zo aangeven (in methodnaam of ev in de
  > comment) of moet je deze testcode verplaatsen naar een
  > zTryout-klasse.
  > Zo niet, dan krijg je de reviewer(s) op uw dak  :-D


---

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

Title: [ICT-1348] SVG-025 ' aanpassen naar $$$NOT() + naamgeving van variabelen uitbreiden
Statement of Objectives:


State: Closed

Summary: 

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