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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation: Use place_meeting() Examples on the Collisions Page Instead of !place_meeting() Which Should Get Updated to place_empty() Anyway #9172

Closed
ParodyKnaveBob opened this issue Jan 27, 2025 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation are required by this issue

Comments

@ParodyKnaveBob
Copy link

ParodyKnaveBob commented Jan 27, 2025

Is your feature request related to a problem?

I have a personal peeve seeing GM users touting !place_meeting() everywhere when place_empty() will suffice. I want to rant, but I won't.

I just now learned that GM's own manual shows this unnecessary legacy practice on the Collisions page. It says, "The simplest way to check for collisions against objects is to use place_meeting," and then for every following code example uses !place_meeting() to check where there is not a collision.

Describe the solution you'd like

I suggest either stating that the simplest way to check for collisions against *cough* instances is to use place_empty() which seems to be what you truly mean here or give examples where you use place_meeting() without all the nots.

I'd just really, really love to see people using place_empty() the way it's been designed to be used for many years now. Tutorials, long-time GMCers, and (now I find) even the manual says, "use not-function despite perfectly good function."

@ParodyKnaveBob ParodyKnaveBob added the feature request New feature (or a request for one) label Jan 27, 2025
@ParodyKnaveBob ParodyKnaveBob changed the title Documentation: Use place_meeting() Examples on the Collisions Page Instead of !place_meeting() Which Should Get Updated to place_empty() Documentation: Use place_meeting() Examples on the Collisions Page Instead of !place_meeting() Which Should Get Updated to place_empty() Anyway Jan 27, 2025
@stuckie stuckie moved this from Triage to Backlog in Team Workload Jan 28, 2025
@stuckie stuckie added documentation Improvements or additions to documentation are required by this issue and removed feature request New feature (or a request for one) labels Jan 28, 2025
@gurpreetsinghmatharoo
Copy link

It's the same anyway, and introducing new coders to !() conditions is pretty cool.

Closing.

@gurpreetsinghmatharoo gurpreetsinghmatharoo closed this as not planned Won't fix, can't repro, duplicate, stale Mar 6, 2025
@gurpreetsinghmatharoo gurpreetsinghmatharoo moved this from Done to Not Planned in Team Workload Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation are required by this issue
Projects
Status: Not Planned
Development

No branches or pull requests

3 participants