diff options
| author | Limin Wang <lance.lmwang@gmail.com> | 2020-06-02 19:18:32 +0800 |
|---|---|---|
| committer | Limin Wang <lance.lmwang@gmail.com> | 2020-06-13 06:59:19 +0800 |
| commit | 2658680df4fc606522e5f65899afb9a98b47d287 (patch) | |
| tree | 7e98041eeb6bd5d144df86380b2265969f11e104 /libavutil/parseutils.c | |
| parent | a408d03ee6eeda98e77301dcdea3bdf40c0d4afc (diff) | |
| download | ffmpeg-2658680df4fc606522e5f65899afb9a98b47d287.tar.gz | |
avutil/internal: remove FF_ALLOCx{_ARRAY}_OR_GOTO macros
These functions have a terrible design, let us fix them before extending
them.
First design mistake: no error code. A helper function for testing
memory allocation failure where AVERROR(ENOMEM) does not appear is
absurd.
Second design mistake: printing a message. Return the error code, let
the caller print the error message.
Third design mistake: hard-coded use of goto.
http://ffmpeg.org/pipermail/ffmpeg-devel/2020-May/262544.html
Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
Diffstat (limited to 'libavutil/parseutils.c')
0 files changed, 0 insertions, 0 deletions