hg: jdk7/tl/langtools: 6907575: [classfile] add support for classfile dependency analysis

Rémi Forax forax at univ-mlv.fr
Sat Dec 12 15:40:01 PST 2009


Le 13/12/2009 00:35, Rémi Forax a écrit :
> Le 12/12/2009 18:33, jonathan.gibbons at sun.com a écrit :
>> Changeset: fbeb560f39e7
>> Author:    jjg
>> Date:      2009-12-12 09:28 -0800
>> URL:       http://hg.openjdk.java.net/jdk7/tl/langtools/rev/fbeb560f39e7
>>
>> 6907575: [classfile] add support for classfile dependency analysis
>> Reviewed-by: ksrini
>>
>> + src/share/classes/com/sun/tools/classfile/Dependencies.java
>> + src/share/classes/com/sun/tools/classfile/Dependency.java
>> + test/tools/javap/classfile/deps/GetDeps.java
>> + test/tools/javap/classfile/deps/T6907575.java
>> + test/tools/javap/classfile/deps/T6907575.out
>> + test/tools/javap/classfile/deps/p/C1.java
>>
>
> Hi Jon,
> I have some comments about this patch set.

[...]

I've forget to say that in GetDeps, there is a typo in originComparator,
getTarget is used instead of getOrigin:

private Comparator<Dependency> originComparator = new 
Comparator<Dependency>() {
             public int compare(Dependency o1, Dependency o2) {
                 return 
o1.getTarget().toString().compareTo(o2.getOrigin().toString());
                                 ^^^^^^^

             }
         };

Cheers,
Rémi



More information about the compiler-dev mailing list