Status: New
Owner: ----
New issue 531 by [email protected]: Followup matters to issue 526 - last
part of neatly separating the c.g.i.internal.util package from
c.g.i.internal
http://code.google.com/p/google-guice/issues/detail?id=531
As the issue tracker doesn't let me reopen an issue, I'm creating a new one:
Summary of overall matter from the previous issue:
I would like to propose that the c.g.i.internal.util package should have no
dependencies on c.g.i.internal (other than @Nullable, which cannot really
be avoided).
Some modifications in this direction were committed in response to issue
526. However, one remains:
In order to achieve this, I propose that the memberType and toString
methods from c.g.i.internal.MoreTypes move to c.g.i.internal.util.Classes.
Not only does this break the last (non-annotation) dependency of the
internal.util package on the main package, it also neatly removes from
MoreTypes these two methods which have nothing to do with generic Types and
the rest of the MoreTypes class.
(Granted this is merely a code-cleanliness issue, and a minor one at that,
but we've come this far towards a well-defined separation of utility code
from the core logic of Guice, so I felt it should be raised.)
--
You received this message because you are subscribed to the Google Groups
"google-guice-dev" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/google-guice-dev?hl=en.
