Bug#474908: [Fwd: [ cdk-Bugs-1938953 ] Build fails with new JavaCC beta - MakeJavafilesFiles]

[Paul Cager]

It's now been fixed upstream as well.

---------------------------- Original Message ----------------------------
Subject: [ cdk-Bugs-1938953 ] Build fails with new JavaCC beta -
MakeJavafilesFiles
From:    "SourceForge.net" <noreply at sourceforge.net>
Date:    Mon, April 21, 2008 21:16
To:      noreply at sourceforge.net
--------------------------------------------------------------------------

Bugs item #1938953, was opened at 2008-04-09 23:20
Message generated for change (Settings changed) made by egonw
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=120024&aid=1938953&group_id=20024

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: build.xml
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Paul Cager (paulcager)
Assigned to: Egon Willighagen (egonw)
Summary: Build fails with new JavaCC beta - MakeJavafilesFiles

Initial Comment:
The new version of JavaCC (not yet released, but available from JavaCC's
CVS) generates code with comments of the form:

   /** Token Manager. */

Unfortunately MakeJavafilesFiles.java doesn't handle this type of comment
- if the comment starts with "/**" it only looks for "**/" as a comment
terminator (if it is a single-line comment). I have attached a patch I use
in the Debian build to fix MakeJavafilesFiles.java (although you may want
to fix it in a more robust way).

(If you need the newer version of JavaCC I would be happy to send you the
jar).

Thanks.

----------------------------------------------------------------------

>Comment By: Egon Willighagen (egonw)
Date: 2008-04-21 22:15

Message:
Logged In: YES
user_id=25678
Originator: NO

Hi  Paul,

at some point I've looked for a good Java source parser that also parser
JavaDoc, but never found a free tool for that. No idea, why it specifically
looks for **/... I'll apply the patch.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=120024&aid=1938953&group_id=20024