Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 333835 - TestValidator4 produces different byte codes with new compiler
Summary: TestValidator4 produces different byte codes with new compiler
Status: RESOLVED FIXED
Alias: None
Product: WTP Common Tools
Classification: WebTools
Component: wst.validation (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Carl Anderson CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 333678
  Show dependency tree
 
Reported: 2011-01-10 02:08 EST by David Williams CLA
Modified: 2011-01-13 14:48 EST (History)
1 user (show)

See Also:
ccc: review+

Attachments
hope this helps (1.15 KB, patch)
2011-01-10 02:08 EST, David Williams CLA 		no flags Details | Diff
View All 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 David Williams CLA 2011-01-10 02:08:59 EST 
Created attachment 186360 [details]
hope this helps

I'm assuming similar to other bugs in bug 333678, but this case I think involves incorrect, or at least odd, logic. 

It has a line similar to 

for (long i=0; i< 10000000; i++)j = i + 1;

which basically just does a million computations for unknown reasons. There might be some reason ... but sure looks like an error to me. 

I'll attach patch that keeps the behavior pretty much as it has been (still sleeps for 2 seconds) but takes out the computation. There may be other ways of fixing the code, if anyone knows what its supposed to be accomplishing ... but, if it was me ... I'd just cutout anything that wasn't understood.
Comment 1 Carl Anderson CLA 2011-01-13 14:48:28 EST 
Committed to R3_2_maintenance and HEAD for WTP 3.2.3 and WTP 3.3 M5