bug in szap utility

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux