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

Declutter IAddRemoveComponent to use explicit Link and Relate functions rather than Overloads #35

Open
thygrrr opened this issue Nov 6, 2024 · 0 comments
Open

Declutter IAddRemoveComponent to use explicit Link and Relate functions rather than Overloads #35

thygrrr opened this issue Nov 6, 2024 · 0 comments
Labels
feature New Functionality and Changes
Milestone
0.6.0-beta

Comments

@thygrrr
Copy link
Collaborator

thygrrr commented Nov 6, 2024 •
edited
Loading

The overload space in Add and Remove is getting very cramped, obscuring a core feature of ECS by letting two more fringe features get in the way.

Disadvantages

  • hard to pick the right overload
  • picking the wrong overload can cause issues such as creating an Object Link when trying to create a normal component, etc.
  • code is difficult to read

Impact

Breaking Change, but [Obsolete] attributes will help migrating code.

Requires extensive Rework of Documentation and samples!
@thygrrr thygrrr converted this from a draft issue Nov 6, 2024
@thygrrr thygrrr added the feature New Functionality and Changes label Nov 6, 2024
@thygrrr thygrrr moved this to Ready in fennecs ⚡🦊 Nov 6, 2024
@thygrrr thygrrr added this to the 0.6.0 milestone Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New Functionality and Changes
Projects
fennecs ⚡🦊
Status: Ready
Milestone
0.6.0-beta
Development

No branches or pull requests
1 participant
@thygrrr