Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 333835

Summary:

TestValidator4 produces different byte codes with new compiler

Product:

[WebTools] WTP Common Tools

Reporter:

David Williams <david_williams>

Component:

wst.validation

Assignee:

Carl Anderson <ccc>

Status:

RESOLVED FIXED

QA Contact:

Chuck Bridgham <cbridgha>

Severity:

normal

Priority:

CC:

ccc

Version:

unspecified

Flags:

ccc: review+

Target Milestone:

3.2.3

Hardware:

OS:

Windows 7

Whiteboard:

Bug Depends on:

Bug Blocks:

333678

Attachments:

Description	Flags
hope this helps	none

Description David Williams

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

2011-01-13 14:48:28 EST

Committed to R3_2_maintenance and HEAD for WTP 3.2.3 and WTP 3.3 M5