-
Chun-wei Fan authored
On Windows, when IME is used, each keystroke results in the WM_IME_COMPOSITION event being sent first. This means that in our case when one decides on to accept the input that is in the preedit buffer, we first get from Windows the WM_IME_COMPOSITION event (where we emit the commit signal), followed by the WM_IME_ENDCOMPOSITION event (where we emit the pair of preedit-changed and preedit-end signals). Since commit f11f989f (GtkEntry: Remove recompute idle), we do the input recomputation directly, this will cause a pair of "Pango-WARNING: Assertion failed: (index >= 0 && index <= layout->length)" being shown, as gtkentry.c's priv->preedit_length and priv->preedit_cursor was unable to be reset to 0 in time as a result of the recomputation triggered by the commit being done before the reset of priv->preedit_length and priv->preedit_cursor (which are no longer valid as we essentially say that we are done with the preedit buffer). As we could only acquire the final string that was entered in this preedit session when we handle the WM_IME_COMPOSITION event, fix this by saving up the final string we acquire from Windows IME in UTF-8 when we handle the WM_IME_COMPOSITION event from Windows, and emit the commit signal with that string after we emit the preedit-changed and preedit-end signals when we handle the WM_IME_ENDCOMPOSITION event from Windows, which comes afterwards. Also fix the formatting of the code around the parts of the files that was changed. https://bugzilla.gnome.org/show_bug.cgi?id=787142
c255ba68