diff options
author | Andrew Donnellan <andrew.donnellan@au1.ibm.com> | 2016-04-18 17:05:12 +1000 |
---|---|---|
committer | Samuel Mendoza-Jonas <sam@mendozajonas.com> | 2016-04-20 16:09:29 +1000 |
commit | 9f0f86150e9bf14760e155768dcb7df682257327 (patch) | |
tree | b9b8d375ecee1ff20a06446875b6551561f17fb8 /ui | |
parent | 5c16bba845d94ef86ad6fe6fa211cbc148e77717 (diff) | |
download | talos-petitboot-9f0f86150e9bf14760e155768dcb7df682257327.tar.gz talos-petitboot-9f0f86150e9bf14760e155768dcb7df682257327.zip |
Fix sparse warnings
Fix some sparse warnings by using NULL instead of 0 and static-ifying a
variable not used outside its file.
Signed-off-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Signed-off-by: Samuel Mendoza-Jonas <sam@mendozajonas.com>
Diffstat (limited to 'ui')
-rw-r--r-- | ui/common/timer.c | 2 | ||||
-rw-r--r-- | ui/ncurses/generic-main.c | 2 | ||||
-rw-r--r-- | ui/twin/pbt-scr.c | 4 |
3 files changed, 4 insertions, 4 deletions
diff --git a/ui/common/timer.c b/ui/common/timer.c index 3d80898..b0cae0d 100644 --- a/ui/common/timer.c +++ b/ui/common/timer.c @@ -49,7 +49,7 @@ static int timer_cb(void *arg) struct ui_timer *timer = arg; timer->handle_timeout(timer); - timer->waiter = 0; + timer->waiter = NULL; return 0; } diff --git a/ui/ncurses/generic-main.c b/ui/ncurses/generic-main.c index 669053c..27744d4 100644 --- a/ui/ncurses/generic-main.c +++ b/ui/ncurses/generic-main.c @@ -149,7 +149,7 @@ static char *default_log_filename(void) return name; } -struct cui *cui; +static struct cui *cui; /* * struct pb_cui - Main cui program instance. diff --git a/ui/twin/pbt-scr.c b/ui/twin/pbt-scr.c index b774f19..e10cab1 100644 --- a/ui/twin/pbt-scr.c +++ b/ui/twin/pbt-scr.c @@ -402,8 +402,8 @@ struct pbt_scr *pbt_scr_init(void *talloc_ctx, #if !defined(HAVE_LIBTWIN_TWIN_X11_H) assert(0); #else - scr->twin_ctx.x11 = twin_x11_create_ext(XOpenDisplay(0), width, - height, 0); + scr->twin_ctx.x11 = twin_x11_create_ext(XOpenDisplay(NULL), + width, height, 0); if (!scr->twin_ctx.x11) { pb_log("%s: twin_x11_create_ext failed.\n", __func__); |