matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Rename Store::retrieve_copy to Store::get_copy

Browse source 
The function was named falsely and nobody noticed. So nobody uses this
function yet - hopefully - as this change is a breaking API change.
This commit is contained in:
Matthias Beyer 2018-01-07 20:17:42 +01:00
parent 5e8feca6bb
commit 689310edfe
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
doc/src/09020-changelog.md
View file

@ -26,6 +26,9 @@ This section contains the changelog from the last release to the next release.
* `imag-log` was introduced
* `imag-init` was introduced
* `libimagdiary` supports second-granularity now.
* `libimagstore::store::Store::retrieve_copy` was renamed to
`libimagstore::store::Store::get_copy`, which describes the semantics of
the function way better.
* Minor changes
* Internals were refactored from `match`ing all the things into function
chaining

5
lib/core/libimagstore/src/store.rs
View file

@ -502,8 +502,7 @@ impl Store {
Ok(())
}
/// Retrieve a copy of a given entry, this cannot be used to mutate
/// the one on disk
/// Get a copy of a given entry, this cannot be used to mutate the one on disk
///
/// # Return value
///
@ -514,7 +513,7 @@ impl Store {
/// - RetrieveCopyCallError(IdLocked()) if the Entry is borrowed currently
/// - Errors StoreEntry::new() might return
///
pub fn retrieve_copy<S: IntoStoreId>(&self, id: S) -> Result<Entry> {
pub fn get_copy<S: IntoStoreId>(&self, id: S) -> Result<Entry> {
let id = id.into_storeid()?.with_base(self.path().clone());
debug!("Retrieving copy of '{}'", id);
let entries = self.entries.write()