Merge lp:~verifypn-stub/verifypn/encoder-fix into lp:verifypn

Proposed by Peter Gjøl Jensen
Status: Merged
Approved by: Jiri Srba
Approved revision: 195
Merged at revision: 195
Proposed branch: lp:~verifypn-stub/verifypn/encoder-fix
Merge into: lp:verifypn
Diff against target: 12 lines (+1/-1)
1 file modified
PetriEngine/Structures/AlignedEncoder.cpp (+1/-1)
To merge this branch: bzr merge lp:~verifypn-stub/verifypn/encoder-fix
Reviewer Review Type Date Requested Status
Jiri Srba Approve
Review via email: mp+336115@code.launchpad.net

Description of the change

Fixes bug in encoder due to typo in "memcpy".

To post a comment you must log in.
Revision history for this message
Jiri Srba (srba) wrote :

I hope it will stay in trunk for ever now :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'PetriEngine/Structures/AlignedEncoder.cpp'
2--- PetriEngine/Structures/AlignedEncoder.cpp 2017-11-26 13:43:04 +0000
3+++ PetriEngine/Structures/AlignedEncoder.cpp 2018-01-15 15:07:56 +0000
4@@ -96,7 +96,7 @@
5 {
6 if(sizeof(T) == sizeof(uint32_t))
7 {
8- memcmp(&(_scratchpad.raw()[offset]), data, _places*sizeof(T));
9+ memcpy(&(_scratchpad.raw()[offset]), data, _places*sizeof(T));
10 }
11 else
12 {

Subscribers

People subscribed via source and target branches