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

Support @Inject inheritence #73

Closed
gissuebot opened this issue Jul 7, 2014 · 2 comments
Closed

Support @Inject inheritence #73

gissuebot opened this issue Jul 7, 2014 · 2 comments
Labels
bug imported invalid Priority-Medium

Comments

@gissuebot
Copy link

From chandra.patni on March 16, 2007 16:54:26

public interface Cache {
  public void setMemory(long bytes);
  @Inject public void setMemory(@Named("memorySize") String size);
}

This doesn't work unless I move the annoation to the implementation class.

Original issue: http://code.google.com/p/google-guice/issues/detail?id=73
@gissuebot
Copy link
Author

From kevinb9n on March 16, 2007 15:22:51

Can you please explain why you believe this should be supported?  It's hard to make a
decision without that.

@gissuebot
Copy link
Author

From kevinb9n on March 19, 2007 11:28:09

In the absence of compelling new information, this doesn't make sense to us.  An
interface is a contract and a contract says what information it needs; not where
you're supposed to get the information from.  That's the province of impl code.

Status: Invalid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug imported invalid Priority-Medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gissuebot