[FFmpeg-devel] [PATCH 1/3] Improve selftest coverage for libavutil library
Michael Niedermayer
michael at niedermayer.cc
Wed Oct 12 21:38:25 EEST 2016
On Mon, Oct 10, 2016 at 08:56:51PM -0700, Thomas Turner wrote:
> Code coverage for av_fifo_generic_peek(...) and av_fifo_grow(...) .
> ---
> libavutil/tests/fifo2.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 77 insertions(+)
> create mode 100644 libavutil/tests/fifo2.c
>
> diff --git a/libavutil/tests/fifo2.c b/libavutil/tests/fifo2.c
> new file mode 100644
> index 0000000..923feee
> --- /dev/null
> +++ b/libavutil/tests/fifo2.c
We generally put all tests for "something" into the same file
these could be put into fifo.c
[...]
> + data_arr = (int32_t*)malloc(fifo_size);
the cast is unneeded in C
also please dont add code that is changed in the immedeatly next patch
that makes review difficult
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161012/0ab02e94/attachment.sig>
More information about the ffmpeg-devel
mailing list