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

reflectModule and friends should be renamed #6156

Closed
scabug opened this issue Jul 30, 2012 · 4 comments
Closed

reflectModule and friends should be renamed #6156

scabug opened this issue Jul 30, 2012 · 4 comments
Assignees
@xeno-by
Labels
reflection won't fix

Comments

@scabug
Copy link

scabug commented Jul 30, 2012

We've been thinking how to name a family of functions in reflection that work with objects (aka modules in compiler speak). A few examples: staticModule, reflectModule, ModuleSymbol etc.

There's a concern that module would be ambiguous because of the associations with packaging, OSGI and stuff. Hence we're looking into the alternative names we could use.
@scabug
Copy link
Author

scabug commented Jul 30, 2012

Imported From: https://issues.scala-lang.org/browse/SI-6156?orig=1
Reporter: @xeno-by
Affected Versions: 2.10.0-M6

@scabug
Copy link
Author

scabug commented Jul 30, 2012

@xeno-by said:
http://groups.google.com/group/scala-internals/browse_thread/thread/bc416d01d8b26f29/1c7744c98d90dbc2?lnk=raot

@scabug
Copy link
Author

scabug commented Jul 30, 2012

@soc said (edited by @xeno-by on Jul 30, 2012 10:32:39 PM UTC):
The REPL response when defining an object probably also needs an update:

scala> object Foo
defined module Foo

@scabug
Copy link
Author

scabug commented Aug 2, 2012

@xeno-by said:
We cannot use Scala's natural terminology and go for "object" here (as in "reflectObject", "staticObject" or "ObjectSymbol"), because that'd be very ambiguous. Therefore we need to come up with a new concept anyways. And if we have to do that, let's go for something that's already in the spec and in the compiler. Following this line of thought we decided to keep "reflectModule".

@scabug scabug closed this as completed Aug 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xeno-by xeno-by

Labels
reflection won't fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xeno-by @scabug