aboutsummaryrefslogtreecommitdiffstats
path: root/src/common/input
diff options
context:
space:
mode:
authorYutaka Kamei <kamei@yykamei.me>2021-04-28 13:40:04 +0900
committerYutaka Kamei <kamei@yykamei.me>2021-04-28 14:14:27 +0900
commit82c5f505f123a3a1cc92545d39f7bd1dc8e7bea1 (patch)
tree4910359895f7099555456c14110eba61d37c67dc /src/common/input
parentMerge pull request #391 from a-kenji/config-loading (diff)
downloadsl0ppij-82c5f505f123a3a1cc92545d39f7bd1dc8e7bea1.tar.gz
sl0ppij-82c5f505f123a3a1cc92545d39f7bd1dc8e7bea1.tar.xz
sl0ppij-82c5f505f123a3a1cc92545d39f7bd1dc8e7bea1.zip
refactor(handler): refer to self.config.keybinds internally
Because `handle_key` can refer to `self.config.keybinds` by itself, it doesn't have to receive the third argument. Also, this change may slightly improve the performance as it deletes `clone()`.
Diffstat (limited to 'src/common/input')
-rw-r--r--src/common/input/handler.rs10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/common/input/handler.rs b/src/common/input/handler.rs
index 66fb6271..197f8315 100644
--- a/src/common/input/handler.rs
+++ b/src/common/input/handler.rs
@@ -58,7 +58,6 @@ impl InputHandler {
fn handle_input(&mut self) {
let mut err_ctx = OPENCALLS.with(|ctx| *ctx.borrow());
err_ctx.add_call(ContextType::StdinHandler);
- let keybinds = self.config.keybinds.clone();
let alt_left_bracket = vec![27, 91];
loop {
if self.should_exit {
@@ -70,14 +69,14 @@ impl InputHandler {
Ok((event, raw_bytes)) => match event {
termion::event::Event::Key(key) => {
let key = cast_termion_key(key);
- self.handle_key(&key, raw_bytes, &keybinds);
+ self.handle_key(&key, raw_bytes);
}
termion::event::Event::Unsupported(unsupported_key) => {
// we have to do this because of a bug in termion
// this should be a key event and not an unsupported event
if unsupported_key == alt_left_bracket {
let key = Key::Alt('[');
- self.handle_key(&key, raw_bytes, &keybinds);
+ self.handle_key(&key, raw_bytes);
}
}
termion::event::Event::Mouse(_) => {
@@ -90,8 +89,9 @@ impl InputHandler {
}
}
}
- fn handle_key(&mut self, key: &Key, raw_bytes: Vec<u8>, keybinds: &Keybinds) {
- for action in Keybinds::key_to_actions(&key, raw_bytes, &self.mode, &keybinds) {
+ fn handle_key(&mut self, key: &Key, raw_bytes: Vec<u8>) {
+ let keybinds = &self.config.keybinds;
+ for action in Keybinds::key_to_actions(&key, raw_bytes, &self.mode, keybinds) {
let should_exit = self.dispatch_action(action);
if should_exit {
self.should_exit = true;