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

Cleanup z_collision_check 2: Rename fields pertaining to "other" colliders #1436

Closed
wants to merge 1 commit into from

Conversation

Dragorn421
Copy link
Collaborator

Split from #1427

Collider.actor -> Collider.self
Collider.ac -> Collider.otherAT
Collider.at -> Collider.otherAC
Collider.oc -> Collider.otherOC

ColliderInfo.atHit -> ColliderInfo.otherColAC
ColliderInfo.acHit -> ColliderInfo.otherColAT
ColliderInfo.atHitInfo -> ColliderInfo.otherElemAC
ColliderInfo.acHitInfo -> ColliderInfo.otherElemAT
(note: ColliderInfo -> ColliderElement in #1427)

Relevant comment thread from #1427:
#1427 (comment)

@Dragorn421
Copy link
Collaborator Author

Closing because I'm not sure about the names and I may as well restart from scratch anyway with all the conflicts

@Dragorn421 Dragorn421 closed this Jan 28, 2024
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.

2 participants