Skip to content

a few bugfixes#2

Open
flachsenberg wants to merge 2 commits intoantonbarty:masterfrom
flachsenberg:master
Open

a few bugfixes#2
flachsenberg wants to merge 2 commits intoantonbarty:masterfrom
flachsenberg:master

Conversation

@flachsenberg
Copy link
Copy Markdown

@flachsenberg flachsenberg commented Sep 14, 2017

  • fixed missing includes
  • fixed RAW/Calibrated detection
  • fixed a wrong datatype
    • sizeof(int16) is 2 bytes
    • casted to float (4 bytes)
      => nasty behaviour (memory corruption)
  • the way the max pulse/train ID are determined
    the corresponding loops should be inclusive

Florian Flachsenberg added 2 commits September 14, 2017 21:48
  - fixed missing includes
  - fixed RAW/Calibrated detection
  - fixed a wrong datatype
    - sizeof(int16) is 2 bytes
    - casted to float (4 bytes)
      => nasty behaviour (memory corruption)
  - the way the max pulse/train ID are determined
    the corresponding loops should be inclusive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant