Repository: groovy Updated Branches: refs/heads/GROOVY_2_5_X 53d2d7508 -> 8587cb271
Update javadoc for `GenericsTypeName` (cherry picked from commit bd9a237b177834f6dd1363a84832fffeb6b745ba) Project: http://git-wip-us.apache.org/repos/asf/groovy/repo Commit: http://git-wip-us.apache.org/repos/asf/groovy/commit/8587cb27 Tree: http://git-wip-us.apache.org/repos/asf/groovy/tree/8587cb27 Diff: http://git-wip-us.apache.org/repos/asf/groovy/diff/8587cb27 Branch: refs/heads/GROOVY_2_5_X Commit: 8587cb271b70afef4101375041829b42fe367f3c Parents: 53d2d75 Author: Daniel Sun <[email protected]> Authored: Sun Aug 19 20:47:30 2018 +0800 Committer: Daniel Sun <[email protected]> Committed: Sun Aug 19 20:48:05 2018 +0800 ---------------------------------------------------------------------- src/main/java/org/codehaus/groovy/ast/GenericsType.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/groovy/blob/8587cb27/src/main/java/org/codehaus/groovy/ast/GenericsType.java ---------------------------------------------------------------------- diff --git a/src/main/java/org/codehaus/groovy/ast/GenericsType.java b/src/main/java/org/codehaus/groovy/ast/GenericsType.java index 5041fb1..6e8dcc0 100644 --- a/src/main/java/org/codehaus/groovy/ast/GenericsType.java +++ b/src/main/java/org/codehaus/groovy/ast/GenericsType.java @@ -501,7 +501,15 @@ public class GenericsType extends ASTNode { /** * Represents GenericsType name - * TODO In order to distinguish GenericsType with same name(See GROOVY-8409), we should add a property to keep the declaring class full name. + * TODO In order to distinguish GenericsType with same name(See GROOVY-8409), we should add a property to keep the declaring class. + * + * fixing GROOVY-8409 steps: + * 1) change the signature of constructor GenericsTypeName to `GenericsTypeName(String name, ClassNode declaringClass)` + * 2) try to fix all compilation errors(if `GenericsType` has declaringClass property, the step would be a bit easy to fix...) + * 3) run all tests to see whether the change breaks anything + * 4) if all tests pass, congratulations! but if some tests are broken, try to debug and find why... + * + * We should find a way to set declaring class for `GenericsType` first, it can be completed at the resolving phase. */ public static class GenericsTypeName { private String name;
