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

Bug 329808

Summary:	Compiler warnings in N20101108-2000 build
Product:	[Eclipse Project] PDE	Reporter:	Olivier Thomann <Olivier_Thomann>
Component:	UI	Assignee:	Curtis Windatt <curtis.windatt.public>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	curtis.windatt.public
Version:	3.7
Target Milestone:	3.7 M4
Hardware:	PC
OS:	Windows 7
Whiteboard:

Description Olivier Thomann

2010-11-09 10:19:04 EST

1. WARNING in /src/org/eclipse/pde/internal/core/util/CoreUtility.java
 (at line 211)
long currentLen = 0;
The value of the local variable currentLen is not used

1. WARNING in /src/org/eclipse/pde/ui/tests/target/TargetDefinitionResolutionTests.java
 (at line 402)
int bin = 0;
The value of the local variable bin is not used
2. WARNING in /src/org/eclipse/pde/ui/tests/target/TargetDefinitionResolutionTests.java
 (at line 453)
int bin = 0;
The value of the local variable bin is not used


1. WARNING in /src/org/eclipse/pde/internal/ui/editor/contentassist/ManifestContentAssistProcessor.java
 (at line 186)
int length = fHeader.length;
The value of the local variable length is not used

Comment 1 Curtis Windatt

2010-11-09 10:41:03 EST

Fixed in HEAD.  Most of the problems I just had to remove the unused variable, but in the test case I added an assert.