On Tue, Jun 09, 2015 at 03:00:26PM -0700, Jon A. Cruz wrote: > Instead of a full strcmp against an empty string, could we just check > the first char? > > if (!text_backend->input_method.path[0])
does this really gain us anything worthwhile? Cheers, Peter > On 06/09/2015 01:28 PM, Murray Calavera wrote: > > This allows a user to explicitly disable the input > > method by setting path to blank; > > > > Signed-off-by: Murray Calavera <[email protected]> > > --- > > src/text-backend.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/src/text-backend.c b/src/text-backend.c > > index 64a3c1b..3c349d4 100644 > > --- a/src/text-backend.c > > +++ b/src/text-backend.c > > @@ -957,6 +957,9 @@ launch_input_method(struct text_backend *text_backend) > > if (!text_backend->input_method.path) > > return; > > > > + if (strcmp(text_backend->input_method.path, "") == 0) > > + return; > > + > > if (text_backend->input_method.process.pid != 0) > > return; > > > > > _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
