This forum uses cookies
This forum makes use of cookies to store your login information if you are registered, and your last visit if you are not. Cookies are small text documents stored on your computer; the cookies set by this forum can only be used on this website and pose no security risk. Cookies on this forum also track the specific topics you have read and when you last read them. Please confirm whether you accept or reject these cookies being set.

A cookie will be stored in your browser regardless of choice to prevent you being asked this question again. You will be able to change your cookie settings at any time using the link in the footer.

Feature request: new mkv tags
#6
Okay, this is more complicated than I would have thought, since I didn't think about what this would require.
Message processing is separated from job processing and all the other stuff.
So DebugOutput and Report creation are independent of job processing.
(Both can be turned off and on during job processing, without influencing the job processing.)
Adding the report isn't possible without major changes, which would break 'separation of concern' in the code, which I will not do.
(Hybrid's internal logic and dependencies are complicated/bad enough as they are; ~250k lines of code.)

So Hybrid would need to create a separate file before the muxing call, which includes all the wanted data and that could be added to the muxing call.
But have you ever checked how normal media players handle unknown mkv text attachments?
And how do you plan to look at the attachment?
The only thing that pops into my mind would be to extract the file with mkvextract and then look at it, but then simply using the report file as a separate file is probably more convenient.

=> atm. keeping the normal report file seems to be the most sensible solution.

Cu Selur
----
Dev versions are in the 'experimental'-folder of my GoogleDrive, which is linked on the download page.
Reply


Messages In This Thread
Feature request: new mkv tags - by Dan64 - 19.09.2024, 19:48
RE: Feature request: new mkv tags - by Selur - 19.09.2024, 20:08
RE: Feature request: new mkv tags - by Dan64 - 19.09.2024, 20:40
RE: Feature request: new mkv tags - by Selur - 19.09.2024, 20:53
RE: Feature request: new mkv tags - by Dan64 - 20.09.2024, 08:27
RE: Feature request: new mkv tags - by Selur - 20.09.2024, 16:41

Forum Jump:


Users browsing this thread: 1 Guest(s)