Commit80514f7

authored

Merge pull request#10524 from lrytz/t12858

Fix spurious "nullary overrides nilary" warning

2 parentsf7e30a6 +96d74ac commit80514f7Copy full SHA for 80514f7

File tree

+15

-3

lines changed

+15

-3

lines changed

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -487,9 +487,9 @@ abstract class RefChecks extends Transform {`
`487`	`487`	`warnAdaptedNullaryOverride()`
`488`	`488`	`}`
`489`	`489`	`elseif (member.paramLists.isEmpty) {`
`490`		`-//NullaryOverrideAdapted is only added to symbols being compiled, so check for a mismatch`
`491`		`-//if both symbols are mixed in from the classpath`
`492`		`-if (!member.isStable&& other.paramLists.nonEmpty&&!exempted)`
	`490`	+//Definitions that directly override get a parameter list and a `NullaryOverrideAdapted` attachment
	`491`	`+//in Namers. Here we also warn when there's a mismatch between two mixed-in members.`
	`492`	`+if (!member.isStable&& other.paramLists.nonEmpty&&!exempted&&!other.overrides.exists(javaDetermined))`
`493`	`493`	`warnAdaptedNullaryOverride()`
`494`	`494`	`}`
`495`	`495`	`elseif (other.paramLists.isEmpty) {`

Lines changed: 3 additions & 0 deletions

Lines changed: 9 additions & 0 deletions

Comments

(0)