Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316352 - [CSS] CTabRendering throws NPE on context dispose
Summary: [CSS] CTabRendering throws NPE on context dispose
Status: RESOLVED WORKSFORME
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Bogdan Gheorghe CLA
QA Contact: Bogdan Gheorghe CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-09 14:23 EDT by Paul Webster CLA
Modified: 2011-08-16 17:05 EDT (History)
3 users (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 Paul Webster CLA 2010-06-09 14:23:31 EDT 
@Inject
@Optional
public void setCornerRadius(@Named("radius") Integer radius) {
	cornerSize = radius.intValue();
	parent.redraw();
}

These methods say they're optional (and can take null) but NPE.

PW
Comment 1 Oleg Besedin CLA 2010-06-10 10:01:53 EDT 
On context disposal we call with injected values == null.
Comment 2 Thomas Schindl CLA 2011-08-15 14:28:11 EDT 
I guess this is fixed because we are not facing any NPEs, right? So can we close this one?
Comment 3 Bogdan Gheorghe CLA 2011-08-15 22:34:28 EDT 
Well, I should add a null check and then close it.
Comment 4 Bogdan Gheorghe CLA 2011-08-16 17:05:55 EDT 
Closing this as we are no longer using injection to set this property.