diff options
author | Rolf Eike Beer <eb@emlix.com> | 2015-12-10 12:31:05 +0100 |
---|---|---|
committer | Alex Blasche <alexander.blasche@theqtcompany.com> | 2015-12-11 08:49:06 +0000 |
commit | 09c85ebde76311ddf46413a38a023f0ffefa5a31 (patch) | |
tree | 9530a483cff1150abff4e587d94ec3410ec09b6b /src/plugins/canbus/socketcan/socketcanbackend.cpp | |
parent | f13aaee1c81f46176b50642e9889f09220a8249a (diff) |
QCanBusFrame: do not initialize payload byte by byte
The data is in a buffer that can directly be put into a QByteArray
constructor, using a more efficient copy.
Change-Id: I0b96969a881aa76e773eec73009e9511eea935ef
Reviewed-by: Alex Blasche <alexander.blasche@theqtcompany.com>
Diffstat (limited to 'src/plugins/canbus/socketcan/socketcanbackend.cpp')
-rw-r--r-- | src/plugins/canbus/socketcan/socketcanbackend.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/plugins/canbus/socketcan/socketcanbackend.cpp b/src/plugins/canbus/socketcan/socketcanbackend.cpp index 39ca2c9..4c142d0 100644 --- a/src/plugins/canbus/socketcan/socketcanbackend.cpp +++ b/src/plugins/canbus/socketcan/socketcanbackend.cpp @@ -544,9 +544,7 @@ void SocketCanBackend::readSocket() bufferedFrame.setFrameId(frame.can_id & CAN_EFF_MASK); - QByteArray load; - for (int i = 0; i < frame.len ; i++) - load.insert(i, frame.data[i]); + QByteArray load(reinterpret_cast<char *>(frame.data), frame.len); bufferedFrame.setPayload(load); newFrames.append(bufferedFrame); |