Thema: kaffeine scan patch

[amin]

Der patch checkt jetzt ob channel neu ist oder nur geupdatet werden muss.
Bis jetzt war nach jedem scan immer mehr channels mit -X in name und die Liste war  immer länger.
//edit
<link unten>
gruss amin.

[named]

Could you repost the patch somewhere else? The address above doesn't work atm.

[amin]

//edit
scandialog.diff - 0.00MB

[mtron]

hallo amin!

Danke für den Patch. Eine Kleinigkeit ist mir aufgefallen:
wenn der Kanal bereits vorhanden ist, und nur aktualisiert wird, verliert der Kanal das vor dem Scan  zugewiesene Icon.

Könntest du das noch bitte fixen?

lg, mtron

[named]

Another problem occurs when you have reordered the audio pids. After an update with your patch, they are back to their original order.

[amin]

wenn der Kanal bereits vorhanden ist, und nur aktualisiert wird, verliert der Kanal das vor dem Scan  zugewiesene Icon.

Aber nach neuen start von Kaffeine sind alle Icons wieder da, oder ?
Das mit audio pids.
Bei dvb werden oft audio-pids geupdatet, deswegen wollte ich die update eben haben.
Mal sehen, was man da noch verbessern kann.

[mtron]

stimmt, hast recht, sorry....

Nach einem Neustart sind die Icons wieder da 

[amin]

Ich glaube da ist Fehler in dvbpanel.
Hier patch der das beheben soll:

Code: [Auswählen]

Index: dvbpanel.cpp
===================================================================
--- dvbpanel.cpp (revision 659778)
+++ dvbpanel.cpp (working copy)
@@ -1166,6 +1166,7 @@
  continue;
  it = new KListViewItem( channelsCb, QString().sprintf("%04d", chan->num), chan->name, chan->tp.source );
  it->setDragEnabled( true );
+ chan->pix.load( dvbConfig->dvbConfigIconsDir+chan->name );
  if ( !chan->pix.isNull() )
  it->setPixmap( 1, chan->pix );
  else {
Damit sollte es ohne neu start gehen.

Apropos Icons. mtron du hast immer so schöne Icons bei screenshot. Kannst du deine Icons irgendwo uploaden ?

<edit>
Ich habe bei mir noch etwas geändert.
Die Audiopids muss man dann anders regeln, sonst kann man sich scan auch sparen.

Code: [Auswählen]

bool ScanDialog::checkChannUpdate( ChannelDesc *chan )
{
int i;
for ( i=0; i<(int)chandesc->count(); i++ ) {
if ( chan->tp.source == chandesc->at(i)->tp.source && chan->tp.tsid == chandesc->at(i)->tp.tsid && chan->sid == chandesc->at(i)->sid ) {
chan->category=chandesc->at(i)->category;
chan->num=chandesc->at(i)->num;
chan->name=chandesc->at(i)->name;
chan->fta=chandesc->at(i)->fta;
chandesc->replace(i,new ChannelDesc(*chan));
return false;
}
}
return true;
}



[mtron]

ja, der dvbpanel fix mit den kanal - icons funktioniert und auch der Scan benimmt sich bei den Apids updates wie es sein sollte, Wunderbar

ich habe mal die scanpatches und den dvbpanel fix in einem Patch gegen das aktuelle SVN (kaffeine revision 663183) zusammengefasst:

http://www.dvbcube.org/index.php?topic=20143.msg78557#msg78557

[amin]

Ich habe noch was an der Apids gemacht.
Also wenn ein andre apid als aktive pid gewählt wurde, wird der nach scan weiter als aktive geführt.
Ich denke sowas wollte named .

<link unten>

[named]

I'm itching to give it a try but it doesn't apply against current svn (revision 665340).

[amin]

@named
Was ist damit ?
http://pastebin.ca/491328

[named]

Great! Works just as expected.

It would be even better if apids are updated upon tuning a channel. I think vdr does this. This is very useful because some channels don't broadcast all used apids all of the time. E.g., ORF1 usually has 161(ger) and 163(ac3). However when airing Desparate Howswives (may be other shows also) they also broadcast 162(eng). So if you update channels when 162 is not broadcast, you lose it.
Another useful thing about vdr is the preferred language setting. Btw, that's how I discovered apid 162 on ORF1.

I also talked to hftom about adding some dcop calls/key bindings for switching between audios, he added this on his TODO but ... nothing so far.

Anyway, thanks so much. Hope this patch finds its way into svn very soon.

[amin]

Jo, das ist was ich an kaffeine nicht mag.
Normale weise sollte kaffeine erst bei umschallten auf ein Sender alle pids (vpid apid teletxt-pid usw.) von pmt live-lesen.

Preferred language setting will ich auch noch machen.

cu amin

[named]

patch already in svn