Dmitry Timoshkov dmitry@codeweavers.com writes:
Austin Lund austin.lund@gmail.com wrote:
I'm not sure how that's relevant because if the messages aren't passed then as long as a WM_NCCREATE is passed then the editor variable is initialised by this.
What happens if an application doesn't pass WM_NCCREATE?
It would be legitimate for a window to not be properly initialized in that case. Richedit may or may not have a workaround for this, but it's not a requirement of the API.
I'm reading this comment carefully for a suggestion as to how to "fix" things, but it seems totally obfuscated.
Editor should be unconditionally initialized if it's NULL, and WM_NCDESTROY should be made an explicit case in the 'switch' statement.
Of course with appropriate test cases demonstrating that this is actually the correct behavior...