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

Bug 290450

Summary:	[Forms] TextSegment should call drawText instead of drawString
Product:	[Eclipse Project] Platform	Reporter:	Kevin Barnes <cocoakevin>
Component:	User Assistance	Assignee:	platform-ua-inbox <platform-ua-inbox>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P3	CC:	cgold, remy.suen
Version:	3.6
Target Milestone:	3.6 M3
Hardware:	PC
OS:	Mac OS X
Whiteboard:

Description Kevin Barnes

2009-09-24 15:15:53 EDT

org.eclipse.ui.internal.forms.widgets.TextSegment calls GC.textExtent when measuring strings and drawString to draw them. For consistency, it should call drawText instead of drawString.

Comment 1 Chris Goldthorpe

2009-09-25 13:09:37 EDT

Have you noticed anything drawn incorrectly as a result of this? I'd like to have a way to test the change.

Comment 2 Kevin Barnes

2009-09-25 14:13:45 EDT

No, it's just good practice to match textExtent/drawText, and stringExtent/drawString.

Comment 3 Chris Goldthorpe

2009-10-01 15:03:08 EDT

Fixed in HEAD.

Comment 4 Chris Goldthorpe

2010-04-16 15:57:00 EDT

That fix went into 3.6M3, changing milestone.