amqp: avoid panic!() and expect() in library

Antti / rust-amqp

AMQP client in pure rust. Corresponds to rabbitmq spec.

MIT License

249 stars 45 forks source link

amqp: avoid panic!() and expect() in library #14

Closed lucab closed 9 years ago

lucab commented 9 years ago

For improved safety, this commit replaces all panic!() and expect() in the library with graceful error logging and bubbling-up, in order to avoid crashing applications on malformed input.

Signed-off-by: Luca Bruno lucab@debian.org

Antti commented 9 years ago

@lucab This is really useful, thank you!

lucab commented 9 years ago

Rebased to fix the style issues and changed commit message. unwrap() usage will be covered separately.

Antti commented 9 years ago

OK, let's merge what we got now, and continue improving afterwards.