• Tim-Philipp Müller's avatar
    rtph264depay: simplify buffer accumulation control flow · a68a7fb6
    Tim-Philipp Müller authored
    There is no difference between pushing out a buffer directly
    with gst_rtp_base_depayload_push() and returning it from the
    process function. The base class will just call _depayload_push()
    on the returned buffer as well.
    
    So instead of marshalling buffers through three layers and back,
    just push them from one place in handle_nal() and always return
    NULL from the process vfunc. This simplifies the code a little.
    
    Also rename _push_fragmentation_unit() to _finish_fragmentation_unit()
    for clarity. Push sounds like it means being pushed out, whereas
    it might just be pushed into an adapter.
    
    This change has the side-effect that multiple NALs in a single STAP
    (such as SPS/PPS) may no longer be pushed out as a single buffer if
    we output NALs in byte-stream format (i.e. not aggregate AUs), but
    that shouldn't really make any difference to anyone.
    a68a7fb6
Name
Last commit
Last update
common @ 48a5d85e Loading commit data...
docs Loading commit data...
ext Loading commit data...
gst Loading commit data...
gst-libs/gst Loading commit data...
hooks Loading commit data...
m4 Loading commit data...
pkgconfig Loading commit data...
po Loading commit data...
sys Loading commit data...
tests Loading commit data...
.gitignore Loading commit data...
.gitmodules Loading commit data...
AUTHORS Loading commit data...
COPYING Loading commit data...
ChangeLog Loading commit data...
MAINTAINERS Loading commit data...
Makefile.am Loading commit data...
NEWS Loading commit data...
README Loading commit data...
README.static-linking Loading commit data...
RELEASE Loading commit data...
REQUIREMENTS Loading commit data...
autogen.sh Loading commit data...
configure.ac Loading commit data...
gst-plugins-good.doap Loading commit data...
meson.build Loading commit data...
meson_options.txt Loading commit data...