Fix test stream and remove warnings (!137) · Merge requests · LidarView / LidarView-Core · GitLab

Snippets Groups Projects

System updates will be applied on Mar 14th between 7am and 9am, EST (UTC-05:00). This site will have intermittent downtime during that time.

Merged MelanieCarriere requested to merge FixTestStreamAndRemoveWarnings into master 4 years ago

All threads resolved!

Activity

MelanieCarriere @melanie.carriere · 4 years ago

Author Contributor

@nick.laurenson can you have a look please ?
MelanieCarriere @melanie.carriere · 4 years ago

Author Contributor

@nicolas.cadart @jerome.dias can one of you have a look please ?
Nicolas Cadart @nicolas.cadart · 4 years ago

Contributor

I'm not sure of all the behavior modification this could change, but it looks ok.

And for the warning removal !

Edited 4 years ago by Nicolas Cadart
Nicolas Cadart approved this merge request 4 years ago

approved this merge request
Nick Laurenson @nick.laurenson started a thread on the diff 4 years ago

Resolved 4 years ago by MelanieCarriere
Last reply by Nick Laurenson 4 years ago

MelanieCarriere added 1 commit 4 years ago

added 1 commit

de7657a8 - [fix] Avoid segFault in getLastFrameAvailable

Compare with previous version

MelanieCarriere added 9 commits 4 years ago

added 9 commits

6e9748b7...0721fc27 - 7 commits from branch master
c4f835b8 - [test] Fix the stream test
600a88ff - [test] remove warnings

Compare with previous version

MelanieCarriere added 1 commit 4 years ago

added 1 commit

e4e0a0c9 - [test] check that the Stream and its interpreter are not nullptr

Compare with previous version

MelanieCarriere resolved all threads 4 years ago

resolved all threads

MelanieCarriere mentioned in commit 4 years ago

mentioned in commit 30b0d327

MelanieCarriere merged 4 years ago

merged

Please register or sign in to reply