ebassi / taglib-rust

TagLib bindings for Rust
MIT License
27 stars 11 forks source link

Use optional types for missing fields #8

Closed kalouantonis closed 6 years ago

kalouantonis commented 6 years ago

Instead of returning an empty string, None is returned instead.

This is more inline with what is considered idiomatic rust.

cdown commented 6 years ago

Thanks! There are now tests, so I modified the tests to conform and merged as 1e7f279059087822b38ec5a6a577cac67ecaa038.