WineHQ Bugzilla – Bug 29140 – ScriptTextOut passes wrong widths to ExtTextOutW

Bugzilla

Last modified: 2014-01-17 13:58:53 UTC

ScriptTextOut passes wrong widths to ExtTextOutW

Bug 29140 - ScriptTextOut passes wrong widths to ExtTextOutW


Summary:	ScriptTextOut passes wrong widths to ExtTextOutW

Status:	CLOSED FIXED

AppDB:	Show Apps affected by this bug
Product:	Wine
Classification:	Unclassified
Component:	usp10
Version:	1.3.33
Hardware:	Other other

Importance:	P2 minor
Target Milestone:	---
Assigned To:	Mr. Bugs

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree

Reported:	2011-11-21 12:16 UTC by disposable593-wine
Modified:	2014-01-17 13:58 UTC (History)
CC List:	0 users

See Also:
Regression SHA1:
Fixed by SHA1:	b65d0b96ce31c0e9f1cfbbe34fa4c4a822e8d74f
Distribution:	---
Staged patchset:

Attachments
Add an attachment (back traces, logs, proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description disposable593-wine 2011-11-21 12:16:46 UTC

in http://source.winehq.org/source/dlls/usp10/usp10.c#L2673

ScriptTextOut always passes NULL to ExtTextOutW instead of the proper width arrays specified in piJustify or piAdvance.

Comment 1 Vitaliy Margolen 2011-11-21 19:45:59 UTC

So the problem with this is what exactly? Do you have a specific application that doesn't work because of this? or has some issues?

Comment 2 disposable593-wine 2011-11-21 21:16:57 UTC

(In reply to comment #1)
> So the problem with this is what exactly? Do you have a specific application
> that doesn't work because of this? or has some issues?

Yes, a program I wrote doesn't work because of this. The text appears in wrong places because ScriptTextOut ignores justification. This is annoying because the blinking text cursor shows up in the correct place, which is now somewhere else on the screen. It does work as it should in Windows.

Comment 3 Austin English 2013-12-05 20:22:15 UTC

This should be fixed by b65d0b96ce31c0e9f1cfbbe34fa4c4a822e8d74f, please retest in current (1.7.7 or newer) wine.

Comment 4 Nikolay Sivov 2014-01-06 13:05:20 UTC

Marking fixed, problem was clearly stated in a description.

Comment 5 Alexandre Julliard 2014-01-17 13:58:53 UTC

Closing bugs fixed in 1.7.11.