Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow gyrotron controller to be buffered #16193

Merged
merged 2 commits into from Oct 8, 2017

Conversation

DamianX
Copy link
Contributor

@DamianX DamianX commented Oct 7, 2017

Fixes .1 of #16190

馃啈

  • rscadd: The gyrotron control computer can be buffered in multitools.

@@ -72,6 +72,16 @@
</ul>
"}

/obj/machinery/rust/gyrotron/canLink(var/obj/machinery/computer/rust_gyrotron_controller/object, var/link/context)
. = (istype(object) && get_dist(src, object) < RUST_GYROTRON_RANGE)
Copy link
Contributor

Choose a reason for hiding this comment

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

Use return, it's much more readable.

Copy link
Contributor

Choose a reason for hiding this comment

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

Once this is fixed I'll merge. Tiny thing but it makes a difference.

/obj/machinery/computer/rust_gyrotron_controller
name = "Gyrotron Remote Controller"
icon_state = "engine"
circuit = /obj/item/weapon/circuitboard/rust_gyrotron_control
light_color = LIGHT_COLOR_BLUE

var/id_tag // Required for multitool buffering
Copy link
Contributor

Choose a reason for hiding this comment

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

Uh, is it actually? If it is then you should fix that instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed, I don't really want to do it right now, though.

@@ -0,0 +1 @@
#undef RUST_GYROTRON_RANGE
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't bother honestly.

@Shadowmech88 Shadowmech88 added the Bug / Fix This is either a bug or a fix for a bug. label Oct 7, 2017
@N3X15 N3X15 added Needs Fixing 鉂わ笍 Quality of Life 鉂わ笍 This is a change that removes hassle and improves quality of life. labels Oct 8, 2017
@N3X15 N3X15 removed the Needs Fixing label Oct 8, 2017
@N3X15 N3X15 merged commit 47df97e into vgstation-coders:Bleeding-Edge Oct 8, 2017
@DamianX DamianX deleted the GyrotronLinker branch March 2, 2018 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug / Fix This is either a bug or a fix for a bug. 鉂わ笍 Quality of Life 鉂わ笍 This is a change that removes hassle and improves quality of life.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants