Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Check trivial trait args [ci: last-only]#11034

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
som-snytt wants to merge3 commits intoscala:2.13.x
base:2.13.x
Choose a base branch
Loading
fromsom-snytt:issue/6805-trait-parameters

Conversation

som-snytt
Copy link
Contributor

@scala-jenkinsscala-jenkins added this to the2.13.17 milestoneMar 31, 2025
@som-snyttsom-snyttforce-pushed theissue/6805-trait-parameters branch from5edb059 to720c1cdCompareMarch 31, 2025 06:32
@lrytz
Copy link
Member

Scala 3 also allows the empty argument list, so maybe it's not worth cleaning that up in Scala 2?

@som-snytt
Copy link
ContributorAuthor

som-snytt commentedMar 31, 2025
edited
Loading

TIL. I'll try to understand the Scala 3 feature well enough to know if that is plain wrong.

Edit: haven't gotten around to that, but for now I did allow just thenew syntax

new T() {} // looks like Java, and arguably they are the parens to the anon class

Should be a lint warning.

Before committing to this PR, what I really wanted to do is try backporting trait params. That way, I'll understand how trait params work (until I forget again).

@som-snyttsom-snyttforce-pushed theissue/6805-trait-parameters branch 4 times, most recently from41aafb7 to831b71dCompareApril 1, 2025 18:57
@som-snyttsom-snytt changed the titleCheck trivial trait argsCheck trivial trait args [ci: last-only]Apr 4, 2025
@som-snyttsom-snyttforce-pushed theissue/6805-trait-parameters branch from831b71d todee9cacCompareApril 4, 2025 23:20
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.13.17
Development

Successfully merging this pull request may close these issues.

Compiler allows empty argument lists for traits
3 participants
@som-snytt@lrytz@scala-jenkins

[8]ページ先頭

©2009-2025 Movatter.jp