[FFmpeg-devel] [PATCH] lavu/opencl: apply misc cosmetics fixes
Stefano Sabatini
stefasab at gmail.com
Mon Apr 1 23:57:47 CEST 2013
On date Monday 2013-04-01 17:02:31 -0400, Derek Buitenhuis encoded:
> On 2013-04-01 7:55 AM, Stefano Sabatini wrote:
> > Split long lines, reindent code, add missing spaces between operators,
> > remove possibly confusing comment from av_opencl_buffer_read_image()
> > doxy.
> > ---
> > libavutil/opencl.c | 237 ++++++++++++++++++++++++++++------------------------
> > libavutil/opencl.h | 38 +++++----
> > 2 files changed, 150 insertions(+), 125 deletions(-)
>
> [...]
>
>
> > -static const AVClass openclutils_class = {"OPENCLUTILS", av_default_item_name,
> > - NULL, LIBAVUTIL_VERSION_INT,
> > - offsetof(OpenclUtils, log_offset),
> > - offsetof(OpenclUtils, log_ctx)};
> > +static const AVClass openclutils_class = {
> > + "OPENCLUTILS", av_default_item_name,
> > + NULL, LIBAVUTIL_VERSION_INT,
> > + offsetof(OpenclUtils, log_offset),
> > + offsetof(OpenclUtils, log_ctx)
> > +};
>
> One per line is probably more readable, and using designated initializers even more so.
>
> Rest OK.
Changed and pushed, thanks.
--
FFmpeg = Furious Faithful Mythic Political Ecumenical Guru
More information about the ffmpeg-devel
mailing list