@rrichardson While reviewing #101, I realized that we can remove the use rkv and use tempfile declarations in examples/, as they're redundant with later declarations that import specific symbols from those crates.
Also, the use self::rkv::{ … } declaration happens to work because of the preceding use rkv declaration, but self:: is intended to reference local modules/items, and rkv is actually an external crate in this case (from the perspective of the example, which demonstrates how an external consumer would use rkv).
So this followup branch removes the use rkv and use tempfile declarations and changes use self::rkv::{ … } to use rkv::{ … }.
rrichardson
commented
5 years ago
@rrichardson While reviewing #101, I realized that we can remove the
use rkvanduse tempfiledeclarations in examples/, as they're redundant with later declarations that import specific symbols from those crates.Also, the
use self::rkv::{ … }declaration happens to work because of the precedinguse rkvdeclaration, butself::is intended to reference local modules/items, and rkv is actually an external crate in this case (from the perspective of the example, which demonstrates how an external consumer would use rkv).So this followup branch removes the
use rkvanduse tempfiledeclarations and changesuse self::rkv::{ … }touse rkv::{ … }.Does this seem reasonable?