[ 
https://issues.apache.org/jira/browse/GROOVY-8204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16025690#comment-16025690
 ] 

ASF GitHub Bot commented on GROOVY-8204:
----------------------------------------

Github user paulk-asert commented on a diff in the pull request:

    https://github.com/apache/groovy/pull/551#discussion_r118622384
  
    --- Diff: src/main/org/codehaus/groovy/ast/ClassHelper.java ---
    @@ -440,12 +440,15 @@ private static boolean 
hasUsableImplementation(ClassNode c, MethodNode m) {
         /**
          * Returns a super class or interface for a given class depending on a 
given target.
          * If the target is no super class or interface, then null will be 
returned.
    +     * For a non=primitive array type, returns an array of the 
componentType's superclass.
    --- End diff --
    
    fixed the typo, thanks


> @Delegate on arrays causes NPE during compilation
> -------------------------------------------------
>
>                 Key: GROOVY-8204
>                 URL: https://issues.apache.org/jira/browse/GROOVY-8204
>             Project: Groovy
>          Issue Type: Bug
>    Affects Versions: 2.4.11
>            Reporter: Krzysztof SzafraƄski
>            Priority: Minor
>
> I have stumbled upon this bug partly by accident, and I'm not convinced it's 
> a valid use case, but either way it should not crash the compiler with a NPE.
> Trying to compile the following simple class:
> {code}
> import groovy.lang.Delegate
> class BugsMe {
>       @Delegate
>       byte[] file
> }
> {code}
> Results in the following exception:
> {code}
> >>> a serious error occurred: BUG! exception in phase 'canonicalization' in 
> >>> source unit 'BugsMe.groovy' unexpected NullpointerException
> >>> stacktrace:
> BUG! exception in phase 'canonicalization' in source unit 'BugsMe.groovy' 
> unexpected NullpointerException
>       at 
> org.codehaus.groovy.control.CompilationUnit.applyToPrimaryClassNodes(CompilationUnit.java:1070)
>       at 
> org.codehaus.groovy.control.CompilationUnit.doPhaseOperation(CompilationUnit.java:603)
>       at 
> org.codehaus.groovy.control.CompilationUnit.processPhaseOperations(CompilationUnit.java:581)
>       at 
> org.codehaus.groovy.control.CompilationUnit.compile(CompilationUnit.java:558)
>       at 
> org.codehaus.groovy.control.CompilationUnit.compile(CompilationUnit.java:537)
>       at 
> org.codehaus.groovy.tools.FileSystemCompiler.compile(FileSystemCompiler.java:61)
>       at 
> org.codehaus.groovy.tools.FileSystemCompiler.doCompilation(FileSystemCompiler.java:217)
>       at 
> org.codehaus.groovy.tools.FileSystemCompiler.commandLineCompile(FileSystemCompiler.java:150)
>       at 
> org.codehaus.groovy.tools.FileSystemCompiler.commandLineCompileWithErrorHandling(FileSystemCompiler.java:180)
>       at 
> org.codehaus.groovy.tools.FileSystemCompiler.main(FileSystemCompiler.java:164)
>       at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>       at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
>       at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>       at java.lang.reflect.Method.invoke(Method.java:498)
>       at 
> org.codehaus.groovy.tools.GroovyStarter.rootLoader(GroovyStarter.java:109)
>       at org.codehaus.groovy.tools.GroovyStarter.main(GroovyStarter.java:131)
> Caused by: java.lang.NullPointerException
>       at 
> org.codehaus.groovy.ast.ClassHelper.getNextSuperClass(ClassHelper.java:454)
>       at 
> org.codehaus.groovy.ast.ClassHelper.getNextSuperClass(ClassHelper.java:449)
>       at 
> org.codehaus.groovy.ast.tools.GenericsUtils.extractSuperClassGenerics(GenericsUtils.java:444)
>       at 
> org.codehaus.groovy.transform.DelegateASTTransformation.addDelegateMethod(DelegateASTTransformation.java:233)
>       at 
> org.codehaus.groovy.transform.DelegateASTTransformation.visit(DelegateASTTransformation.java:129)
>       at 
> org.codehaus.groovy.transform.ASTTransformationVisitor.visitClass(ASTTransformationVisitor.java:134)
>       at 
> org.codehaus.groovy.transform.ASTTransformationVisitor$2.call(ASTTransformationVisitor.java:178)
>       at 
> org.codehaus.groovy.control.CompilationUnit.applyToPrimaryClassNodes(CompilationUnit.java:1065)
>       ... 15 more
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to