Details
-
Type:
Bug
-
Status:
Resolved
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.6
-
Fix Version/s: 1.6
-
Component/s: None
-
Labels:None
-
Number of attachments :
Description
The parser should support generics in field assignments.
class X {
Object y = new HashMap<A,B>();
}
It used to support this, however I had to roll it back as it introduced a critical bug that caused the parser to barf when parsing fields containing < or > characters (QDOX-71), which is common in existing source code.
The following tests are already in place, but are failing:
GenericsTest.FIXME_testJiraQdox66()
LexerTest.FIXME_testGenericTypeAssignment()
Activity
Dirk Moebius <dmoebius@gmx.net> has contributed a (much appreciated) patch.
I've attached the patch and his email.
Show
Mike Williams
added a comment - Dirk Moebius <dmoebius@gmx.net> has contributed a (much appreciated) patch.
I've attached the patch and his email.
Mike Williams
added a comment - Dirk Moebius <dmoebius@gmx.net> has contributed a (much appreciated) patch.
I've attached the patch and his email. Mike Williams
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Attachment | qdox-77.patch [ 14193 ] | |
| Attachment | qdox-77-dirk-email.txt [ 14192 ] |
Well, that took a while to commit 
Actually, Mario Siegenthaler submitted a patch which looks very similar to Dirk's, and I ended up committing that.
Show
Mike Williams
added a comment - Well, that took a while to commit
Actually, Mario Siegenthaler submitted a patch which looks very similar to Dirk's, and I ended up committing that.
Mike Williams
added a comment - Well, that took a while to commit
Actually, Mario Siegenthaler submitted a patch which looks very similar to Dirk's, and I ended up committing that. Mike Williams
made changes -
| Resolution | Fixed [ 1 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] |
The changes I made to rollback the support are here.
http://cvs.codehaus.org/changelog/qdox/?cs=MAIN:joe:20050103131703