Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9210f10d authored by Patrick Williams's avatar Patrick Williams
Browse files

Avoid unnecessary WindowInfo copies 

Bug: 352744079
Flag: EXEMPT refactor
Test: presubmits
Change-Id: Iad7cefdf8fb2c1bb75b2f8b4970e3de422529523
parent e17dfdd3

services/inputflinger/dispatcher/InputDispatcher.cpp

+2 −6
Original line number Diff line number Diff line
@@ -836,13 +836,9 @@ bool isStylusActiveInDisplay(ui::LogicalDisplayId displayId, 
}



Result<void> validateWindowInfosUpdate(const gui::WindowInfosUpdate& update) {

    struct HashFunction {

        size_t operator()(const WindowInfo& info) const { return info.id; }

    };



    std::unordered_set<WindowInfo, HashFunction> windowSet;

    std::unordered_set<int32_t> windowIds;

    for (const WindowInfo& info : update.windowInfos) {

        const auto [_, inserted] = windowSet.insert(info);

        const auto [_, inserted] = windowIds.insert(info.id);

        if (!inserted) {

            return Error() << "Duplicate entry for " << info;

        }