[Bug 14281] New: Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

Summary: Richedit's EM_SETTEXTEX handler doesn't notice \urtf as
                    valid RTF sequence
           Product: Wine
           Version: 1.1.0
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: richedit
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Per http://source.winehq.org/source/dlls/riched20/editor.c#L2184 (links to
Richedit's EM_SETTEXTEEX handler), it doesn't check for \urtf as specified by
http://msdn.microsoft.com/en-us/library/bb774284(VS.85).aspx ("If lParam starts
with a valid Rich Text Format (RTF) ASCII sequence—for example, "{\rtf" or
"{urtf"—the text is read in using the RTF reader"). I have verified that
Windows XP SP2 behaves as specified by MSDN.

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

Dylan Smith  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|[e-mail]        |[e-mail]
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1

--- Comment #1 from Dylan Smith   2008-10-03 03:08:22 ---
Created an attachment (id=16442)
 --> (http://bugs.winehq.org/attachment.cgi?id=16442)
richedit: Use RTF reader for text starting with {\urtf.

I wrote a test to confirm and fixed the bug.

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

Dylan Smith  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

--- Comment #2 from Dylan Smith   2008-10-06 16:10:24 ---
My patch is in the git tree if you want to test it.

commit bdf181b4d3621e4a392304bcd6b505bd15bcc0bf is the one that I think fixes
the bug.

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

--- Comment #3 from [e-mail]  2008-10-08 19:10:43 ---
(In reply to comment #2)
> My patch is in the git tree if you want to test it.
>
> commit bdf181b4d3621e4a392304bcd6b505bd15bcc0bf is the one that I think fixes
> the bug.

I can confirm that this patch fixes this bug for me as well.

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

James Hawkins  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from James Hawkins   2008-10-08 19:13:18 ---
Reported fixed.

[Bug 14281] Richedit's EM_SETTEXTEX handler doesn't notice \urtf as valid RTF sequence

http://bugs.winehq.org/show_bug.cgi?id=14281

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #5 from Alexandre Julliard   2008-10-10 10:09:59 ---
Closing bugs fixed in 1.1.6.