fn lookup_cache(&self, username: &GerritUsername) -> Synchronization<ProfileId> {
        match self.cache.get(username) {
            Some(status) => status.clone(),
            // No entry found most likely means we haven't tried yet.
            None => Synchronization::NotMappedYet,
        }
    }
/**
     * Returning Synchronization here, is just easier than
     * `Option<Option<ProfileId>>`, after all, we need to make sure to understand
     * the difference between a missing entry and an entry that can't be mapped
     * (e.g. multiple results when asking API).
     */
Returning Synchronization here, is just easier than
`Option>`, after all, we need to make sure to understand
the difference between a missing entry and an entry that can't be mapped
.
Returning Synchronization here is just easier than ` Option > ` after all we need to make sure to understand the difference between a missing entry and an entry that can ' t be mapped .
fn lookup_cache(&self, username: &GerritUsername) -> Synchronization<ProfileId> {
        match self.cache.get(username) {
            Some(status) => status.clone(),
            // No entry found most likely means we haven't tried yet.
            None => Synchronization::NotMappedYet,
        }
    }
fn lookup_cache ( & self , username : & GerritUsername ) -> Synchronization < ProfileId > { match self . cache . get ( username ) { Some ( status ) => status . clone ( ) , // No entry found most likely means we haven't tried yet. None => Synchronization :: NotMappedYet , } }
Returning Synchronization here, is just easier than
 `Option<Option<ProfileId>>`, after all, we need to make sure to understand
 the difference between a missing entry and an entry that can't be mapped
 (e.g.
Returning Synchronization here is just easier than ` Option<Option<ProfileId >> ` after all we need to make sure to understand the difference between a missing entry and an entry that can ' t be mapped ( e . g .
&self
username
&GerritUsername
&self
username
&GerritUsername