Scaladoc: fixes and improvements to context bounds and extension methods#22156
Conversation
67ee8ec to
7540be2
Compare
|
The change works for me. |
|
@Florian3k , if you do not have more changes, maybe we could remove the draft status, what do you think? @mbovel please could you consider a review? Thanks! |
|
There's one more thing to fix. Changes in this PR uncovered some bugs in rendering of |
Co-authored-by: Florian3k <mfornal@virtuslab.com>
7540be2 to
86c6894
Compare
|
Note for @Florian3k, about my adjustments with this-types: |
|
The new changes work for me. |
|
Those new changes seem to be working fine. |
| sealed trait CaseImplementThis(id: Int) | ||
|
|
||
| case class IAmACaseClass(x: Documentation.this.T, id: Int) extends CaseImplementThis/*<-*/(id)/*->*/ | ||
| case class IAmACaseClass(x: T, id: Int) extends CaseImplementThis(id) //expected: case class IAmACaseClass(x: Documentation.this.T, id: Int) extends CaseImplementThis |
There was a problem hiding this comment.
I've adjusted this test case to better reflect what is really going on here.
Closes #21662
More specifically:
\:and\\:inextensionMethodSignatures.scala, also see Question: desugaring of right associative extenstion methods #22170)><inextensionMethodSignatures.scala)Additionally fixed and improved simplifying of type lambdas (
TypesSupport.scala)Removed
MemberInfoandunwrapMemberInfo, which were quite complicated, doing several things at the same time and causedparseMemberto have two sources of truth.