Skip to content

Rename RemovedComponents::iter #9755

Closed
@JoJoJet

Description

What problem does this solve or what need does it fill?

The existence of a method named .iter() that modifies state is confusing, as methods with that name are usually immutable.

This is the same motivation as #9624 -- this change probably should have been done at the same time as the EventReader::iter rename.

What solution would you like?

Rename the function to .read(), which more accurately communicates that the operation is stateful.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    A-ECSEntities, components, systems, and eventsC-UsabilityA targeted quality-of-life change that makes Bevy easier to useD-TrivialNice and easy! A great choice to get started with Bevy

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions