Parser fails on package with annotations

Paul Hammant added a comment - 01/Sep/06 1:31 AM Can you check again please, there has been a release in the last month.

Franz Allan Valencia See added a comment - 06/Sep/07 11:48 PM What is the status of this issue? I tried this with com.thoughtworks.qdox:qdox:1.6.3 but I'm still getting that error. com.thoughtworks.qdox.parser.ParseException: syntax error @[2,1] in file:/home/franz/sources/com/mycompany/MyClass.java at com.thoughtworks.qdox.parser.impl.Parser.yyerror(Parser.java:716) at com.thoughtworks.qdox.parser.impl.Parser.yyparse(Parser.java:826) at com.thoughtworks.qdox.parser.impl.Parser.parse(Parser.java:697) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:300) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:316) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:312) at com.thoughtworks.qdox.JavaDocBuilder$1.visitFile(JavaDocBuilder.java:369) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:43) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.scan(DirectoryScanner.java:52) at com.thoughtworks.qdox.JavaDocBuilder.addSourceTree(JavaDocBuilder.java:366)

Dennis Lundberg added a comment - 08/Jul/08 3:37 PM This is still broken in the latest version built from svn.

Robert Scholte added a comment - 23/Jan/09 11:22 AM I found a solution. This way the parser won't fail anymore. But the annotation is out of reach if we want to stay backwards compatible. The problem is clear: javaSource.getPackage returns a string. To support annotations for a package it has to return some object like JavaPackage. So I'm not sure what to do...

Paul Hammant added a comment - 04/Feb/09 9:21 AM MATHUS Baptiste wrote: > Hi all, > > I just encountered that very problem: https://svn.cargo.codehaus.org/browse/QDOX-98 when running the paranamer-maven-plugin on a > project that dynamically generates a whole package using JAXB. > > [...] > If the answer is to change 'String javaSource.getPackage()' to 'JavaPackage javaSource.getPackage()', then that is what we have to do. It would be a break of backwards compatibility, but not enough to warrant a jump to QDox 2.x perhaps.

Baptiste MATHUS added a comment - 04/Feb/09 10:01 AM Here's the full stacktrace: [INFO] syntax error @[9,1] in file:/C:/tests/cgacore1.2/src/java/fr/mm/cga/moniteur/queue/xml/genere/ package -info.java [INFO] ------------------------------------------------------------------------ [INFO] Trace com.thoughtworks.qdox.parser.ParseException: syntax error @[9,1] in file:/C:/tests/cgacore1.2/src/java/fr/mm/cga/moniteur/queue/xml/genere/ package -info.java at com.thoughtworks.qdox.parser.impl.Parser.yyerror(Parser.java:987) at com.thoughtworks.qdox.parser.impl.Parser.yyparse(Parser.java:1293) at com.thoughtworks.qdox.parser.impl.Parser.parse(Parser.java:968) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:304) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:324) at com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:320) at com.thoughtworks.qdox.JavaDocBuilder$2.visitFile(JavaDocBuilder.java:400) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:43) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.walk(DirectoryScanner.java:34) at com.thoughtworks.qdox.directorywalker.DirectoryScanner.scan(DirectoryScanner.java:52) at com.thoughtworks.qdox.JavaDocBuilder.addSourceTree(JavaDocBuilder.java:397) at com.thoughtworks.qdox.JavaDocBuilder.addSourceTree(JavaDocBuilder.java:386) at com.thoughtworks.paranamer.generator.QdoxParanamerGenerator.getClassesSortedByName(QdoxParanamerGenerator.java:66) at com.thoughtworks.paranamer.generator.QdoxParanamerGenerator.processSourcePath(QdoxParanamerGenerator.java:60) at com.thoughtworks.paranamer.mojo.ParanamerGeneratorMojo.execute(ParanamerGeneratorMojo.java:77) at org.apache.maven.plugin.DefaultPluginManager.executeMojo(DefaultPluginManager.java:451) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoals(DefaultLifecycleExecutor.java:558) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoalWithLifecycle(DefaultLifecycleExecutor.java:499) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoal(DefaultLifecycleExecutor.java:478) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoalAndHandleFailures(DefaultLifecycleExecutor.java:330) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeTaskSegments(DefaultLifecycleExecutor.java:291) at org.apache.maven.lifecycle.DefaultLifecycleExecutor.execute(DefaultLifecycleExecutor.java:142) at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:336) at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:129) at org.apache.maven.cli.MavenCli.main(MavenCli.java:287) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) at java.lang.reflect.Method.invoke(Method.java:585) at org.codehaus.classworlds.Launcher.launchEnhanced(Launcher.java:315) at org.codehaus.classworlds.Launcher.launch(Launcher.java:255) at org.codehaus.classworlds.Launcher.mainWithExitCode(Launcher.java:430) at org.codehaus.classworlds.Launcher.main(Launcher.java:375) And the content of the parsed file: // // This file was generated by the JavaTM Architecture for XML Binding(JAXB) Reference Implementation, v2.0.5-b02-fcs // See <a href= "http://java.sun.com/xml/jaxb" >http://java.sun.com/xml/jaxb</a> // Any modifications to this file will be lost upon recompilation of the source schema. // Generated on: 2009.02.04 at 01:49:27 PM CET // @javax.xml.bind.annotation.XmlSchema(namespace = "urn: //mm:cga:pgih-administratif-1.0:xsd" ) package fr.mm.cga.moniteur.queue.xml.genere; Don't hesitate to ask for more informations if necessary. HTH.

Paul Hammant added a comment - 14/Feb/09 9:59 AM > if the answer is to change 'String javaSource.getPackage()' to 'JavaPackage javaSource.getPackage()', then that is what we have to do. And we could make JavaPackage implement CharSequence too, as some (poor?) concession to backwards compatibility.

Robert Scholte added a comment - 16/Feb/09 12:19 PM patch ready for review. This code can break backwards compatilbility. Most important modification to keep the code running: getPackage() has be to changed to getPackage().getName()

Paul Hammant added a comment - 16/Feb/09 12:30 PM That was fast work. Looks good to me, commit it Robert

Robert Scholte added a comment - 16/Feb/09 1:10 PM r546 | rfscholte | 2009-02-16 20:09:28 CET fix/improvement for qdox-98 ----------------------------------------------------------------------------