Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 353523 - SWT_AWT broken on Mac OS X Leopard (10.5.8)
Summary: SWT_AWT broken on Mac OS X Leopard (10.5.8)
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.1   Edit
Hardware: Macintosh Mac OS X
: P3 critical with 2 votes (vote)
Target Milestone: 3.7.1   Edit
Assignee: Lakshmi P Shanmugam CLA
QA Contact: Silenio Quarti CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-01 15:57 EDT by Patrick Talbot CLA
Modified: 2011-11-29 02:50 EST (History)
4 users (show)

See Also:
Silenio_Quarti: review+

Attachments
Simple test case (1.91 KB, text/plain)
2011-08-01 15:58 EDT, Patrick Talbot CLA 		no flags Details
Output of the terminal session during the test, with StackTrace (6.32 KB, text/plain)
2011-08-01 15:58 EDT, Patrick Talbot CLA 		no flags Details
patch (4.31 KB, patch)
2011-08-03 07:11 EDT, Lakshmi P Shanmugam 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 Patrick Talbot CLA 2011-08-01 15:57:37 EDT 
Build Identifier: 3.7 final

NPE at org.eclipse.swt.widgets.Shell.createHandle(Shell.java:735)

Using the simple test case attached, there is a NPE each time one tries to create a new Shell, see the terminal output attached with stackTrace.

OS is Mac OS X 10.5.8
I've tried with:
- Java 6 1.6.0_26 - 64-bit
- Java 5 1.5.0_30 - 64-bit
- Java 5 1.5_0_30 - 32-bit

All with the same result.
Some users reported the same issue with Java 6 1.6.0_24 - always on Leopard 10.5.8

The same code runs fine in Snow Leopard 10.6.8

This is a critical issue for us since it basically breaks Leopard compatibility.

Thanks for having a look.

Reproducible: Always

Steps to Reproduce:
1. Run the sample code attached
2. NPE happends at line 32 - caused by Shell.createHandle(Shell.java:735)
Comment 1 Patrick Talbot CLA 2011-08-01 15:58:15 EDT 
Created attachment 200668 [details]
Simple test case
Comment 2 Patrick Talbot CLA 2011-08-01 15:58:50 EDT 
Created attachment 200669 [details]
Output of the terminal session during the test, with StackTrace
Comment 3 Lakshmi P Shanmugam CLA 2011-08-03 06:47:14 EDT 
Null check is missing
Comment 4 Lakshmi P Shanmugam CLA 2011-08-03 07:11:37 EDT 
Created attachment 200795 [details]
patch

Added null check and removed OS.HIWindowFindAtLocation() since we are not using it now.
Comment 5 Lakshmi P Shanmugam CLA 2011-08-05 04:35:58 EDT 
Thanks Silenio!
Fixed in 3.7.1.
http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?h=R3_7_maintenance&id=8fbd6f7b609d8d48b2b8f9d0d4bc600048b376cb

Will release patch to 3.8 after M1.
Comment 7 Lakshmi P Shanmugam CLA 2011-09-13 08:03:19 EDT 
Verified in I20110912-2126