matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

Merge pull request #1207 from matthiasbeyer/libimagstore-retrievecopy-rename

Browse source 
Rename `Store::retrieve_copy` to `Store::get_copy`
This commit is contained in:
Matthias Beyer 2018-01-08 23:11:41 +01:00 committed by GitHub
commit ab78805312
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
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()