re: drm: Remove useless else block

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dave,

There is a bad merge in drm-next and it causes a static checker warning.

The patch 2ca651d15aff: "drm: Remove useless else block" from Dec 10,
2014, leads to the following static checker warning:

	drivers/gpu/drm/drm_crtc.c:4380 drm_property_change_valid_get()
	info: ignoring unreachable code.

drivers/gpu/drm/drm_crtc.c
  4371                  }
  4372          } else {
  4373                  int i;
  4374                  for (i = 0; i < property->num_values; i++)
  4375                          if (property->values[i] == value)
  4376                                  return true;
  4377                  return false;
  4378          }
  4379  
  4380          for (i = 0; i < property->num_values; i++)
  4381                  if (property->values[i] == value)
  4382                          return true;
  4383          return false;

This block was in the else statement before, but now it's moved to here
as well.

  4384  }

regards,
dan carpenter
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux