-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Should a comment reference on a type alias be able to refer to members of the aliased type? #56259
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
Comments
I think we should support referencing the members of the aliased type. It seems sensible, given someone wants to document a type alias. |
I'd say "no". The rule I use to understand what's in scope is that it's what's in scope inside the commented declaration (where "in scope" really means "can be referenced by a raw identifier" rather than the actual lexical scope). That said, aliases that allow static member access could be considered "special", and provide access to at least those static members. Could say that they give access to the instance members too, for good measure. But it's because they're special, not because it's obvious. |
As a concrete example, if we wanted to keep a doc comment on SetMixin and refer to members of |
The language specification unambiguously defines the current scope for a documentation comment:
This basically implies that a documentation comment on a class can "see" the members declared in the body of that class, the formal type parameters of the class, if any, and anything in the enclosing library scope (including imported stuff). We should probably make this specification slightly more detailed and say that it is possible to refer to an identifier However, it is not obvious to me that the relevant references in the documentation of a declaration are exactly the ones that are the result of resolving an identifier in the body of said declaration, even with that clarification about You could also say that the members of the interface of a type are relevant to the documentation of a declaration that introduces said type into the enclosing scope. Some declarations introducing a type will also have formal type parameters, and they'd be relevant as well. I think this means "no change" to the current behavior, except that members of a type which is aliased in a An obvious corner case to consider would be I think it would make sense to generalize the rules about documentation comments along these lines. Presumably, it would be a non-breaking change. |
Today we place a declaration's members into the resolvable scope of the declaration's doc comment, including inherited ones:
That comment reference resolves to
C.foo
.But we don't do anything similar for type aliases. For example, should the reference here resolve to
int.isEven
?There are several attempts to reference like this in the Dart SDK, e.g. on SetMixin (the comments resolve in dartdoc because it has a separate resolution mechanism).
The text was updated successfully, but these errors were encountered: