On Thu Jul 3 21:09:06 2025 +0000, Elizabeth Figura wrote:
Ah, I see, and looking more carefully, there is a "NONE" alignment that's used if the alignment is missing. I don't see any sign that there's an intended caps value for NONE alignment, so leaving the field out is probably indeed best. However, that does mean the commit message is incorrect, and should not say 'nal'.
You are both correct. I've just updated the commit message. It looks like `h264parse` will just pass through the provided alignment if it is set, but will correct and provide the alignment if it is not. I just ran the following tests: ``` mkdir /tmp/video
# create nal aligned test video gst-launch-1.0 videotestsrc num-buffers=300 ! video/x-raw,format=I420,framerate=30/1 ! openh264enc ! h264parse ! video/x-h264,alignment=nal,stream-format=byte-stream ! multifilesink location=/tmp/video/test_nal_aligned_%05d.h264
# create unaligned test video cat /tmp/video/test_nal_aligned_* | split -d -a 5 -b 2048 --additional-suffix=.h264 - /tmp/video/test_unaligned_
# plays 'nal' alignment correctly gst-launch-1.0 multifilesrc location=/tmp/video/test_nal_aligned_%05d.h264 ! video/x-h264,width=320,height=240,framerate=30/1,stream-format=byte-stream,alignment=au ! h264parse ! avdec_h264 ! autovideosink
# plays unaligned incorrectly (expects 'au' alignment) gst-launch-1.0 multifilesrc location=/tmp/video/test_unaligned_%05d.h264 ! video/x-h264,width=320,height=240,framerate=30/1,stream-format=byte-stream,alignment=au ! h264parse ! avdec_h264 ! autovideosink
# plays unaligned correctly (no alignment in caps) gst-launch-1.0 multifilesrc location=/tmp/video/test_unaligned_%05d.h264 ! video/x-h264,width=320,height=240,framerate=30/1,stream-format=byte-stream ! h264parse ! avdec_h264 ! autovideosink ```