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

deprecates resetAllAttrs and resetLocalAttrs in favor of the new API #3348

Closed
wants to merge 2 commits into from

Conversation

xeno-by
Copy link
Member

@xeno-by xeno-by commented Jan 10, 2014

We now have c.untypecheck, which is supposed to be a counterpart of c.typecheck
in the sense that it goes back from typed trees to untyped ones:
http://stackoverflow.com/questions/20936509/scala-macros-what-is-the-difference-between-typed-aka-typechecked-an-untyped.

Let’s hope that c.untypecheck will soon be able to solve our problems
with partially/incorrectly attributed trees emitted by macros:
https://groups.google.com/forum/#!topic/scala-internals/TtCTPlj_qcQ.

@ghost ghost assigned retronym Jan 10, 2014
@xeno-by
Copy link
Member Author

xeno-by commented Jan 10, 2014

review @retronym

@xeno-by
Copy link
Member Author

xeno-by commented Jan 10, 2014

Please feel free to decide whether this should go into M8 or RC1.

*
* Read http://stackoverflow.com/questions/20936509/scala-macros-what-is-the-difference-between-typed-aka-typechecked-an-untyped
* for more information.
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way to avoid copy-paste here? If not, a side-comment that somehow textually links the two would be helpful.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure whether there are scaladoc macros that can span multiple modules. @heathermiller @VladUreche could you help here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, they can't 👎 Sorry :(

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about a link ("more details in ...") from the documentation one method to the docs of the other?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, we could link the toolbox doc (arguably, much less important) to the context doc. Will do now.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That would work, thanks Jason! Btw, the last line:

   *  Read http://stackoverflow.com/questions/20936509/scala-macros-what-is-the-difference-between-typed-aka-typechecked-an-untyped
   *  for more information.

could also be written as:

   *  @see [[http://stackoverflow.com/questions/20936509/scala-macros-what-is-the-difference-between-typed-aka-typechecked-an-untyped]]

@xeno-by
Copy link
Member Author

xeno-by commented Jan 10, 2014

@retronym @densh Addresses the feedback

This cute little type is necessary for importers to work correctly.
I wonder how we could overlook its existence for almost 2 years.
We now have c.untypecheck, which is supposed to be a counterpart of c.typecheck
in the sense that it goes back from typed trees to untyped ones:
http://stackoverflow.com/questions/20936509/scala-macros-what-is-the-difference-between-typed-aka-typechecked-an-untyped.

Let’s hope that c.untypecheck will soon be able to solve our problems
with partially/incorrectly attributed trees emitted by macros:
https://groups.google.com/forum/#!topic/scala-internals/TtCTPlj_qcQ.
@xeno-by
Copy link
Member Author

xeno-by commented Jan 11, 2014

PLS REBUILD ALL

@scala-jenkins
Copy link

(kitty-note-to-self: ignore 32096705)
🐱 Roger! Rebuilding pr-scala for 917de78, db623cf. 🚨

@retronym
Copy link
Member

PLS REBUILD ALL

@scala-jenkins
Copy link

(kitty-note-to-self: ignore 32246131)
🐱 Roger! Rebuilding pr-scala for 917de78, db623cf. 🚨

@xeno-by
Copy link
Member Author

xeno-by commented Jan 16, 2014

PLS REBUILD ALL

@scala-jenkins
Copy link

(kitty-note-to-self: ignore 32529856)
🐱 Roger! Rebuilding pr-scala for 917de78, db623cf. 🚨

@xeno-by
Copy link
Member Author

xeno-by commented Jan 20, 2014

PLS REBUILD ALL

@scala-jenkins
Copy link

(kitty-note-to-self: ignore 32759335)
🐱 Roger! Rebuilding pr-scala for 917de78, db623cf. 🚨

@xeno-by
Copy link
Member Author

xeno-by commented Jan 21, 2014

This PR seems to be particularly unlucky with the kitten. I'll close it and reopen it again.

@xeno-by xeno-by closed this Jan 21, 2014
@xeno-by
Copy link
Member Author

xeno-by commented Jan 21, 2014

#3392

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