Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wrapping ST 428-7 fails when wrapping more than 1516 PNG files #76

Open
dcbullock opened this issue Apr 1, 2021 · 2 comments
Open

wrapping ST 428-7 fails when wrapping more than 1516 PNG files #76

dcbullock opened this issue Apr 1, 2021 · 2 comments

Comments

@dcbullock
Copy link
Contributor

The count of resources (1516) was determined experimentally and may not be exact.

@dcbullock
Copy link
Contributor Author

dcbullock commented Apr 1, 2021

Issue in src/AC_DCP_TimedText.cpp

The calculation to increase the header buffer size contract (m_HeaderSize) is incorrect. Because the default m_HeaderSize is 16384, this is not an issue for smaller sets of PNG files.

src/AS_DCP_TimedText.cpp ASDCP::TimedText::MXFWriter::h__Writer::SetSourceStream

The growth constant, 72, is too small. With a large enough set of sub-descriptors, m_HeaderSize will be too small to write out the header.

@stephw999
Copy link

with 2.10.39 (compiled with openssl 1.1.1k and Xerces-C 3.2.3) I have wrapped 1744 png in a smpte MXF file. Not yet tested on a server. Best regards. Stéphane.W

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants