Skip to content

Commit

Permalink
Resolves issue #3487 -- Throwing bug fix (#3522)
Browse files Browse the repository at this point in the history
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request

This fix removes a second increment to the dist_travelled variable. This
line appears to have caused the throwing range of items to be halved,
effectively. This fix should make it so that thrown objects go as far as
they are intended to.

## Why It's Good For The Game

Resolves the issue linked here:
shiptest-ss13/Shiptest#3487

## Changelog

:cl:
fix: fixed halved throw range
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->
  • Loading branch information
zimon9 authored and Constellado committed Nov 19, 2024
1 parent 3d441b7 commit 3212f73
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions code/controllers/subsystem/throwing.dm
Original file line number Diff line number Diff line change
Expand Up @@ -160,8 +160,6 @@ SUBSYSTEM_DEF(throwing)
finalize()
return

dist_travelled++

if(actual_target && !(actual_target.pass_flags_self & LETPASSTHROW) && actual_target.loc == AM.loc) // we crossed a movable with no density (e.g. a mouse or APC) we intend to hit anyway.
finalize(TRUE, actual_target)
return
Expand Down

0 comments on commit 3212f73

Please sign in to comment.