search

ros / rosconsole

17 stars 61 forks source link

Fix double index increment in rosconsole_glog #15

Closed NikolausDemmel closed 5 years ago

NikolausDemmel commented 6 years ago

The double index increment looks like a typo. Clang found this one.

dirk-thomas commented 6 years ago

Can you describe how you tested the behavior before and after the patch.

NikolausDemmel commented 6 years ago

Can you describe how you tested the behavior before and after the patch.

Not at all. I'm not even using that code. It got included in a project by accident and the compiler gave an error for the double increment. Briefly looking at the code confirmed for me that this is likely unintentional. Therefore I filed this "bug report".