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

Bug 493722

Summary: GC#setLineAttributes does not apply its argument to GC
Product: [Eclipse Project] Platform Reporter: Peter Severin <peter>
Component: SWTAssignee: Alexander Kurtakov <akurtakov>
Status: VERIFIED FIXED QA Contact:
Severity: major    
Priority: P3 CC: akurtakov, ansgar.radermacher, arunkumar.thondapu, markus.kell.r, niraj.modi
Version: 4.6Flags: niraj.modi: review+
arunkumar.thondapu: review+
Target Milestone: 4.6 RC2   
Hardware: PC   
OS: Linux   
See Also: https://git.eclipse.org/r/72800
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=fe4f8bb968ea3bc87cd534a5996cb18aa246f3b0
Whiteboard:
Bug Depends on:    
Bug Blocks: 479614, 493882    

Description Peter Severin CLA 2016-05-16 03:29:11 EDT
Tested with Eclipse 4.6 RC1. The code in question looks like this:

public void setLineAttributes(LineAttributes attributes) {
	if (handle == 0) SWT.error(SWT.ERROR_GRAPHIC_DISPOSED);
	if (attributes == null) SWT.error(SWT.ERROR_NULL_ARGUMENT);
	attributes.width = DPIUtil.autoScaleUp(attributes.width);
}

This method should be calling as its last op the setLineAttributesInPixels method that was introduced with HiDPI support.
Comment 1 Eclipse Genie CLA 2016-05-16 04:17:44 EDT
New Gerrit change created: https://git.eclipse.org/r/72800
Comment 2 Niraj Modi CLA 2016-05-16 04:49:04 EDT
Thanks Alex for taking care of this, patch looks good.
Comment 4 Alexander Nyßen CLA 2016-05-23 12:39:48 EDT
*** Bug 494321 has been marked as a duplicate of this bug. ***
Comment 5 Markus Keller CLA 2016-05-30 10:58:29 EDT
Verified in 4.6.0.I20160529-2000.