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

that with default #199

Closed
vendethiel opened this issue Jan 26, 2013 · 9 comments
Closed

that with default #199

vendethiel opened this issue Jan 26, 2013 · 9 comments

Comments

@vendethiel
Copy link
Contributor

That's maybe something designed, but that isn't anaphorized (switch a; case 1 then; default that) because default is not in Switch's aTargets (I know you know).

@satyr
Copy link
Owner

satyr commented Jan 26, 2013

something designed

Yep. Same for else.

@vendethiel
Copy link
Contributor Author

I can see the why for else, since it's always falsy, but that's not the case for default

@satyr
Copy link
Owner

satyr commented Jan 26, 2013

Hm. Normal switch is an oddball for anaphorization to begin with: the topic isn't a truthy-check unlike others.

Options:

  1. Quit anaphorizing normal switch (with topic).
  2. Quit anaphorizing switch at all.
  3. Anaphorize for else/default as well.
  4. Leave them as is.

@vendethiel
Copy link
Contributor Author

Not sure what you mean "with topic" ?

I suppose it could be added to else too (got into this case only once) for consistency with default, but that doesn't make must sense to me.

@satyr
Copy link
Owner

satyr commented Jan 26, 2013

switch topic  # <--
case candidate
  ...

switch  # topic-less
case condition
  ...

@vendethiel
Copy link
Contributor Author

  1. or 5) anaphorize default but do not anaphorize topicless switch :D.

@satyr
Copy link
Owner

satyr commented Jan 26, 2013

do not anaphorize topicless switch

Why?

@vendethiel
Copy link
Contributor Author

Because I had no idea it was giving this result. My bad, I retract that point. (I hardly see anaphorization for default in topic-less switch)

@dbkaplun
Copy link

I would opine to anaphorize default as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants