[Patch] Fix pzapit -gsi bei Kabelboxen

[GetAway]

Ändert die Ausgabe von:

/ # pzapit -gsi
392295.0 MHzdiseqc = 255
onid = 0x0001
sid = 0x6d70
tsid = 0x0437
...

in:

/ # pzapit -gsi
frequency = 392295.0 MHz
onid = 0x0001
sid = 0x6d70
tsid = 0x0437
...

Code: Alles auswählen

Index: apps/dvb/zapit/src/pzapit.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/dvb/zapit/src/pzapit.cpp,v
retrieving revision 1.64
diff -U3 -r1.64 pzapit.cpp
--- a/apps/dvb/zapit/src/pzapit.cpp	24 Feb 2009 13:50:25 -0000	1.64
+++ b/apps/dvb/zapit/src/pzapit.cpp	10 Apr 2009 19:49:17 -0000
@@ -709,13 +709,18 @@
 		CZapitClient::CCurrentServiceInfo si;
 		si = zapit.getCurrentServiceInfo();
 
-		printf("%d.%d MHz", si.tsfrequency/1000, si.tsfrequency%1000);
+		printf("frequency = %d.%d MHz", si.tsfrequency/1000, si.tsfrequency%1000);
 
 		if (si.polarisation != 2) /* only satellite has polarisation */
+		{
 			printf(" (%c)\n", (si.polarisation == HORIZONTAL) ? 'h' : 'v');
-		//satellite
-		printf("diseqc = %d\n", si.diseqc);
-
+			//satellite
+			printf("diseqc = %d\n", si.diseqc);
+		}
+		else
+		{
+			printf("\n");
+		}
 		printf("onid = 0x%04x\n", si.onid);
 		printf("sid = 0x%04x\n", si.sid);
 		printf("tsid = 0x%04x\n", si.tsid);

Link entfernt, im CVS

[MTM]

Hallo,
leicht OT: ich habe gerade im Wiki die Beschreibung der einzelnen Parameter umsortiert, sie waren IMHO durcheinander und damit unnötig kompliziert zu durchschauen. Bin mir bei manchen nicht 100% sicher, aber so sieht es schon einfacher aus, zumindest grob nach Funktionsgruppen sortiert. Falls es so zusagt, könnte die Hilfe ja entsprechend angepasst werden (s.u.).

Dabei sind mir allerdings auch einige Sachen aufgefallen:
- es sind längst nicht alle möglichen Parameter aufgelistet (u.a. das hier angesprochene -gsi), das sollte noch ergänzt werden (auch in der pzapit-eigenen Hilfe)
- manche Parameter brauchen zwingend einen doppelten Bindestrich "--", die meisten nur einen einfachen, warum? Laut Wikipedia:

Bei Programme, die sowohl einbuchstabige als auch mehrbuchstabige Optionen annehmen, sind letztere mit „--“ einzuleiten.

Keine Ahnung inwieweit das allg. Konvention ist, bei pzapit wird es jedenfalls ignoriert.

MfG,
MTM.

[GetAway]

Hallo,
- es sind längst nicht alle möglichen Parameter aufgelistet (u.a. das hier angesprochene -gsi), das sollte noch ergänzt werden (auch in der pzapit-eigenen Hilfe)

Mach doch mal eine Auflistung der Fehlenden. Hilfetext für -gsi ist aber drin.

Bei Programme, die sowohl einbuchstabige als auch mehrbuchstabige Optionen annehmen, sind letztere mit „--“ einzuleiten.

Das sind Empfehlungen, aber jeder hat ja seinen eigenen Programmierstil.

[rhabarber1848]

pzapit-outputfix.diff

committed to CVS: http://article.gmane.org/gmane.comp.vid ... ox.scm/218

[GetAway]

Thx rhabarber1848

[MTM]

Hallo,

Mach doch mal eine Auflistung der Fehlenden.

Folgende Optionen sind zwar im Quelltext enthalten (soweit ich das lesen kann ), aber nicht in der pzapit-eigenen Hilfe:
-nvod
-fastzap
Ausserdem existiert dort ein -motor, aber laut Hilfe müsste es -m sein.

Im Wiki fehlen noch die folgenden Optionen:
-zi
-rz
-gi
-gm
-gsi
--getpids
-sbo

MfG,
MTM.

[rhabarber1848]

-fastzap

Btw, nur für Dreambox: http://cvs.tuxbox-cvs.sourceforge.net/l ... 00020.html

[MTM]

Hallo,
ok, das fällt dann weg.

MfG,
MTM.