Review request: 8029548 (jdeps) use @jdk.Exported to determine supported vs JDK internal API

[Daniel Fuchs]

Hi Mandy,

On 7/8/14 4:59 PM, Mandy Chung wrote:
> I took out the special case for javax.jnlp and follow up the change in
> deploy to use @jdk.Exported to indicate its supportedness.  I also added
> a new test case to check the dot file output:
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8029548/webrev.01

I installed the new patch and played with it, and I find the behavior
of jdeps -R much clearer than in your first version.

Thanks for clarifying the help messages too!

Otherwise - I browsed through the changes and didn't see anything
obviously wrong.

best regards,

-- daniel

>
> thanks
> Mandy
>
> On 6/25/14 1:21 PM, Mandy Chung wrote:
>> This patch is also intended to target for 8u40 backport.  It fixes
>> the following issues:
>>
>> JDK-8029548 (jdeps) use @jdk.Exported to determine supported vs JDK
>> internal API
>> JDK-8039007 jdeps incorrectly reports javax.jnlp as JDK internal APIs
>> JDK-8031092 jdeps does not recognize --help option.
>> JDK-8048063 (jdeps) Add filtering capability
>>
>> Webrev at:
>> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8029548/webrev.00/
>>
>> jdeps will determine if it's a JDK internal API by checking the
>> @jdk.Exported.
>> It also reads tools.jar and flags if they are non-exported. javax.jnlp is
>> supported API but not annotated with @jdk.Exported and so we special case
>> it as a workaround. A new -filter option is added to provide an easy
>> way to do filtering e.g. -filter java.lang.*, -filter:archive to filter
>> out all dependences from the same archive (i.e the input JAR file or
>> directory).  The default is changed to filter out the dependences
>> within the same package.  I also have cleaned up and refactored some code
>> between JdepsTask and Analyzer to prepare for source restructuring
>> and modularization work.
>>
>> thanks
>> Mandy
>>
>