QDox
  1. QDox
  2. QDOX-61

Problems with non-US-ASCII characters in identifiers

          Details

          • Type: Bug Bug
          • Status: Closed Closed
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 1.6
          • Component/s: None
          • Labels:
            None
          • Number of attachments :
            0

            Description

            If a class has an identifier, say a field, that uses characters other than the ones specified below, the parsing stops there and the rest of the fields will not be parsed.

            This is from the QDox source code:
            [A-Za-z_$0-9]*

            { return Parser.IDENTIFIER; }

            Somewhere inside either QDox or vdoclet (@sourceforge) there should be a loop that goes through all the fields in a class. The parser will probably throw a ParseException when it discovers a "bad" field. Now if that exception was caught inside the loop instead of outside the loop, we should at least be able to parse the rest of the fields. I am not sure whether the problem is in QDox or vdoclet.

            This has also been raised as an issue in the maven-tasklist-plugin. A full explanation, including a test-class, can be found here:
            http://jira.codehaus.org/browse/MPTASKLIST-11

              Activity

              Hide
              Permalink
              Mike Williams added a comment -

              Most likely a problem with the QDox parser.

              I shall try to sort it later this week (unless Aslak beats me to it).

              Show
              Mike Williams added a comment - Most likely a problem with the QDox parser. I shall try to sort it later this week (unless Aslak beats me to it).
              Mike Williams made changes -
              Field Original Value New Value
              Fix Version/s 1.6 [ 10814 ]
              Mike Williams made changes -
              Assignee Mike Williams [ mdub ]
              Mike Williams made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Hide
              Permalink
              Mike Williams added a comment -

              Dennis, sorry about the long wait for this bug-fix!

              Show
              Mike Williams added a comment - Dennis, sorry about the long wait for this bug-fix!
              Mike Williams made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Dennis Lundberg added a comment -

              Thanks for your help Mike!

              I have tried to see if it solves my problems over in mavenland. Here's what I have done:

              1. Downloaded the qdox-1.6-SNAPSHOT.jar
              2. Checked out vdoclet from cvs @ sourceforge
              3. Updated vdoclets dependency on qdox to 1.6-SNAPSHOT
              4. Built my own 20041104-version of vdoclet
              5. Updated the dependencies in maven-plugins/vdoclet to vdoclet-20041104 and qdox-1.6-SNAPSHOT
              6. Rebuild the vdoclet plugin and installed it
              7. Ran maven site:generate to create a tasklist report on one a mavenized project, using my patched version of the tasklist-plugin

              And it works like a charm!

              Is there an ETA on qdox-1.6 final?

              Show
              Dennis Lundberg added a comment - Thanks for your help Mike! I have tried to see if it solves my problems over in mavenland. Here's what I have done: 1. Downloaded the qdox-1.6-SNAPSHOT.jar 2. Checked out vdoclet from cvs @ sourceforge 3. Updated vdoclets dependency on qdox to 1.6-SNAPSHOT 4. Built my own 20041104-version of vdoclet 5. Updated the dependencies in maven-plugins/vdoclet to vdoclet-20041104 and qdox-1.6-SNAPSHOT 6. Rebuild the vdoclet plugin and installed it 7. Ran maven site:generate to create a tasklist report on one a mavenized project, using my patched version of the tasklist-plugin And it works like a charm! Is there an ETA on qdox-1.6 final?
              Hide
              Permalink
              Mike Williams added a comment -

              > Here's what I have done:
              > ...
              >
              > And it works like a charm!

              Hi Dennis. Thanks for the feedback.

              > Is there an ETA on qdox-1.6 final?

              ETA: late November.

              Show
              Mike Williams added a comment - > Here's what I have done: > ... > > And it works like a charm! Hi Dennis. Thanks for the feedback. > Is there an ETA on qdox-1.6 final? ETA: late November.
              Mike Williams made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mike Williams
                  Reporter:
                  Dennis Lundberg
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: