0

Use base::EraseIf()

This patch is just a code simplification.

It's much easier to write:
  base::EraseIf(container, ...);
than:
  container.erase(std::remove_if(container.begin(),
      container.end(), ...), container.end());

Bug: 875665

Change-Id: I099304b13682f73d3fff214314ffa7c5d886bd45
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4881791
Reviewed-by: Yao Xiao <yaoxia@chromium.org>
Commit-Queue: Yao Xiao <yaoxia@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1200365}
This commit is contained in:
Aditya Sharma
2023-09-22 18:25:24 +00:00
committed by Chromium LUCI CQ
parent d3ccd069f4
commit 785e40b292
2 changed files with 3 additions and 4 deletions
AUTHORS
components/browsing_topics

@ -37,6 +37,7 @@ Addanki Gandhi Kishor <kishor.ag@samsung.com>
Adenilson Cavalcanti <a.cavalcanti@samsung.com>
Aditi Singh <a20.singh@samsung.com>
Aditya Bhargava <heuristicist@gmail.com>
Aditya Sharma <a2.sharma@samsung.com>
Adrian Belgun <adrian.belgun@intel.com>
Adrian Ratiu <adrian.ratiu@collabora.corp-partner.google.com>
Adrià Vilanova Martínez <me@avm99963.com>

@ -395,13 +395,11 @@ AnnotatorImpl::ExtractCategoriesFromModelOutput(
// Prune out categories that do not meet the minimum threshold.
if (category_params.min_category_weight() > 0) {
categories.erase(
std::remove_if(categories.begin(), categories.end(),
base::EraseIf(categories,
[&](const std::pair<int32_t, float>& category) {
return category.second <
category_params.min_category_weight();
}),
categories.end());
});
}
// Prune out none weights.