Re: [libdrm][PATCH 2/2] Fix gcc -Wextra warnings

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

 



On 10/02/15 21:37, Jan Vesely wrote:
> On Tue, 2015-02-10 at 00:27 +0000, Emil Velikov wrote:
>> On 10 February 2015 at 00:02, Jan Vesely <jan.vesely@xxxxxxxxxxx> wrote:
>>> On Mon, 2015-02-09 at 23:32 +0000, Emil Velikov wrote:
>>>> On 9 February 2015 at 21:39, Jan Vesely <jan.vesely@xxxxxxxxxxx> wrote:
>>>>> Signed-off-by: Jan Vesely <jan.vesely@xxxxxxxxxxx>
>>>> Nice one Jan. I've sent similar fixes for drmOpenDevice and
>>>> drmGetStats a few days ago.
>>>>
>>>> Considering you drop the last hunk that Ian spotted both patches are
>>>> Reviewed-by: Emil Velikov <emil.l.velikov@xxxxxxxxx>
>>>
>>> Thanks, I sent v2 of that patch few minutes ago.
>>>
>>> I think your 4/6 and 5/6 overlap with this one. Should I go ahead or do
>>> you plan to push yours?
>>>
>> I would go with your series - it handles more cases, plus already has
>> move reviews :-P
>> If you feel like looking through some of my series that would be appreciated.
> 
> I wasn't subscribed to the list so I can't reply to those emails (don't
> know the message-ids).
> I looked at the series from Jan 29th [0]. 
> 
> 1/6[1], there is no tests/util directory, I guess it depends on
> Thierry's series? since it hasn't landed yet does it make sense to
> squash it there (like your 04.1/11 SQUASH: tests: misc cleanups) ?
> 
> 2/6[2], also does not apply cleanly (needs Thierry's 5/11), if you want
> to push a version rebased on master you can add
> 
> Reviewed-by: Jan Vesely <jan.vesely@xxxxxxxxxxx>
> to that one
> 
I'll these on hold and revive as Theirry's series lands.

> 4/6 and 5/6 were superseded, and I don't know enough about android to
> look at the other two, but
> 
> 6/6 looks trivial enough
> Acked-by: Jan Vesely <jan.vesely@xxxxxxxxxxx>
> 
> with a small nit:
> Why keep two assignments to LOCAL_SHARED_LIBRARIES in intel/Android.mk ?
> 
Good catch. I'm assuming that (a) either I messed up at cherry-picking
the patch or (b) git got confused as their three does not have
libpciaccess in the list.

Thanks
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