[patch] zapit ,kleine optimierung (paar events weniger)

[re_Look]

Code: Alles auswählen

Index: src/zapsi/pmt.cpp                                              
===================================================================   
RCS file: /cvs/tuxbox/apps/dvb/zapit/src/zapsi/pmt.cpp,v              
retrieving revision 1.55                                              
diff -u -r1.55 pmt.cpp                                                
--- a/src/zapsi/pmt.cpp 11 Mar 2009 20:42:10 -0000      1.55          
+++ b/src/zapsi/pmt.cpp 11 Apr 2009 12:37:13 -0000                    
@@ -349,6 +349,10 @@                                                  
        unsigned char filter[DMX_FILTER_SIZE];                        
        unsigned char mask[DMX_FILTER_SIZE];                          
                                                                      
+#ifndef SKIP_CA_STATUS                                               
+       bool CaStatusSent = false;
+#endif
+
        if (channel->getPmtPid() == 0)
                return -1;

@@ -389,7 +393,9 @@
                        case 0x09:
                                caPmt->addCaDescriptor(buffer + i);
 #ifndef SKIP_CA_STATUS
-                               eventServer->sendEvent(CZapitClient::EVT_ZAP_CA_LOCK, CEventServer::INITID_ZAPIT);
+                               if (!CaStatusSent)
+                                       eventServer->sendEvent(CZapitClient::EVT_ZAP_CA_LOCK, CEventServer::INITID_ZAPIT);
+                               CaStatusSent = true;
 //                             INFO("Event_PMT: CA_LOCK send");
 #endif
                                break;

[rhabarber1848]

+#ifndef SKIP_CA_STATUS

Gibt es eigentlich irgendjemanden, der die CA-Anzeige
aus seinem Image entfernt hat oder entfernen möchte?

Mehr Infos dazu hier: http://forum.tuxbox-cvs.sourceforge.net ... 54#p366754
PS: und hier: http://forum.tuxbox-cvs.sourceforge.net ... 07#p274307

Ich teste gerade einen Patch für cdk/configure.ac & Co.,
der besagtes Feature als configure-Option zugänglich macht.

Nur, IMHO, sehe ich den Sinn in einer solchen Option
nicht. Der Speicherbedarf ist minimal, ich würde die
Funktion in jedes Image packen wollen und die ifdef's
entfernen, dadurch wird auch der Code übersichtlicher.

Just my 2 cents.

[rhabarber1848]

Nach reiflicher Überlegung habe ich mich gegen eine
neue cdk/configure-Option entschieden.
Der Patch beinhaltet ebenfalls den o.g. Patch.
Was haltet ihr davon? EDIT: Patch ist im CVS

[dbt]

Ich habe bisher nie ein Image ohne dem CA-Zeugs gebaut, also wenn du das so machen willst, kein Problem.

[rhabarber1848]

Die hier besprochenen Patches sind im CVS.