-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ducktap fix #155
Ducktap fix #155
Conversation
Is there any reason why you moved the code down? It's very hard to see what the actual code differences are this way. Could you move it back where it was so the difference is readable please? |
I'm using in_duck.state check in function which is undeclared if i don't move function down |
cl_dll/input.cpp
Outdated
static bool duck_was_pressed = false; | ||
static bool should_release_duck; | ||
|
||
if (in_duck.state & 1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
static bool duck_was_pressed = false; | |
static bool should_release_duck; | |
if (in_duck.state & 1) | |
static bool should_release_duck; | |
bool duck_was_pressed = false; | |
if (in_duck.state & 1) |
Doesn't need to be a static if it's set every time.
Also please rename to duck_is_pressed
and remove the set in the should_release_duck
condition below as it does nothing.
Thanks! |
* ducktap fix * remove cl_duck_priority * some changes * remove unnecessary set
* ducktap fix * remove cl_duck_priority * some changes * remove unnecessary set
Fix +ducktap that you weren't able to interfere in "ducking" process, now its working like +bxt_tas_ducktap