Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 365985 - Compile warning in official build
Summary: Compile warning in official build
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.8   Edit
Hardware: All All
: P3 minor (vote)
Target Milestone: 3.8 M5   Edit
Assignee: Dani Megert CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-08 02:13 EST by Dani Megert CLA
Modified: 2011-12-14 11:22 EST (History)
3 users (show)

See Also:
Michael_Rennie: review+

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 Dani Megert CLA 2011-12-08 02:13:13 EST 
20111207-2118.

1. WARNING in /src/org/eclipe/debug/tests/viewer/model/PerformanceTests.java
 (at line 269)
int numElements = (int)Math.pow(2, getTestModelDepth());
The value of the local variable numElements is not used
Comment 1 Pawel Piech CLA 2011-12-09 19:09:16 EST 
It was a left-over from the integration testing.  Committed fix to master: http://git.eclipse.org/c/platform/eclipse.platform.debug.git/commit/?id=cc853b9b874eccb186d807a16cf173c90b0ce77b

Thanks
Comment 2 Dani Megert CLA 2011-12-12 05:57:45 EST 
Verified in N20111211-2000.
Comment 3 Dani Megert CLA 2011-12-14 08:27:03 EST 
Mike or Pawel: it looks like this bug is in the I-build. Did you forget to do the build input for Debug?
Comment 4 Pawel Piech CLA 2011-12-14 09:30:04 EST 
(In reply to comment #3)
> Mike or Pawel: it looks like this bug is in the I-build. Did you forget to do
> the build input for Debug?

Yes I'm afraid so.  I hadn't seen a notification from Mike but I didn't follow up.  Sorry about that.
Comment 5 Michael Rennie CLA 2011-12-14 11:22:03 EST 
(In reply to comment #3)
> Mike or Pawel: it looks like this bug is in the I-build. Did you forget to do
> the build input for Debug?

Yup, this was my fault, I was out of the office yesterday afternoon and forgot to do it last night.