search

ros / roscpp_core

ros distribution sandbox
88 stars 116 forks source link

fix the fix that was supposed to fix #48. also added a unit test. #50

Closed pgao closed 8 years ago

pgao commented 8 years ago

I made a unit test to test issue #48 and found that it failed. So I made a fix to the fix in #49 that was supposed to fix the issue that wasn't fixed, but now the test is fixed and the fix is fixed and I think the issue is fixed by the fix to the fix and yeah.

@televator

televator commented 8 years ago

Thanks, no idea how that got in there.

pgao commented 8 years ago

@dirk-thomas I've moved the C++11 into the test target. Let me know if there's anything else I can fix for the fix to the fix to the fix.

pgao commented 8 years ago

@dirk-thomas look right?

dirk-thomas commented 8 years ago

Thank you for providing the test and fixing the problem.