Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 329814 - Compiler warnings in N20101108-2000 build
Summary: Compiler warnings in N20101108-2000 build
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7 M4   Edit
Assignee: Bogdan Gheorghe CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-09 10:24 EST by Olivier Thomann CLA
Modified: 2010-11-12 11:56 EST (History)
1 user (show)

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 Olivier Thomann CLA 2010-11-09 10:24:29 EST 
1. WARNING in /temp.folder/@dot.src/org/eclipse/swt/internal/image/JPEGDecoder.java
 (at line 2303)
int ci, access_rows;
The value of the local variable access_rows is not used
2. WARNING in /temp.folder/@dot.src/org/eclipse/swt/internal/image/JPEGDecoder.java
 (at line 3824)
int ci, block_row, block_rows, access_rows;
The value of the local variable access_rows is not used

1. WARNING in /temp.folder/@dot.src/org/eclipse/swt/internal/theme/ComboDrawData.java
 (at line 50)
int y_border = ythickness + focus_padding;
The value of the local variable y_border is not used
2. WARNING in /temp.folder/@dot.src/org/eclipse/swt/internal/theme/ComboDrawData.java
 (at line 112)
int y_border = ythickness + focus_padding;
The value of the local variable y_border is not used

1. WARNING in _64/temp.folder/@dot.src/org/eclipse/swt/graphics/Cursor.java
 (at line 378)
for (int i = 3, ap=0; i < srcData.length; i+=4, ap++) {
The value of the local variable ap is not used

All platforms should be validated.
Comment 1 Bogdan Gheorghe CLA 2010-11-09 17:27:35 EST 
Corrected most warnings for tonight's build; there is 1 more that I need to check with Grant tomorrow.
Comment 2 Bogdan Gheorghe CLA 2010-11-10 10:39:55 EST 
Fixed in HEAD > 20101110
Comment 3 Olivier Thomann CLA 2010-11-11 10:01:57 EST 
N20101110-2000 still contains a warning for some swt fragments:

/plugins/org.eclipse.swt.gtk.aix.ppc64_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.aix.ppc_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.linux.ppc64_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.linux.s390_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.linux.s390x_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.linux.x86_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.linux.x86_64_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.solaris.sparc_3.7.0.N20101110-2000
/plugins/org.eclipse.swt.gtk.solaris.x86_3.7.0.N20101110-2000
Comment 4 Bogdan Gheorghe CLA 2010-11-11 13:08:12 EST 
Hmm, not sure how that didn't get committed. It was one line. Trying again...

Fixed in HEAD > 20101111
Comment 5 Olivier Thomann CLA 2010-11-12 11:56:06 EST 
Verified in N20101111-2000.