| commit | e2f38aca465ef7edeff914102d848dbe784e4b85 | [log] [tgz] |
|---|---|---|
| author | Michael Wright <michaelwr@google.com> | Tue Jan 19 00:59:08 2021 +0000 |
| committer | Michael Wright <michaelwr@google.com> | Tue Jan 19 00:59:08 2021 +0000 |
| tree | 927ef3621122b07d72f1595d39387def2bd2892e | |
| parent | 88ce0fa5e59d2b19ece38708672baebbc4a19bc0 [diff] |
Fix for-loop type. Technically the iterator provides a std::pair<const KEY, VALUE>&, so we're constructing a std::pair<KEY, VALUE> from it and then taking a const reference to that. This creates an unnecessary copy but then obscures that by taking a reference to it; we should either take a reference to the iterator's actual type or use the value type of the temporary we construct directly. Bug: N/A Test: compile Change-Id: I0ed11c1a80a5ac419f74f100573d4a2f042141fd