refactor: use std::map::try_emplace()
over std::map::insert()
(#46761)
refactor: prefer std::map::try_emplace() over std::map::insert()
This commit is contained in:
parent
8fa7d324d1
commit
b40b4dc015
10 changed files with 15 additions and 21 deletions
|
@ -105,10 +105,8 @@ void UsbChooserContext::InitDeviceList(
|
|||
std::vector<device::mojom::UsbDeviceInfoPtr> devices) {
|
||||
for (auto& device_info : devices) {
|
||||
DCHECK(device_info);
|
||||
if (ShouldExposeDevice(*device_info)) {
|
||||
devices_.insert(
|
||||
std::make_pair(device_info->guid, std::move(device_info)));
|
||||
}
|
||||
if (ShouldExposeDevice(*device_info))
|
||||
devices_.try_emplace(device_info->guid, std::move(device_info));
|
||||
}
|
||||
is_initialized_ = true;
|
||||
|
||||
|
@ -289,7 +287,7 @@ void UsbChooserContext::OnDeviceAdded(
|
|||
DCHECK(!devices_.contains(device_info->guid));
|
||||
if (!ShouldExposeDevice(*device_info))
|
||||
return;
|
||||
devices_.insert(std::make_pair(device_info->guid, device_info->Clone()));
|
||||
devices_.try_emplace(device_info->guid, device_info->Clone());
|
||||
|
||||
// Notify all observers.
|
||||
for (auto& observer : device_observer_list_)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue