[FFmpeg-devel] [PATCH 1/2] lavu: add AV_DYNARRAY_ADD() macro.

[Michael Niedermayer]

On Wed, Mar 12, 2014 at 11:44:41PM +0100, Lukasz Marek wrote:
> On 08.03.2014 21:26, Nicolas George wrote:
> >TODO minor bump and APIchanges entry

left for the addition of av_dynarray_add_nofree() so that we dont
need to bump twice and it gives a bit extra time to make changes
to the macro if needed ...

[...]
> >+        if (av_size_new) { \
> >+            { av_success } \
> >+            (av_size)++; \
> >+        } else { \
> >+            { av_failure } \
> 
> braces around av_failure seems redundant

changed


> 
> >+        } \
> >+    } while (0)
> >+
> >+#endif /* AVUTIL_DYNARRAY_H */
> >
> 
> Seems OK

applied as there are dependant patches on this

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140322/6c49501d/attachment.asc>