Opened 11 years ago

Closed 11 years ago

#7966 closed defect (wontfix)

ATSC should be a valid tvformat option.

Reported by: mythtv@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: MythTV - General Version: head
Severity: medium Keywords:
Cc: Ticket locked: no


Change 15776 removed ATSC as a valid option for tvformat. This was done for ticket 4343. While it's not a valid V4L2 format, ATSC won't be recorded by a V4L2 device, and ATSC is used by some other modules, such as the transcoder for determining sane defaults. I believe it should be added back to channelsettings.cpp. I've attached a very simple patch which does this.

If ATSC is not a valid option for tvformat then all other code which looks for it should be removed.

Attachments (1)

atsc.diff (407 bytes) - added by mythtv@… 11 years ago.
Add ATSC back as an option for tvformat

Download all attachments as: .zip

Change History (2)

Changed 11 years ago by mythtv@…

Attachment: atsc.diff added

Add ATSC back as an option for tvformat

comment:1 Changed 11 years ago by robertm

Resolution: wontfix
Status: newclosed

Reading the commits related to #4343, it appears that the intend is to only have the analog v4l2 formats supported in the dropdown. As you suggested, the remaining ATSC code should be rewritten or removed, but the removal appears to have been intentional.

Note: See TracTickets for help on using tickets.