]> git.sesse.net Git - ffmpeg/commit
avformat/matroskadec: Simplify control flow of parsing laces
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Tue, 3 Dec 2019 17:09:07 +0000 (18:09 +0100)
committerJames Almer <jamrial@gmail.com>
Thu, 5 Dec 2019 02:11:37 +0000 (23:11 -0300)
commita69f92a94638984cfc1203a43184e662365ab953
tree5e7b1601b95018731242338057140004850043be
parent9ad1a6d64cc6cefccaa94c105dd2e74951281987
avformat/matroskadec: Simplify control flow of parsing laces

Up until now, when an error happened in one of the inner loops in
matroska_parse_laces, a variable designated for the return value has
been set to an error value and break has been used to exit the
current loop/case. This was done so that the end of matroska_parse_laces
is reached, because said function allocated memory which is later used
and freed in the calling function and passed at the end of
matroska_parse_laces.

But given that there is no allocation any more, one can now return
immediately. And this commit does this.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: James Almer <jamrial@gmail.com>
libavformat/matroskadec.c