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.

Sign up for GitHub

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

Jump to bottom

fix: Remove mustCallSuper from onComponentTypeCheck #2561

Merged
merged 3 commits into from
Jun 2, 2023
Merged

fix: Remove mustCallSuper from onComponentTypeCheck #2561

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
762fac3
fix: onComponentTypeCheck really MUST call super only for hitboxes
ASGAlex May 31, 2023
1188b79
doc: detailed description about super.onComponentTypeCheck usage
ASGAlex Jun 2, 2023
5c4491e
Apply suggestions from code review
spydon Jun 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions packages/flame/lib/src/collisions/collision_callbacks.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ mixin GenericCollisionCallbacks<T> {
/// reimplement [onComponentTypeCheck]. The result of calculation is cached
/// so you should not check any dynamical parameters here, the function
/// intended to be used as pure type checker.
@mustCallSuper
bool onComponentTypeCheck(PositionComponent other);

/// Assign your own [CollisionCallback] if you want a callback when this
Expand Down Expand Up @@ -136,7 +135,6 @@ mixin CollisionCallbacks on Component
}

@override
@mustCallSuper
bool onComponentTypeCheck(PositionComponent other) {
final myParent = parent;
final otherParent = other.parent;
Expand Down