]> git.ipfire.org Git - thirdparty/ipxe.git/commitdiff
[mucurses] Fix GCC 6 nonnull-compare errors
authorVinson Lee <vlee@freedesktop.org>
Fri, 15 Apr 2016 22:38:35 +0000 (22:38 +0000)
committerMichael Brown <mcb30@ipxe.org>
Wed, 4 May 2016 15:01:32 +0000 (16:01 +0100)
Remove null checks for arguments declared as nonnull.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Signed-off-by: Michael Brown <mcb30@ipxe.org>
src/hci/mucurses/windows.c

index 7f39bdea2b7da4c77132b25d1468f0f0e6878af1..5f5d1f4e2a16801d4b24bae4add2443410080d9c 100644 (file)
@@ -18,9 +18,6 @@ FILE_LICENCE ( GPL2_OR_LATER_OR_UBDL );
  * @ret rc     return status code
  */
 int delwin ( WINDOW *win ) {
-       if ( win == NULL )
-               return ERR;
-
        /* I think we should blank the region covered by the window -
           ncurses doesn't do this, but they have a buffer, so they
           may just be deleting from an offscreen context whereas we
@@ -51,8 +48,6 @@ int delwin ( WINDOW *win ) {
 WINDOW *derwin ( WINDOW *parent, int nlines, int ncols,
                                 int begin_y, int begin_x ) {
        WINDOW *child;
-       if ( parent == NULL )
-               return NULL;
        if ( ( child = malloc( sizeof( WINDOW ) ) ) == NULL )
                return NULL;
        if ( ( (unsigned)ncols > parent->width ) || 
@@ -75,8 +70,6 @@ WINDOW *derwin ( WINDOW *parent, int nlines, int ncols,
  */
 WINDOW *dupwin ( WINDOW *orig ) {
        WINDOW *copy;
-       if ( orig == NULL )
-               return NULL;
        if ( ( copy = malloc( sizeof( WINDOW ) ) ) == NULL )
                return NULL;
        copy->scr = orig->scr;
@@ -99,8 +92,6 @@ WINDOW *dupwin ( WINDOW *orig ) {
  * @ret rc     return status code
  */
 int mvwin ( WINDOW *win, int y, int x ) {
-       if ( win == NULL )
-               return ERR;
        if ( ( ( (unsigned)y + win->height ) > LINES ) ||
             ( ( (unsigned)x + win->width ) > COLS ) )
                return ERR;
@@ -149,8 +140,6 @@ WINDOW *newwin ( int nlines, int ncols, int begin_y, int begin_x ) {
 WINDOW *subwin ( WINDOW *parent, int nlines, int ncols,
                                 int begin_y, int begin_x ) {
        WINDOW *child;
-       if ( parent == NULL )
-               return NULL;
        if ( ( child = malloc( sizeof( WINDOW ) ) ) == NULL )
                return NULL;
        child = newwin( nlines, ncols, begin_y, begin_x );