Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 358676 - [Test] Remove unnecessary reflection code in org.eclipse.ua.tests
Summary: [Test] Remove unnecessary reflection code in org.eclipse.ua.tests
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: User Assistance (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 trivial (vote)
Target Milestone: 3.8 M3   Edit
Assignee: Chris Goldthorpe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-22 18:38 EDT by Chris Goldthorpe CLA
Modified: 2011-09-22 19:04 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Goldthorpe CLA 2011-09-22 18:38:19 EDT 
org.eclipse.ua.tests.help.util.LoadServletUtil.setTimeout(URLConnection, int) uses reflection to invoke two methods which were introduced in Java 5. Since this bundle now has a minimum execution environment of J2SE-1.5 this can be replaced with calls to the methods directly.
Comment 1 Chris Goldthorpe CLA 2011-09-22 18:44:10 EDT 
Fixed in master, the commit message is:
Bug 358676 - [Test] Remove unnecessary reflection code in org.eclipse.ua.tests
Comment 2 Chris Goldthorpe CLA 2011-09-22 19:04:54 EDT 
There were two more similar cases which have been committed with the message

Bug 358676 ( Part 2 of fix ) - [Test] Remove unnecessary reflection code in org.eclipse.ua.tests