Skip to content

Simplify Stream API #18141

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

Open
wants to merge 1 commit into
base: Pharo14
Choose a base branch
from

Conversation

jecisc
Copy link
Member

@jecisc jecisc commented May 6, 2025

In stream API we have some methods calling methods of the same name without the #basic prefix. Those methods are never overriden.

They seems useless currently and probably comes from past constraints. I propose to deprecate them to simplify Stream API

In stream API we have some methods calling methods of the same name without the #basic prefix. Those methods are never overriden. 

They seems useless currently and probably comes from past constraints. I propose to deprecate them to simplify Stream API
@Ducasse
Copy link
Member

Ducasse commented May 6, 2025

Normally basic* should never be overridden so this is normal that they are not.
So that the subclass can always access the default basic behavior.
Now so I see why we have basic for such methods but may be nobody use them.

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

Successfully merging this pull request may close these issues.

2 participants