ivi-shell: ivi-shell.c, You could get rid of one level of indenting

            wl_resource_post_error(resource,
            IVI_APPLICATION_ERROR_ROLE,
            "wl_surface is invalid");
            return;
            }

    That's nice and clear: one can immediately read from each branch
what went wrong if execution ends up there.

    You could get rid of one level of indenting but moving the if
(!weston_surface) check to the top, since it ends in return. No need for
an else block then.

Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA@xddp.denso.co.jp>