vimalapugazhan / pe

0 stars 0 forks source link

Size of index for summarise command #7

Open vimalapugazhan opened 5 months ago

vimalapugazhan commented 5 months ago

image.png

image.png

As previously put forth, using summarise command /to 100 even with total of 3 entries in the list still gives a correct output however using larger numbers outputs an error.

nus-se-bot commented 5 months ago

Team's Response

The user is encountering a similar bug that causes integer overflow

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

List command amount to list does not work

Even with 2 items in the expenses list, inputting "list /a 5" or "list /a 1000000000" shows all the items in the list. However "list /a 100000000000000" does not work as intend

image.png


[original: nus-cs2113-AY2324S2/pe-interim#1673] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

The input provided exceeds the amount that an int variable can handle, causing integer overflow. As per PE guidelines, this is not a bug

image.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]