search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

Delete command sequence diagram #4869

Open nus-se-script opened 5 months ago

nus-se-script commented 5 months ago

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

d:DeleteCommand should be deleted after returning r since it is no longer used and ready for garbage collection.

image.png

Creation of Command and calling of Command::execute() in LogicManager. Command object will not be used after returning from Command::execute() image.png

[original: nus-cs2103-AY2324S2/pe-interim#4375] [original labels: type.DocumentationBug severity.VeryLow]

iynixil commented 5 months ago

Team's Response

Thanks for raising this. The team believes that omission of the "X" marker does not affect the understanding of the diagram.

Duplicate status (if any):

--