Merge pull request #325 from chrisvest/faster-with_color-ncurses

Use a single hashmap lookup instead of two for resolving ncurses colours.
This commit is contained in:
Alexandre Bury 2019-03-12 09:36:02 -07:00 committed by GitHub
commit af566dd57f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -153,12 +153,13 @@ impl Backend {
let mut pairs = self.pairs.borrow_mut();
// Find if we have this color in stock
let (front, back) = find_closest_pair(pair);
if pairs.contains_key(&(front, back)) {
let result = find_closest_pair(pair);
let lookup = pairs.get(&result);
if lookup.is_some() {
// We got it!
pairs[&(front, back)]
*lookup.unwrap()
} else {
self.insert_color(&mut *pairs, (front, back))
self.insert_color(&mut *pairs, result)
}
}