Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

[s] - #7292

Merged
alexkar598 merged 2 commits into
masterfrom
alexkar598-patch-1
Dec 11, 2019
Merged

[s]#7292
alexkar598 merged 2 commits into
masterfrom
alexkar598-patch-1

Conversation

@alexkar598

Copy link
Copy Markdown
Member

E

@monster860

Copy link
Copy Markdown
Member

what does this even fix

@yacabo

yacabo commented Dec 10, 2019

Copy link
Copy Markdown
Contributor

10/10 documentation

@monster860 monster860 changed the title [s] a pr that I have literally no idea what it fixes Dec 10, 2019

@monster860 monster860 left a comment

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.

what does this fix

return

/turf/open/floor/engine/remove_tile(mob/user, silent = FALSE, make_tile = TRUE)
return

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
return
.=..()
ScrapeAway(flags = CHANGETURF_INHERIT_AIR)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No that makes it be stupid and counter the point of reinforced floors being nearly indestructible by any means

@alexkar598 alexkar598 changed the title a pr that I have literally no idea what it fixes [s] Dec 10, 2019
@yogstation13 yogstation13 deleted a comment Dec 10, 2019
@yogstation13 yogstation13 locked and limited conversation to collaborators Dec 10, 2019
@alexkar598

Copy link
Copy Markdown
Member Author

Let's not talk about unpatched exploits in public, ya?

@alexkar598
alexkar598 merged commit 8befc85 into master Dec 11, 2019
@Amelia0010
Amelia0010 deleted the alexkar598-patch-1 branch December 12, 2019 08:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants