search

nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Confusing diagram #6000

Open nus-se-script opened 1 year ago

nus-se-script commented 1 year 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.

In the below diagram you drew an X to presumably show the deletion of the instance. However, I cannot find the part of the code where you manually deleted the instance from the memory. If you instead meant that Java would have deleted the instance automatically, then you should have drawn the X for other instances and not just this particular instance.

Screenshot 2023-11-17 172530.png

[original: nus-cs2103-AY2324S1/pe-interim#5981] [original labels: severity.Low type.DocumentationBug]

Nid21cs commented 1 year ago

Team's Response

Good catch thank you.

Duplicate status (if any):

--