[FFmpeg-devel] [PATCH 1/2] libavutil/libavfilter: add opencl wrapper to ffmpeg

[Stefano Sabatini]

On date Sunday 2013-03-24 18:36:37 +0800, Wei Gao encoded:
> 2013/3/24 Stefano Sabatini <stefasab at gmail.com>
[...]
> OK, I will fix them and re-submit, one more question, the following comment
> you give me, I didn't get what you mean,sorry
> 
> > +    if (status != CL_SUCCESS) {
> > +        av_log(&openclutils,AV_LOG_ERROR,"generate_bin_from_kernel_source
> error,clGetProgramInfo:%s\n",opencl_errstr(status));
> > +        return AVERROR_EXTERNAL;
> > +    }
> > +    devices = av_mallocz(sizeof(cl_device_id) * numdevices);
> > +    if (!devices)
> > +        return AVERROR(ENOMEM);
> > +    /* grab the handles to all of the devices in the program. */
> > +    status = clGetProgramInfo(program,
> > +                              CL_PROGRAM_DEVICES,
> > +                              sizeof(cl_device_id) * numdevices,
> > +                              devices,
> > +                              NULL);
> 
> this code could be possibly be factorized

Discard the comment, not really important.
-- 
FFmpeg = Fierce and Foolish Magical Perfectionist Epic Gnome