-
Notifications
You must be signed in to change notification settings - Fork 417
Convert from type to 'var';Convert from 'var' to the appropriate type #722
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
Conversation
I opened https://bugs.eclipse.org/bugs/show_bug.cgi?id=537083 so that we keep track of the need to migrate org.eclipse.jdt.internal.corext.fix.TypeParametersFix (and dependencies) to jdt.core. cc @rgrunber @jjohnstn |
@@ -231,5 +289,52 @@ private int rank(ITypeBinding type) { | |||
}); | |||
} | |||
|
|||
private void handledInferredParametrizedType(ASTNode node, ASTNode declaringNode, AST ast, ASTRewrite rewrite, ImportRewrite importRewrite, ImportRewriteContext context) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
handledInferredParameterizedType
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fbricon Fixed.
test this please |
@fbricon I have done it. |
@@ -0,0 +1,73 @@ | |||
/******************************************************************************* | |||
* Copyright (c) 2016-2017 Red Hat Inc. and others. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2018
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fbricon I have fixed it.
Signed-off-by: Snjezana Peco <[email protected]>
Fixes #696 and #697
The #696 and #697 issues use the same code. That is the reason I have fixed them in the same PR.
Signed-off-by: Snjezana Peco [email protected]