Skip to content

Fix wrong calculation of weapon Angle in demo player#369

Merged
H-M-H merged 4 commits into
ddnet:masterfrom
user578:master
Nov 15, 2015
Merged

Fix wrong calculation of weapon Angle in demo player#369
H-M-H merged 4 commits into
ddnet:masterfrom
user578:master

Conversation

@user578

@user578 user578 commented Nov 13, 2015

Copy link
Copy Markdown

Fix wrong calculation of weapon Angle in demo player, more here: http://forum.ddnet.tw/viewtopic.php?f=45&t=2717
and
Add *.dll to .gitignore (for Windows)

@ghost

ghost commented Nov 14, 2015

Copy link
Copy Markdown

Where does these values (800 and 1600) come from? 256.0f*pi ?

@user578

user578 commented Nov 14, 2015

Copy link
Copy Markdown
Author

Yes. Sorry, I had to use pi, my bad.
It should be 256.0f*pi and 256.0f*pi*2 instead of 800 and 1600.

@H-M-H

H-M-H commented Nov 14, 2015

Copy link
Copy Markdown
Member

Well I tested it and it seems to work, maybe you could shortly explain what was wrong.

@H-M-H

H-M-H commented Nov 15, 2015

Copy link
Copy Markdown
Member

Looks fine now.

H-M-H added a commit that referenced this pull request Nov 15, 2015
Fix wrong calculation of weapon Angle in demo player
@H-M-H H-M-H merged commit 940e46a into ddnet:master Nov 15, 2015

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why you're doing it down here instead of near line 409.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well like this it is just executed less often (this correction is not required if you are the local character). In all other cases it is applied.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By that logic, you should have moved the assignemnt part of line 409 down here as well.

heinrich5991 added a commit to heinrich5991/ddnet that referenced this pull request Nov 15, 2015
H-M-H added a commit that referenced this pull request Nov 15, 2015
SollyBunny pushed a commit to SollyBunny/ddnet that referenced this pull request Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants