[jira] Commented: (HARMONY-2052) [drlvm] Improve/re-implement a parser of generic signatures

["Paulex Yang (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HARMONY-2052?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12465685 ] 

Paulex Yang commented on HARMONY-2052:
--------------------------------------

This is the first part of the proposed refactory, for the comments of interest part. This is part is for decoupling  j.l.r and o.a.h.l.r packages, so that the real implementation can be improved easily (and help myself understand the code as well). Currently the refactory is straightfoward and somewhat naive, no need to say that there are much things left to do, performance, compiler warning, etc, and nothing improved on the implementation itself. I'll go on with it if this patch is fine.

> [drlvm] Improve/re-implement a parser of generic signatures
> -----------------------------------------------------------
>
>                 Key: HARMONY-2052
>                 URL: https://issues.apache.org/jira/browse/HARMONY-2052
>             Project: Harmony
>          Issue Type: Task
>          Components: Classlib, DRLVM
>            Reporter: Alexey Varlamov
>         Assigned To: Paulex Yang
>            Priority: Minor
>         Attachments: HARMONY-2052-1.diff
>
>
> Improve/re-implement a parser of generic signatures in DRLVM kernel classes [1], and move this functionality to classlib (luni ?), so other VMs could reuse it for 1.5 support. The current impl is somewhat messy and half-baked, one need to invent more shaped and modular API to the parser. One more possible issue is parser's dependency on antlr, which may be considered overkill for this duty. I think antlr has its pros, like more illustrative code with clear correlation to formal grammar [2]; unfortunately this is not the case with the impl in question. OTOH minimizing number of dependencies for VM is always good. 
> [1] working_vm\vm\vmcore\src\kernel_classes\javasrc\org\apache\harmony\lang\reflect\** 
> [2]  http://java.sun.com/docs/books/vmspec/2nd-edition/ClassFileFormat-Java5.pdf Para 4.4.4 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira