EM_UNDO message handling for single line edit controls

[Santosh Siddheshwar]

Hi,
  I think there is a defect in the handling of EM_UNDO message for single
line edit controls. If you go by what MSDN says this
message should always return TRUE. But in EDIT_EM_Undo method
(controls/edit.c) we are checking whether the style of
the edit control is Read Only and returning FALSE if it is Read Only. My
question is why should a read only edit control 
not be allowed to handle WM_PASTE, EM_UNDO and similar messages?
Regards
Santosh Siddheshwar
*********************************************************************
Disclaimer: The information in this e-mail and any attachments is
confidential / privileged. It is intended solely for the addressee or
addressees. If you are not the addressee indicated in this message, you may
not copy or deliver this message to anyone. In such case, you should destroy
this message and kindly notify the sender by reply email. Please advise
immediately if you or your employer does not consent to Internet email for
messages of this kind.
*********************************************************************

[Dmitry Timoshkov]

"Santosh Siddheshwar"  wrote:
> I think there is a defect in the handling of EM_UNDO message for single
> line edit controls. If you go by what MSDN says this
> message should always return TRUE. But in EDIT_EM_Undo method
> (controls/edit.c) we are checking whether the style of
> the edit control is Read Only and returning FALSE if it is Read Only. My
> question is why should a read only edit control 
> not be allowed to handle WM_PASTE, EM_UNDO and similar messages?

As with almost every suspicious change in Wine these days
it's better to write a test app (and possibly integrate it
into Wine test suite) and see what Windows does in that case.