Fix movement inputs when holding shift

aidan9382 commented

2024-10-14 19:23:43 +00:00

Member

minor fix but its very annoying since it also breaks releasing keys

aidan9382 added 1 commit 2024-10-14 19:23:44 +00:00

Fix movement inputs when holding shift b0aecc3fd1

Quaternions commented

2024-10-17 03:36:33 +00:00

Owner

This code allocates a new string, which the goal of .as_str() was to avoid that. Please don't use master branch for pull requests, as I can't add any commits!

Quaternions commented

2024-10-17 03:42:52 +00:00

Owner

Here is an alternative option

						if let Some(input_instruction)=match keycode{
							winit::keyboard::Key::Named(winit::keyboard::NamedKey::Space)=>Some(InputInstruction::Jump(s)),
							winit::keyboard::Key::Character(key)=>match key.as_str(){
								"W"|"w"=>Some(InputInstruction::MoveForward(s)),
								"A"|"a"=>Some(InputInstruction::MoveLeft(s)),
								"S"|"s"=>Some(InputInstruction::MoveBack(s)),
								"D"|"d"=>Some(InputInstruction::MoveRight(s)),
								"E"|"e"=>Some(InputInstruction::MoveUp(s)),
								"Q"|"q"=>Some(InputInstruction::MoveDown(s)),
								"Z"|"z"=>Some(InputInstruction::Zoom(s)),
								"R"|"r"=>if s{
									//mouse needs to be reset since the position is absolute
									self.mouse=strafesnet_common::mouse::MouseState::default();
									Some(InputInstruction::ResetAndRestart)
								}else{None},
								"F"|"f"=>if s{Some(InputInstruction::PracticeFly)}else{None},
								_=>None,
							},
							_=>None,
						}

Here is an alternative option ```rust if let Some(input_instruction)=match keycode{ winit::keyboard::Key::Named(winit::keyboard::NamedKey::Space)=>Some(InputInstruction::Jump(s)), winit::keyboard::Key::Character(key)=>match key.as_str(){ "W"|"w"=>Some(InputInstruction::MoveForward(s)), "A"|"a"=>Some(InputInstruction::MoveLeft(s)), "S"|"s"=>Some(InputInstruction::MoveBack(s)), "D"|"d"=>Some(InputInstruction::MoveRight(s)), "E"|"e"=>Some(InputInstruction::MoveUp(s)), "Q"|"q"=>Some(InputInstruction::MoveDown(s)), "Z"|"z"=>Some(InputInstruction::Zoom(s)), "R"|"r"=>if s{ //mouse needs to be reset since the position is absolute self.mouse=strafesnet_common::mouse::MouseState::default(); Some(InputInstruction::ResetAndRestart) }else{None}, "F"|"f"=>if s{Some(InputInstruction::PracticeFly)}else{None}, _=>None, }, _=>None, } ```

aidan9382 force-pushed master from b0aecc3fd1 to fcacf7026f

2024-10-18 19:26:59 +00:00

Compare

aidan9382 commented

2024-10-18 19:29:23 +00:00

Author

Member

I've gone ahead and implemented your suggestion if the allocation is an issue
I also tried moving it off master in my fork, but the PR doesn't allow changing which branch I'm pulling from for some reason, only which one im pushing to, so for now it's gonna have to stay on master

I've gone ahead and implemented your suggestion if the allocation is an issue I also tried moving it off master in my fork, but the PR doesn't allow changing which branch I'm pulling *from* for some reason, only which one im pushing *to*, so for now it's gonna have to stay on master