Skip to content

Update overview.md #927

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

Merged
merged 1 commit into from
Nov 18, 2017
Merged

Update overview.md #927

merged 1 commit into from
Nov 18, 2017

Conversation

karlroberts
Copy link
Contributor

The method Type.delaration has been depricated in Scala 2.11.0 in favour of Type.decls.

The method Type.delaration has been depricated in Scala 2.11.0 in favour of Type.decls.
@SethTisue SethTisue merged commit af59b3e into scala:master Nov 18, 2017
@SethTisue
Copy link
Member

good catch. thank you!

@karlroberts karlroberts deleted the patch-1 branch November 20, 2017 12:08
@@ -132,7 +132,7 @@ The second step involves obtaining a `ClassMirror` for class `Person` using
the `reflectClass` method. The `ClassMirror` provides access to the
constructor of class `Person`.

scala> val ctor = ru.typeOf[Person].declaration(ru.nme.CONSTRUCTOR).asMethod
scala> val ctor = ru.typeOf[Person].decls(ru.nme.CONSTRUCTOR).asMethod
Copy link
Contributor

Choose a reason for hiding this comment

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

@SethTisue This PR introduced a bug, replacing declaration (singular) with decls (plural) here and on line 174 below. A fix is part of #935

@@ -171,7 +171,7 @@ which makes all classes and types available that are loaded by the classloader
that also loaded the class of `p` (`Purchase`), which we need in order to
access member `shipped`.

scala> val shippingTermSymb = ru.typeOf[Purchase].declaration(ru.TermName("shipped")).asTerm
scala> val shippingTermSymb = ru.typeOf[Purchase].decls(ru.TermName("shipped")).asTerm
Copy link
Contributor

Choose a reason for hiding this comment

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

The code as is (ru.typeOf[Purchase].decls(ru.TermName("shipped")).asTerm) produces the following error: "reflect.runtime.universe.MemberScope does not take parameters"

Copy link
Member

Choose a reason for hiding this comment

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

fixed in #935?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes

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

Successfully merging this pull request may close these issues.

3 participants