Fixing warnings (David H. Lynch Jr).

damellis · damellis · commit 1ce911ceabcb · 2011-03-03T22:56:20.000-05:00
diff --git a/utility/socket.cpp b/utility/socket.cpp
@@ -9,7 +9,6 @@ static uint16_t local_port;
  */
 uint8_t socket(SOCKET s, uint8_t protocol, uint16_t port, uint8_t flag)
 {
-  uint8_t ret;
   if ((protocol == SnMR::TCP) || (protocol == SnMR::UDP) || (protocol == SnMR::IPRAW) || (protocol == SnMR::MACRAW) || (protocol == SnMR::PPPOE))
   {
     close(s);
diff --git a/utility/w5100.cpp b/utility/w5100.cpp
@@ -134,7 +134,7 @@ uint8_t W5100Class::write(uint16_t _addr, uint8_t _data)
 
 uint16_t W5100Class::write(uint16_t _addr, uint8_t *_buf, uint16_t _len)
 {
-  for (int i=0; i<_len; i++)
+  for (uint16_t i=0; i<_len; i++)
   {
     setSS();    
     SPI.transfer(0xF0);
@@ -160,7 +160,7 @@ uint8_t W5100Class::read(uint16_t _addr)
 
 uint16_t W5100Class::read(uint16_t _addr, uint8_t *_buf, uint16_t _len)
 {
-  for (int i=0; i<_len; i++)
+  for (uint16_t i=0; i<_len; i++)
   {
     setSS();
     SPI.transfer(0x0F);

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@ static uint16_t local_port;`
`9`	`9`	`*/`
`10`	`10`	`uint8_t socket(SOCKET s, uint8_t protocol, uint16_t port, uint8_t flag)`
`11`	`11`	`{`
`12`		`- uint8_t ret;`
`13`	`12`	`if ((protocol == SnMR::TCP) \|\| (protocol == SnMR::UDP) \|\| (protocol == SnMR::IPRAW) \|\| (protocol == SnMR::MACRAW) \|\| (protocol == SnMR::PPPOE))`
`14`	`13`	`{`
`15`	`14`	`close(s);`
Original file line number	Diff line number	Diff line change
`@@ -134,7 +134,7 @@ uint8_t W5100Class::write(uint16_t _addr, uint8_t _data)`
`134`	`134`
`135`	`135`	`uint16_t W5100Class::write(uint16_t _addr, uint8_t *_buf, uint16_t _len)`
`136`	`136`	`{`
`137`		`- for (int i=0; i<_len; i++)`
	`137`	`+ for (uint16_t i=0; i<_len; i++)`
`138`	`138`	`{`
`139`	`139`	`setSS();`
`140`	`140`	`SPI.transfer(0xF0);`
`@@ -160,7 +160,7 @@ uint8_t W5100Class::read(uint16_t _addr)`
`160`	`160`
`161`	`161`	`uint16_t W5100Class::read(uint16_t _addr, uint8_t *_buf, uint16_t _len)`
`162`	`162`	`{`
`163`		`- for (int i=0; i<_len; i++)`
	`163`	`+ for (uint16_t i=0; i<_len; i++)`
`164`	`164`	`{`
`165`	`165`	`setSS();`
`166`	`166`	`SPI.transfer(0x0F);`