Yeah, agree reviewing them on the mesa list is probably more appropriate.
Anyway patch lgtm as well and is Reviewed-by: Christian König
<[email protected]>
Regards,
Christian.
On 15.12.2015 18:53, Julien Isorce wrote:
+ mesa-dev of course :)
Thx Sean
On 15 December 2015 at 17:46, Sean V Kelley <[email protected]
<mailto:[email protected]>> wrote:
> On Dec 15, 2015, at 1:01 AM, Julien Isorce
<[email protected] <mailto:[email protected]>> wrote:
>
> "image" is not ready yet since it will be set at
> the end of the function by: *image = *img;
>
> Signed-off-by: Julien Isorce <[email protected]
<mailto:[email protected]>>
> ---
> src/gallium/state_trackers/va/image.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
lgtm but you should also add mesa list for this particular patch…
Sean
>
> diff --git a/src/gallium/state_trackers/va/image.c
b/src/gallium/state_trackers/va/image.c
> index 58c9ff7..ccc263f 100644
> --- a/src/gallium/state_trackers/va/image.c
> +++ b/src/gallium/state_trackers/va/image.c
> @@ -264,7 +264,7 @@ vlVaDeriveImage(VADriverContextP ctx,
VASurfaceID surface, VAImage *image)
> img->image_id = handle_table_add(drv->htab, img);
>
> img_buf->type = VAImageBufferType;
> - img_buf->size = image->data_size;
> + img_buf->size = img->data_size;
> img_buf->num_elements = 1;
>
> pipe_resource_reference(&img_buf->derived_surface.resource,
surfaces[0]->texture);
> --
> 1.9.1
>
> _______________________________________________
> Libva mailing list
> [email protected] <mailto:[email protected]>
> http://lists.freedesktop.org/mailman/listinfo/libva
_______________________________________________
mesa-dev mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
