[enhancement] User Select HDMV descriptors

[jcdr428]

Changes the behavior of the "Generate HDMV compatible TS" checkbox in the "General" tab:

For "TS muxing" mode, the box is unchecked by default and can be checked;
For "M2TS muxing" mode, the box is checked by default and can be unchecked;
For all other modes (i.e. Blu-ray and demux), the box is checked by default and greyed out.

Closes issue/request #273 and #208 (comment)

[jcdr428]

@abakum yes you have to build from the commit, or take the build on my fork.

[abakum]

I did it via fork. GUI ok, PGS as conceived, but no warning:

MUXOPT --no-pcr-on-video-pid --vbr  --vbv-len=500
V_MPEG4/ISO/MVC, "C:\msys64\home\Konstantin.Abakumov\mvc.mkv", insertSEI, contSPS, track=1, lang=eng, subTrack=1
V_MPEG4/ISO/AVC, "C:\msys64\home\Konstantin.Abakumov\mvc.mkv", insertSEI, contSPS, track=1, lang=eng, subTrack=2
A_AC3, "C:\msys64\home\Konstantin.Abakumov\mvc.mkv", track=2, lang=rus
#S_HDMV/PGS, "C:\msys64\home\Konstantin.Abakumov\mvc.mkv", fps=25, track=3, lang=rus
S_TEXT/UTF8, "C:\msys64\home\Konstantin.Abakumov\mvc.mkv",font-name="Impact",font-size=65,font-color=0xffffffff,bottom-offset=24,font-border=5,text-align=center,fadein-time=0.1,fadeout-time=0.1,video-width=1920,video-height=1080,fps=25, track=4, lang=rus

tsMuxeR version git-0ece8e8. github.com/justdan96/tsMuxer
Decoding H264 stream (track 1): H.264/MVC Views: 2 Profile: High@4.2  Resolution: 1920:1080p  Frame rate: 25
MVC muxing fps is not set. Get fps from stream. Value: 25
Decoding H264 stream (track 2): Profile: High@4.2  Resolution: 1920:1080p  Frame rate: 25
H.264 muxing fps is not set. Get fps from stream. Value: 25
H264 bitstream changed: insert nal unit delimiters
H264 bitstream changed: insert nal unit delimiters
Decoding AC3 stream (track 3): Bitrate: 256Kbps Sample Rate: 48KHz Channels: 2
Decoding PGS stream (track 4):  Resolution: 1920:1080  Frame rate: 25
Processed 100 video frames
Processed 100 video frames
Flushing write buffer
Mux successful complete
Muxing time: 0 sec

[abakum]

Warning: PGS might not work without HDMV descriptors.

[abakum]

It is a pity that instead of --new-audio-pes now --hdmv-descriptors. This means that the new tsMuxer will not be compatible with the old one.
Anyone who used --new-audio-pes in meta files will be surprised why there are no hdmv-descriptors in ts or m2ts

[jcdr428]

@abakum yes indeed. But does it make sense to keep calling the hdmv descriptor option "new-audio-pes", when this new PES stream_id is only a small part of what the option is doing ? Is there any sense still using the old GUI when xavery has incuded a build for XP ? Note that there will still be HDMV descriptors, only the AC3 stream_id changes.

Edit: for the warning, I will change LTRACE(LT_WARN, 0, to LTRACE(LT_WARN, 2,

[abakum]

It would be great for those who are used to the old tsMuxer to leave the action of the old --new-audio-pes similar to the new --hdmv-descriptors

[jcdr428]

Ok, I'll include both --new-audio-pes --hdmv-descriptors for the option...