search

stefantalpalaru / w_scan2

w_scan fork (channel scan tool which generates ATSC, DVB-C, DVB-S/S2 and DVB-T/T2 channels.conf files)
GNU General Public License v2.0
80 stars 22 forks source link

Include virtual channel number in playlist title #44

Closed troxor closed 2 years ago

troxor commented 2 years ago

I found it helpful to have the virtual channel number printed in the VLC playlist, and this change satisfied that.

stefantalpalaru commented 2 years ago

I'm only getting "0:0" in Italy, with DVB-T. We should probably limit this to DVB-S/S2.

troxor commented 2 years ago

sounds good to me, I only tested with over-the-air ATSC in the US.

stefantalpalaru commented 2 years ago

Try these conditions: https://github.com/stefantalpalaru/w_scan2/blob/a780e3e38e5680bf7087806f084b06c0ef66674b/src/scan.c#L1156-L1157

It's the only place parse_logical_channel_descriptor() is called and that's the only function that sets logical_channel_number: https://github.com/stefantalpalaru/w_scan2/blob/a780e3e38e5680bf7087806f084b06c0ef66674b/src/descriptors.c#L1453

troxor commented 2 years ago

I took the cowardly/simple way out, since the table info doesn't seem to be exposed in dump-vlc-m3u.c.

stefantalpalaru commented 2 years ago

Works for me.