Re: [PATCH libdrm 7/8] Fix unused, and unused-but-set variables warnings

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

 



On 27 February 2015 at 18:07, Jan Vesely <jan.vesely@xxxxxxxxxxx> wrote:
> Signed-off-by: Jan Vesely <jan.vesely@xxxxxxxxxxx>
> ---
>  tests/exynos/exynos_fimg2d_test.c | 4 +---
>  xf86drm.c                         | 9 ++++++---
>  2 files changed, 7 insertions(+), 6 deletions(-)
>
...
> @@ -333,7 +336,6 @@ static int drmOpenDevice(dev_t dev, int minor, int type)
>         drm_server_info->get_perms(&serv_group, &serv_mode);
>         devmode  = serv_mode ? serv_mode : DRM_DEV_MODE;
>         devmode &= ~(S_IXUSR|S_IXGRP|S_IXOTH);
> -       group = (serv_group >= 0) ? serv_group : DRM_DEV_GID;
>      }
>
>  #if !defined(UDEV)
> @@ -354,6 +356,7 @@ static int drmOpenDevice(dev_t dev, int minor, int type)
>      }
>
>      if (drm_server_info) {
> +       group = (serv_group >= 0) ? serv_group : DRM_DEV_GID;
I think you can leave this back where it was. It doesn't seem to bring
anything beneficial.

-Emil
_______________________________________________
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