Page MenuHomeFeedback Tracker

Update set<T>::RemoveItem to return bool
Closed, ResolvedPublic

Description

I am not sure why it was not done, but the remove operation should return if the item was actually removed or not, because the info is available. e.g. Updating it to this:

bool RemoveItem(T value)
{
    int remove_index = Find(value);

    if ( remove_index >= 0 )
    {
        Remove(remove_index);
        return true;
    }
    return false;
}

Details

Severity
Tweak
Resolution
Open
Reproducibility
N/A
Operating System
Windows 10 x64
Category
General

Event Timeline

Arkensor created this task.May 1 2023, 9:41 AM
Geez changed the task status from New to Assigned.May 3 2023, 10:50 AM
Geez closed this task as Resolved.Jun 12 2023, 11:11 AM
Geez claimed this task.
Geez added a subscriber: Geez.

Resolved in one of the future updates.