Opened 18 years ago
Closed 15 years ago
#4807 closed patch (fixed)
Save & use last tuned channel for entering LiveTV
| Reported by: | Owned by: | Stuart Auchterlonie | |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.24 |
| Component: | MythTV - DVB | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | stuartm | Ticket locked: | no |
Description
This patch saves the last (successfully) tuned channel so it can be used by LiveTV when first entering it.
The last tuned channel is stored as chanid in settings for the frontend and only saves after NuppleVideoPlayer::OpenFile() successfully opens and reads the file - to ensure it's a good channel.
To test yet:
- Saving last tuned channel when streaming is on.
Attachments (5)
Change History (18)
by , 18 years ago
| Attachment: | save_last_chan.diff added |
|---|
by , 18 years ago
| Attachment: | save_last_chan2.diff added |
|---|
Make sure the channel IsTunable() before using it.
comment:1 by , 18 years ago
Tested successfully with streaming turned on. I've finalized the patch with an addition of IsTunable() of the saved chanid. If the saved chanid is IsTunable(), or points to an unavailable capture card or is 0/undefined, the logic reverts back to the original behavior.
- Mark.
comment:2 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
by , 17 years ago
| Attachment: | nuppel_save.diff added |
|---|
Additional patch required for PVR-type tuners.
comment:3 by , 17 years ago
I've added an additional patch necessary for PVR-type tuners since they operate differently than DVB-types.
- Mark.
comment:4 by , 17 years ago
| Owner: | removed |
|---|---|
| Status: | assigned → new |
comment:5 by , 16 years ago
See, also #6708 for a different implementation of a patch to only save startchan if tuning succeeds.
comment:6 by , 16 years ago
| Cc: | added |
|---|---|
| Component: | mythtv → MythTV - General |
comment:7 by , 16 years ago
| Component: | MythTV - General → MythTV - DVB |
|---|
Not exactly the right component, but maybe it'll get some visibility outside of General and we can get something like this implemented.
comment:8 by , 15 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
[TICKET CLEANUP] Although I think this is a good idea, closing as it has gone years without any attention. I *do* think something like this should get applied, so if you still feel strongly about it, please feel free to open a new ticket with a patch against trunk.
follow-up: 10 comment:9 by , 15 years ago
Mark,
The closing of this has caused one or two of us to peer at these tickets. Can you comment on the benefits of your patch versus the far simpler patch on #6708.
Does this change the behaviour when using livetv?
Stuart
comment:10 by , 15 years ago
Replying to stuarta:
Does this change the behaviour when using livetv?
That should have been
Does this patch change the behaviour only when using livetv?
Stuart
comment:11 by , 15 years ago
| Milestone: | unknown → 0.24 |
|---|---|
| Resolution: | wontfix |
| Status: | closed → new |
| Version: | head → Trunk Head |
Mark's comment:
Yes, only livetv is effected. I've been using this patch for the past two years with no ill effect to recording.
The difference with this patch vs. 6708 is that this takes a few more things into consideration, like if the channel is tunable (enough free capture cards, etc) and the default channel (chanid) is stored and used per/frontend and can survive a backend restart. The chanid is only stored after the channel has been successfully tuned.
comment:12 by , 15 years ago
| Owner: | set to |
|---|---|
| Status: | new → assigned |
comment:13 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
The patch.