bug in szap utility

Uwe Bugla uwe.bugla at gmx.de
Sat Mar 10 14:49:54 CET 2007


Hello Marcel,

I did not try your patch, but the following one against linuxtv-dvb-apps-1.1.1.tar.bz2.
(available @linuxtv.org).
Unfortunately had to adjust the line numbers plus a blank in lines 3 and 6.

Result: The trouble with the wrong polarization printout is gone. "H" conforms to "h" now.
Guess you can commit that now, so that the Debian packagers can repackage that stuff.


P. S.: I was missing an "install"-section in the Makefile of that tarball.
Can you fix that too, please? Cause without that section it is only "half".

Thanks for the quick response and all your efforts!

--- a/util/szap/szap.c  Fri Feb 02 02:16:24 2007 +0100
+++ b/util/szap/szap.c  Tue Mar 06 18:53:39 2007 +0100
@@ -383,7 +383,7 @@ again:
 	 if (!(field = strsep(&tmp, ":")))
 	    goto syntax_err;
 
-	 pol = (field[0] == 'h' ? 0 : 1);
+	 pol = (tolower(field[0]) == 'h' ? 0 : 1);
 
 	 if (!(field = strsep(&tmp, ":")))
 	    goto syntax_err;

Uwe

P. S.: Still waiting for my 2 contributions to be committed!
(i. e. DST-deselect plus documentary fix)

-- 
Der GMX SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen! 
Ideal für Modem und ISDN: http://www.gmx.net/de/go/smartsurfer



More information about the pkg-vdr-dvb-devel mailing list