Actions
Feature #318
openSource Code des "wlmscpfs" aufräumen/vereinfachen
Status:
New
Priority:
Low
Assignee:
Jan Schlamelcher
Category:
-
Target version:
-
Start date:
Due date:
% Done:
0%
Estimated time:
Module:
dcmwlm
Operating System:
Compiler:
Description
beim Erstellen eines DICOM Conformance Statements aufgefallen
Der ''wlmscpfs'' ist stellenweise sehr umständlich implementiert, z. B. gibt es separate Methoden für Sequence- und Non-Sequence-Attribute oder es gibt für jedes Matching-Attribut (nicht nur für jeden Typ!) eine separate Methode.
An anderer Stelle gibt es Code wie den folgenden:
for (i = 0; i < max; i++) { switch (i) { case 0: /* ... */ break; case 1: /* ... */ break; /* etc. */ } }
Daneben wird sehr viel mit C-Strings (anstatt OFString) hantiert. Das sind natürlich alles Kandidaten für Speicherlecks, Buffer Overflows und dergleichen.
Actions