On 9/6/2012 1:25 PM, Denis S. Fokin wrote:
Hi Alexander,
should not we use SPIF_SENDCHANGE instead of SPIF_UPDATEINIFILE? It is
possible that the user does not want to disable beep in his profile.
On the other hand, I do not see any issues with SPIF_SENDCHANGE.
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/7194469/webrev.01/
The MSDN doc says that the SPIF_SENDCHANGE broadcasts a
message after updating the user profile. Probably we do not want to
notify all that we disable and then enable the beeper for the RICHEDIT
control:
http://msdn.microsoft.com/en-us/library/windows/desktop/ms724947%28v=vs.85%29.aspx
It allso says that this parameter can be zero if you do not
want to update the user profile or broadcast a message.
So the fix sets the fWinIni parameter to 0 in the
SystemParametersInfo method.
Thanks,
Alexandr.
Thank you,
Denis.
On 9/5/2012 5:24 PM, Alexander Scherbatiy wrote:
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7194469
webrev: http://cr.openjdk.java.net/~alexsch/7194469/webrev.00/
The TextField control starts beeping on some keys pressing (enter, up,
down...) after switching the EDIT control to RICHEDIT.
It seems that there are no flags that can disable this beeping for the
RICHEDIT.
At least SES_ALLOWBEEPS flag from EM_GETEDITSTYLE is set to zero by
default but beeping is present.
The fix disables the beeper (in case if it was enabled) before the
necessary keys processing in the AWT TextField and enable it after that.
Thanks,
Alexandr.
