Revision: 7252 http://playerstage.svn.sourceforge.net/playerstage/?rev=7252&view=rev Author: thjc Date: 2009-01-07 18:40:46 +0000 (Wed, 07 Jan 2009)
Log Message: ----------- fixed some compiler warnings in p2os.cc Modified Paths: -------------- code/player/branches/release-2-1-patches/server/drivers/mixed/p2os/p2os.cc Modified: code/player/branches/release-2-1-patches/server/drivers/mixed/p2os/p2os.cc =================================================================== --- code/player/branches/release-2-1-patches/server/drivers/mixed/p2os/p2os.cc 2009-01-07 18:31:15 UTC (rev 7251) +++ code/player/branches/release-2-1-patches/server/drivers/mixed/p2os/p2os.cc 2009-01-07 18:40:46 UTC (rev 7252) @@ -845,7 +845,12 @@ if(this->radio_modemp) { puts("Initializing radio modem..."); - write(this->psos_fd, "WMS2\r", 5); + int ret = write(this->psos_fd, "WMS2\r", 5); + if (ret < 5) + { + PLAYER_ERROR1("P2OS: Write failed to complete (%d)",ret); + return 1; + } usleep(50000); char modem_buf[50]; @@ -864,7 +869,12 @@ while(strstr(modem_buf, "ected to addres") == NULL) { puts("Initializing radio modem..."); - write(this->psos_fd, "WMS2\r", 5); + int ret = write(this->psos_fd, "WMS2\r", 5); + if (ret < 5) + { + PLAYER_ERROR1("P2OS: Failed to write full packet to modem (%d)", ret); + return 1; + } usleep(50000); char modem_buf[50]; @@ -1633,9 +1643,9 @@ } if(packet.packet[0] == 0xFA && packet.packet[1] == 0xFB && - (packet.packet[3] == 0x30 || packet.packet[3] == 0x31) || - (packet.packet[3] == 0x32 || packet.packet[3] == 0x33) || - (packet.packet[3] == 0x34)) + (packet.packet[3] == 0x30 || packet.packet[3] == 0x31 || + packet.packet[3] == 0x32 || packet.packet[3] == 0x33 || + packet.packet[3] == 0x34)) { /* It is a server packet, so process it */ @@ -1695,10 +1705,10 @@ } } else if(packet.packet[0] == 0xFA && packet.packet[1] == 0xFB && - (packet.packet[3] == 0x50 || packet.packet[3] == 0x80) || -// (packet.packet[3] == 0xB0 || packet.packet[3] == 0xC0) || - (packet.packet[3] == 0xC0) || - (packet.packet[3] == 0xD0 || packet.packet[3] == 0xE0)) + (packet.packet[3] == 0x50 || packet.packet[3] == 0x80 || +// packet.packet[3] == 0xB0 || packet.packet[3] == 0xC0 || + packet.packet[3] == 0xC0 || + packet.packet[3] == 0xD0 || packet.packet[3] == 0xE0)) { /* It is a vision packet from the old Cognachrome system*/ @@ -2369,12 +2379,15 @@ imager_config->contrast); if (imager_config->autogain >= 0) + { if (imager_config->autogain == 0) np += sprintf((char*)&cam_command[np], " 19 32"); else np += sprintf((char*)&cam_command[np], " 19 33"); + } if (imager_config->colormode >= 0) + { if (imager_config->colormode == 3) np += sprintf((char*)&cam_command[np], " 18 36"); else if (imager_config->colormode == 2) @@ -2383,6 +2396,7 @@ np += sprintf((char*)&cam_command[np], " 18 44"); else np += sprintf((char*)&cam_command[np], " 18 40"); + } if (np > 6) { This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. ------------------------------------------------------------------------------ Check out the new SourceForge.net Marketplace. It is the best place to buy or sell services for just about anything Open Source. http://p.sf.net/sfu/Xq1LFB _______________________________________________ Playerstage-commit mailing list Playerstage-commit@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/playerstage-commit