On Mon, Sep 26, 2011 at 10:41:49PM -0400, Matt Turner wrote:
> --- a/src/mga_video.c
> +++ b/src/mga_video.c
> @@ -131,8 +131,7 @@ void MGAInitVideo(ScreenPtr pScreen)
>           num_adaptors = 1;
>           adaptors = &newAdaptor;
>       } else {
> -         newAdaptors =  /* need to free this someplace */
> -             xalloc((num_adaptors + 1) * sizeof(XF86VideoAdaptorPtr*));
> +         newAdaptors = malloc((num_adaptors + 1) * 
> sizeof(XF86VideoAdaptorPtr *));

Why did you delete this comment?

Otherwise,

Reviewed-by: Jamey Sharp <[email protected]>

If you'd produce another patch to delete all instances of casting the
return value of malloc/calloc/realloc, that would be awesome. We have
(void *) now...

Jamey

Attachment: signature.asc
Description: Digital signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to