Closed Bug 1644642 Opened 5 years ago Closed 5 years ago

Simplify Erase function in Hashstore.cpp

Categories

(Toolkit :: Safe Browsing, task)

task

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: sg, Assigned: sg)

Details

Attachments

(2 files)

The Erase function template in Hashstore.cpp is unnecessarily complicated, as there is a function in nsTArray_Impl that does exactly the same, which can be called here.

Pushed by sgiesecke@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c96672fb6812 Use iterator-based RemoveElementsAt overload. r=gcp
Pushed by abutkovits@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f985274cf995 Fix ArrayType of nsTArray_Impl iterators. a=fix. CLOSED TREE
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: