On Thu, Jul 3, 2008 at 2:01 PM, Juan Lang <<a href="mailto:juan.lang@gmail.com">juan.lang@gmail.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Dylan,<br>
<br>
- if (!editor->bEmulateVersion10 || (editor->nEventMask & ENM_UPDATE))<br>
+ if (!editor->dwEmulatedVersion || (editor->nEventMask & ENM_UPDATE))<br>
<br>
This change is incorrect, as dwEmulatedVersion is never set to 0. I<br>
believe you mean if (editor->dwEmulatedVersion > 0x100 || ...<br>
</blockquote><div><br>You're right, that was a mistake. Thanks for catching it.<br>
<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I have to ask, though: what bug does this fix?<font color="#888888"><br>
</font></blockquote></div><br>This doesn't fix a bug, but it allows me to add functionality for
msftedit.dll without changing the way riched20.dll works when loaded
directly. Without storing more than a boolean value it will be hard to support the multiple versions of richedit controls.<br>
<br>
I would like to properly implement tables for richedit controls, which greatly differs in how it is implemented between riched20.dll and msftedit.dll. Should I only worry about how it is done in the newer richedit controls? I am worried that this will cause problems considering that amy program that loads msftedit.dll assumes that they are getting a specific implementation.<br>