Reviewer needed - IcedTea6 HEAD: fixed some compiler warnings caused by using deprecated methods in JTReg tool

[Pavel Tisnovsky]

Danesh Dadachanji wrote:
> Hi Pavel,
> 
> Not your usual email address? =)

yeah, today morning the corporate mail does not seem to work (mail were sent to /dev/null :-),
so I used my personal mail instead.


> 
> The patch looks good, just a minor comment about the ChangeLog below.
> 
> On 12/01/12 08:03 AM, Pavel Tisnovsky wrote:
>> Hi all,
>>
>> I've made simple fixes in JTReg tool sources (in IcedTea6 HEAD ATM) to
>> get rid of
>> compiler warnings due to use deprecated API methods. Output of hg diff
>> is stored
>> in an attachment and here is the change log entry:
>>
>> 2012-01-12  Pavel Tisnovsky<ptisnovs at redhat.com>
>>          * src/jtreg/com/sun/interview/wizard/Wizard.java:
>>          * src/jtreg/com/sun/javatest/EditJTI.java:
>>          * src/jtreg/com/sun/javatest/TestSuite.java:
>>          * src/jtreg/com/sun/javatest/agent/AgentFrame.java:
>>          * src/jtreg/com/sun/javatest/batch/ObserverCommand.java:
>>          * src/jtreg/com/sun/javatest/exec/BP_TestListSubpanel.java:
>>          * src/jtreg/com/sun/javatest/exec/ET_FilterHandler.java:
>>          * src/jtreg/com/sun/javatest/exec/LogViewerTools.java:
>>          * src/jtreg/com/sun/javatest/exec/NewReportDialog.java:
>>          * src/jtreg/com/sun/javatest/exec/QuestionLogBrowser.java:
>>          * src/jtreg/com/sun/javatest/exec/TU_ViewManager.java:
>>          * src/jtreg/com/sun/javatest/exec/TestTreePanel.java:
>>          * src/jtreg/com/sun/javatest/tool/ConfigManager.java:
>>          Fixed warnings caused by using deprecated methods.
>>
> 
> Could you s/java:/java,/ for all the files listed except the last one
> (ConfigManager.java), it looks like this is the convention for multiple
> files using the same description.
> 
> Once that's fixed, the patch can be pushed to HEAD.

Thank you, I'll correct change log & push it.
Pavel

> 
> Regards,
> Danesh