From: Gabriel Ivăncescu Subject: Re: [PATCH v2 5/8] shell32/autocomplete: Send messages directly to the edit control's procedure Message-Id: Date: Fri, 21 Sep 2018 14:45:28 +0300 In-Reply-To: <20180921113841.GR3601@merlot.physics.ox.ac.uk> References: <4416482679d9bcb58a14dcd706aa40bec582bbe3.1537444354.git.gabrielopcode@gmail.com> <20180921105531.GP3601@merlot.physics.ox.ac.uk> <20180921113841.GR3601@merlot.physics.ox.ac.uk> On Fri, Sep 21, 2018 at 2:38 PM, Huw Davies wrote: > > Well firstly, I very much doubt the overhead is going to be relevant > for a user input driven auto-complete. > > We don't necessarily want to copy Windows' implementation exactly--- > indeed if it's possible we try to avoid that. As long as apps don't > notice the difference it's fine. > > Just make the changes that you need to add WM_SETTEXT. > > Huw. I think we should avoid wineserver calls where possible, since it disrupts the wineserver also not just the specific application in question (in terms of overhead / blocking). That said, if you don't want Windows behavior, I can get rid of get_edit_proc totally, and just replace SendMessage with CallWindowProc directly, which would still avoid wineserver calls here, and not add anything to the code than just replace the calls. Well of course unless you really want to leave them as SendMessage but I think it's better to avoid wineserver calls if we can.