trivial code refactor
Signed-off-by: fufesou <shuanglongchen@yeah.net>
This commit is contained in:
parent
3d768098f8
commit
ee4aa0d0ac
@ -49,8 +49,14 @@ impl super::service::Reset for StatePos {
|
|||||||
}
|
}
|
||||||
|
|
||||||
impl StatePos {
|
impl StatePos {
|
||||||
|
#[inline]
|
||||||
fn is_valid(&self) -> bool {
|
fn is_valid(&self) -> bool {
|
||||||
self.cursor_pos.0 != INVALID_CURSOR_POS && self.cursor_pos.1 != INVALID_CURSOR_POS
|
self.cursor_pos.0 != INVALID_CURSOR_POS
|
||||||
|
}
|
||||||
|
|
||||||
|
#[inline]
|
||||||
|
fn is_moved(&self, x: i32, y: i32) -> bool {
|
||||||
|
self.is_valid() && (self.cursor_pos.0 != x || self.cursor_pos.1 != y)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -129,8 +135,7 @@ fn update_last_cursor_pos(x: i32, y: i32) {
|
|||||||
fn run_pos(sp: GenericService, state: &mut StatePos) -> ResultType<()> {
|
fn run_pos(sp: GenericService, state: &mut StatePos) -> ResultType<()> {
|
||||||
if let Some((x, y)) = crate::get_cursor_pos() {
|
if let Some((x, y)) = crate::get_cursor_pos() {
|
||||||
update_last_cursor_pos(x, y);
|
update_last_cursor_pos(x, y);
|
||||||
if state.is_valid() {
|
if state.is_moved(x, y) {
|
||||||
if state.cursor_pos.0 != x || state.cursor_pos.1 != y {
|
|
||||||
let mut msg_out = Message::new();
|
let mut msg_out = Message::new();
|
||||||
msg_out.set_cursor_position(CursorPosition {
|
msg_out.set_cursor_position(CursorPosition {
|
||||||
x,
|
x,
|
||||||
@ -148,7 +153,6 @@ fn run_pos(sp: GenericService, state: &mut StatePos) -> ResultType<()> {
|
|||||||
};
|
};
|
||||||
sp.send_without(msg_out, exclude);
|
sp.send_without(msg_out, exclude);
|
||||||
}
|
}
|
||||||
}
|
|
||||||
state.cursor_pos = (x, y);
|
state.cursor_pos = (x, y);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user