[Mesa-dev] [PATCH 11/18] i965: Replace the region in DRIimage with just a BO pointer and stride.

[Kenneth Graunke]

On 04/29/2014 04:34 PM, Eric Anholt wrote:
[snip]
> @@ -558,12 +572,8 @@ intel_dup_image(__DRIimage *orig_image, void *loaderPrivate)
>     if (image == NULL)
>        return NULL;
>  
> -   intel_region_reference(&image->region, orig_image->region);
> -   if (image->region == NULL) {
> -      free(image);
> -      return NULL;
> -   }
> -
> +   drm_intel_bo_reference(orig_image->bo);
> +   image->bo              = orig_image->bo

I noticed you dropped this NULL check, rather than checking image->bo !=
NULL.  Presumably in intel_dup_image, you know that the orig_image BO
actually exists?  If so, this seems fine.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140430/fe146c5f/attachment.sig>