Use compiler flag -Wall. Cleanup code as necessary.

(#171 Thanks to Rolf Ahrenberg)
Modified Files:
	Makefile streamdev-server.c server/connection.h
	server/connectionHTTP.c server/connectionVTP.c tools/source.c

3 files changed, 8 insertions, 10 deletions

diff --git a/server/connection.h b/server/connection.h
index 2df850e..fe828d9 100644
--- a/server/connection.h
+++ b/server/connection.h
@@ -1,5 +1,5 @@
 /*
- *  $Id: connection.h,v 1.4 2007/04/02 10:32:34 schmirl Exp $
+ *  $Id: connection.h,v 1.5 2007/04/16 11:01:02 schmirl Exp $
  */
  
 #ifndef VDR_STREAMDEV_SERVER_CONNECTION_H
@@ -38,8 +38,8 @@ protected:
 	   Only one line at a time may be sent. If there are lines to follow, set
 	   Last to false. Command(NULL) will be called in the next cycle, so you can
 	   post the next line. */
-	virtual bool Respond(const char *Message, bool Last = true, ...)
-			__attribute__ ((format (printf, 2, 4)));
+	virtual bool Respond(const char *Message, bool Last = true, ...);
+			//__attribute__ ((format (printf, 2, 4)));
 
 public:
 	/* If you derive, specify a short string such as HTTP for Protocol, which
diff --git a/server/connectionHTTP.c b/server/connectionHTTP.c
index a526da9..5f0beef 100644
--- a/server/connectionHTTP.c
+++ b/server/connectionHTTP.c
@@ -1,5 +1,5 @@
 /*
- *  $Id: connectionHTTP.c,v 1.10 2006/01/26 19:40:18 lordjaxom Exp $
+ *  $Id: connectionHTTP.c,v 1.11 2007/04/16 11:01:02 schmirl Exp $
  */
 
 #include <ctype.h>
@@ -41,6 +41,8 @@ bool cConnectionHTTP::Command(char *Cmd)
 		}
 		Dprintf("header\n");
 		return true;
+	default:
+		break;
 	}
 	return false; // ??? shouldn't happen
 }
@@ -153,7 +155,7 @@ bool cConnectionHTTP::CmdGET(const std::string &Opts)
 {
 	const char *sp = Opts.c_str(), *ptr = sp, *ep;
 	const cChannel *chan;
-	int apid = 0, pos;
+	int apid = 0;
 
 	ptr = skipspace(ptr);
 	while (*ptr == '/')
diff --git a/server/connectionVTP.c b/server/connectionVTP.c
index 18ea353..514c5a0 100644
--- a/server/connectionVTP.c
+++ b/server/connectionVTP.c
@@ -1,5 +1,5 @@
 /*
- *  $Id: connectionVTP.c,v 1.8 2007/03/02 15:27:07 schmirl Exp $
+ *  $Id: connectionVTP.c,v 1.9 2007/04/16 11:01:02 schmirl Exp $
  */
  
 #include "server/connectionVTP.h"
@@ -153,8 +153,6 @@ cLSTEHandler::~cLSTEHandler()
 
 bool cLSTEHandler::Next(bool &Last)
 {
-	char *buffer;
-
 	if (m_Error != NULL) {
 		Last = true;
 		cString str(m_Error, true);
@@ -562,8 +560,6 @@ bool cConnectionVTP::Command(char *Cmd)
 
 bool cConnectionVTP::CmdCAPS(char *Opts) 
 {
-	char *buffer;
-
 	if (strcasecmp(Opts, "TS") == 0) {
 		m_NoTSPIDS = true;
 		return Respond(220, "Ignored, capability \"%s\" accepted for "