commit 7fff679f added else block and
left continue that didn't make code flow following any easier.

Similar than rtp input does, doing realloc from 64k->1.5k would
usually mean extra malloc+memcpy per packet.

Also don't pass buffer onward if we don't receive any useful stuff (-1)

Wise with a struct that has a bad track record
of read overflows and now contains a pf_

For playlist-autostart, PLAYLIST_PLAY is used now which does not restart
the current item if there is already one running.

This avoids restart of the same item in a quick succession, and thus
workarounds the issue described in refs #11488.

Refs #12298

Regular streams skip if p_read is NULL.

Possible could crash because release function was
not correctly matched du to missing parent id check.

Better save it into box instead of doing another lookup,
and because box could have been detached from parent.

Installed version cannot find lua scripts.
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Detected by CppCheck
Signed-off-by: Boris Egorov <egorov@linux.com>
Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Remove dead code and superfluous width/height variables.

refs #12283

first segment is always 0, not an undefined value
regression introduced by aaebd49a
instead of removing the tautology

kills that signed / unsigned comparison warning

The "```" case could also never happen as this is the
decoding of value 0 which should map to mac.