Opened 19 years ago
Closed 18 years ago
#305 closed defect (fixed)
Single Transport Scan tunes twice.
Reported by: | Owned by: | danielk | |
---|---|---|---|
Priority: | trivial | Milestone: | 0.20 |
Component: | mythtv | Version: | head |
Severity: | low | Keywords: | single transport scan |
Cc: | Ticket locked: | no |
Description
While researching #304 I discovered that when trying to do a single transport scan the multiplex is tuned to twice.
Firstly in ScanWizardScanner::SpawnTune?():212 via
ok = scanner->GetDVBChannel()->TuneMultiplex?(mplexid);
Then tuned again through ScanWizardScanner::HandleTuneComplete?(void):693 via
ok = scanner->ScanTransport?(parent->transport());
One of these tunings is unneccessary. Which one should we remove?
Change History (4)
comment:1 Changed 19 years ago by
Milestone: | 0.19 → 0.20 |
---|---|
Owner: | changed from Isaac Richards to danielk |
Priority: | minor → trivial |
Severity: | medium → low |
Status: | new → assigned |
comment:2 Changed 19 years ago by
I seem to remember that this was needed so that the UI could continue to work while we were doing the tune.
comment:3 Changed 18 years ago by
At a guess I'd say the first one.
As I noted above the first tune is done in a separate thread to keep the UI going. It then adds the transport to the list to be scanned which will then generate another tune (this is also on another thread and so the UI should be running ok). This *should* result in a noop (nothing has changed) and we carry on.
For the "Full Tuned Scan" mode, it looks like we do need the initial tuning in a separate thread, just to generate the mplex list.
I'm wondering if SpawnTune? is needed at all, but I'm not going to touch this for now as the DVB stuff is finally stabilizing.