Add to string conversions for the CapabilityInterface class

osrf / capabilities

Implements the concept of capabilities as part of the robots-in-concert system.

Other

8 stars 26 forks source link

Add to string conversions for the CapabilityInterface class #27

Closed bit-pirate closed 10 years ago

bit-pirate commented 10 years ago

and maybe for the others as well (provider, semantic).

This would make it easier to debug code using those classes.

wjwwood commented 10 years ago

I upgraded this to a pull request.

@bit-pirate can you look at the format of the output and let me know if that works for you?

coveralls commented 10 years ago

Coverage remained the same when pulling 05f9ebfcd102c34ba5f8aded2b545d1f2d616d3a on issue_27 into 69ed148772cc9148e65818b7a47b75b3e93b9063 on master.

bit-pirate commented 10 years ago

Looks good so far. Could you extend this to also print the interfaces (Interface objects) residing inside CapabilityInterface?

wjwwood commented 10 years ago

I updated it, have a look now.

coveralls commented 10 years ago

Coverage remained the same when pulling f5ed71552e6532b6cdd78d86988471d3415cc6ab on issue_27 into 69ed148772cc9148e65818b7a47b75b3e93b9063 on master.

bit-pirate commented 10 years ago

Nice. Small remark: The output does not differentiate required and provided topics.

wjwwood commented 10 years ago

Oh yeah, I can do that too.

wjwwood commented 10 years ago

Done, third times the charm?

coveralls commented 10 years ago

Coverage remained the same when pulling 9db968f26d646e33dd9c0aa00595ce59ebe085bb on issue_27 into efde540589ed6079a09b54e80d677c8347809487 on master.

wjwwood commented 10 years ago

Fixed a bug and getting back to 100% coverage.

coveralls commented 10 years ago

Coverage remained the same when pulling 2a5d6ab06e300ce6d13fc02f5413e28054eb4843 on issue_27 into efde540589ed6079a09b54e80d677c8347809487 on master.

wjwwood commented 10 years ago

I'm going to merge this, please comment here if it doesn't do what you want.