On Thu, Nov 17, 2016 at 09:43:05PM +0100, Silvan Jegen wrote: > Signed-off-by: Silvan Jegen <[email protected]> > --- > clients/editor.c | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/clients/editor.c b/clients/editor.c > index 6805d8a..1ed3eec 100644 > --- a/clients/editor.c > +++ b/clients/editor.c > @@ -944,16 +944,10 @@ text_entry_reset_preedit(struct text_entry *entry) > static void > text_entry_commit_and_reset(struct text_entry *entry) > { > - char *commit = NULL; > - > if (entry->preedit.commit) > - commit = strdup(entry->preedit.commit); > + text_entry_insert_at_cursor(entry, entry->preedit.commit, 0, 0); > > text_entry_reset_preedit(entry); > - if (commit) { > - text_entry_insert_at_cursor(entry, commit, 0, 0); > - free(commit); > - }
This essentially swaps the order of text_entry_reset_preedit() and text_entry_insert_at_cursor(). Does this cause any side effects? text_entry_insert_at_cursor() calls text_entry_update_layout(), which will behave differently if there is a preedit in effect with and without this patch. I'm not super familiar with this code, but on a quick skim it looks like with this patch applied, any existing pre-edits will be applied and finalized before the reset, whereas previously the pre-edits would be discarded? Am I interpreting this correctly? If this is the case, and if that change of behavior is desireable, make sure the behavioral change (and rationale for why it's being done) is documented in the changelog. If I'm not interpreting it correctly, accept my apologies and I look forward to your elucidation (which probably also would be worth referencing in the changelog entry). Thanks, Bryce > zwp_text_input_v1_reset(entry->text_input); > text_entry_update(entry); > -- > 2.10.2 > > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
