Skip to content
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

Jump to bottom

ParameterMirror.isPositional is missing #11938

Closed
DartBot opened this issue Jul 20, 2013 · 3 comments
Closed

ParameterMirror.isPositional is missing #11938

DartBot opened this issue Jul 20, 2013 · 3 comments
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue library-mirrors type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@DartBot
Copy link

DartBot commented Jul 20, 2013

This issue was originally filed by diego.rocha...@gmail.com

ParameterMirror currently has the properties isNamed (which is not working, as seen on issue #11334) and isOptional. There is no isPositional property.

According to the language spec, on "Formal Parameters" section, optional parameters are either named parameters or positional parameter, meaning that isOptional is not the same as isPositional.

The expected behavior is:
isOptional = isNamed || isPositional

And analogously:
isPositional = !isNamed && isOptional
@dgrove
Copy link
Contributor

dgrove commented Jul 22, 2013

Set owner to @gbracha.
Added Area-Library, Library-Mirrors labels.

@dgrove
Copy link
Contributor

dgrove commented Jul 22, 2013

Added Triaged label.

@gbracha
Copy link
Contributor

gbracha commented Jul 22, 2013

We could add this; or one could write

!isNamed && isOptional

Added Accepted label.

@DartBot DartBot added Type-Defect area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-mirrors labels Jul 22, 2013
@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed priority-unassigned labels Feb 29, 2016
@a-siva a-siva added closed-as-intended Closed as the reported issue is expected behavior closed-not-planned Closed as we don't intend to take action on the reported issue and removed closed-as-intended Closed as the reported issue is expected behavior labels Aug 5, 2022
@a-siva a-siva closed this as completed Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-not-planned Closed as we don't intend to take action on the reported issue library-mirrors type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kevmoo @dgrove @gbracha @a-siva @DartBot