On Mon Jun 9 20:46:15 2025 +0000, Nikolay Sivov wrote:
As unfortunate as it looks, it might be close to what we need. However there are two problems with it. First, easy to fix, you're accessing potentially freed "infoPtr". Second is a bit worse - all notifications go through this helper, so we can't really use control data after returning from it because it could have been destroyed. Returning TRUE from TVN_ENDLABELEDITW for example will go through "save text" path, with newly added check too.
I definitely need to fix the bad infoPtr, which should be easy. But for the other problem, I could be wrong on this, but the TVN_ENDLABELEDITW goes through SendRealNotify and not SendSimpleNotify. Doing this in SendSimpleNotify should be OK and prevent similar problems in other notificatins like NM_DBLCLK. Thanks for the feedback.