On 31/08/2016 03:51, Yong Bakos wrote:
On Aug 30, 2016, at 6:38 PM, Bryce Harrington <br...@osg.samsung.com> wrote:

Signed-off-by: Bryce Harrington <br...@osg.samsung.com>

This is exactly what the macro expresses, so this is

Reviewed-by: Yong Bakos <yba...@humanoriented.com>

Thanks, pushed:
b0e4abd..c6ae812  master -> master

Cheers,



---
clients/weston-simple-im.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/clients/weston-simple-im.c b/clients/weston-simple-im.c
index 2d23930..1663d84 100644
--- a/clients/weston-simple-im.c
+++ b/clients/weston-simple-im.c
@@ -30,11 +30,11 @@
#include <unistd.h>
#include <sys/mman.h>

-
#include <linux/input.h>

#include "window.h"
#include "input-method-unstable-v1-client-protocol.h"
+#include "shared/helpers.h"

enum compose_state {
        state_normal,
@@ -398,7 +398,7 @@ simple_im_key_handler(struct simple_im *keyboard,
                if (state == WL_KEYBOARD_KEY_STATE_PRESSED)
                        return;

-               for (i = 0; i < sizeof(ignore_keys_on_compose) / 
sizeof(ignore_keys_on_compose[0]); i++) {
+               for (i = 0; i < ARRAY_LENGTH(ignore_keys_on_compose); i++) {
                        if (sym == ignore_keys_on_compose[i]) {
                                zwp_input_method_context_v1_key(context,
                                                                
keyboard->serial,
@@ -414,7 +414,7 @@ simple_im_key_handler(struct simple_im *keyboard,
                keyboard->compose_seq.keys[i] = sym;

                cs = bsearch (&keyboard->compose_seq, compose_seqs,
-                             sizeof(compose_seqs) / sizeof(compose_seqs[0]),
+                             ARRAY_LENGTH(compose_seqs),
                              sizeof(compose_seqs[0]), compare_compose_keys);

                if (cs) {
--
1.9.1

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel



--

Quentin “Sardem FF7” Glidic
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to