maxmustermann/terminal
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Make loop in IInputEvent more consistent and modern (#4959)

Browse source 
In IInputEvent, there are two for loops. One is a range based loop operating on "records", and the other is a classic for-loop doing the same. For consistency, the for-loop was changed into the more modern variation via a compiler refactoring, which has the exact same behavior as the other for-loop in the main function.

Yes, of course this was tested manually and with the unit tests.

# Validation Steps
Unit testing passed. In addition, for the manual validation tests, I compared the output for sample values between the two loops ensuring the same results.
This commit is contained in:
pi1024e 2020-03-17 13:52:33 -04:00 committed by GitHub
parent 1d8c5bae35
commit 7621994b46
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/types/IInputEvent.cpp
View file

@ -29,7 +29,7 @@ std::deque<std::unique_ptr<IInputEvent>> IInputEvent::Create(gsl::span<const INP
{
std::deque<std::unique_ptr<IInputEvent>> outEvents;
for (auto& record : records)
for (const auto& record : records)
{
outEvents.push_back(Create(record));
}
@ -46,9 +46,9 @@ std::deque<std::unique_ptr<IInputEvent>> IInputEvent::Create(gsl::span<const INP
std::deque<std::unique_ptr<IInputEvent>> IInputEvent::Create(const std::deque<INPUT_RECORD>& records)
{
std::deque<std::unique_ptr<IInputEvent>> outEvents;
for (size_t i = 0; i < records.size(); ++i)
for (const auto& record : records)
{
std::unique_ptr<IInputEvent> event = IInputEvent::Create(records.at(i));
std::unique_ptr<IInputEvent> event = Create(record);
outEvents.push_back(std::move(event));
}
return outEvents;