[jira] Commented: (HARMONY-3133) [drlvm][classloader] Add check of field and method names for classfile of version lower than 49

["Gregory Shimansky (JIRA)" <ji...@apache.org>]

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

Gregory Shimansky commented on HARMONY-3133:
--------------------------------------------

I've tried this patch and it looks like cunit tests on Linux don't work. They fail to link with libharmonyvm.so with the following error message:

     [echo] ## Linking C unit test: test_java_attrs
       [cc] 0 total files to be compiled.
       [cc] Starting link
       [cc] /usr/lib64/gcc-lib/x86_64-suse-linux/3.3.3/../../../../x86_64-suse-linux/bin/ld: warning: libicuuc.so.34, needed by /nfs/ims/proj/drl/mrt1/users/gregory/em64t/trunk/working_vm/build/lnx_em64t_gcc_debug/semis/vm/vmcore/_bin/libharmonyvm.so, not found (try using -rpath or -rpath-link)
       [cc] /nfs/ims/proj/drl/mrt1/users/gregory/em64t/trunk/working_vm/build/lnx_em64t_gcc_debug/semis/vm/vmcore/_bin/libharmonyvm.so: undefined reference to `u_isalpha_3_4'
       [cc] /nfs/ims/proj/drl/mrt1/users/gregory/em64t/trunk/working_vm/build/lnx_em64t_gcc_debug/semis/vm/vmcore/_bin/libharmonyvm.so: undefined reference to `u_isalnum_3_4'
       [cc] collect2: ld returned 1 exit status


> [drlvm][classloader] Add check of field and method names for classfile of version lower than 49
> -----------------------------------------------------------------------------------------------
>
>                 Key: HARMONY-3133
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3133
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: Vera Volynets
>         Attachments: icu_use_classlib.patch, icu_use_vm.patch
>
>
> Accordingly to specification of 1.4 Java field and method names are identifiers, see 4.5, 4.6, 2.2.
> That means that name should consist of Unicode letters and digits, first must be letter.
> To check that letter is Unicode, I use icuuc module.
> Changes in patch:
> 1) link icuuc library, changes in .xml files and make files
> 2) changes in Class_File_Loader.cpp. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.