Austin Lund austin.lund@gmail.com wrote:
This code is completely broken. An app can decide to not even pass these messages to a subclassed richedit control.
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?
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.