author | zecke <zecke> | 2004-11-04 21:53:57 (UTC) |
---|---|---|
committer | zecke <zecke> | 2004-11-04 21:53:57 (UTC) |
commit | 37ff6dd391e4b6e0c537cd1d74e226dfdf1b86ba (patch) (side-by-side diff) | |
tree | b261ff83d2f7b77f8663d433b08b005aabd8c184 | |
parent | 5f56ab623c99c12ce246f775e065632fbfbbfc1f (diff) | |
download | opie-37ff6dd391e4b6e0c537cd1d74e226dfdf1b86ba.zip opie-37ff6dd391e4b6e0c537cd1d74e226dfdf1b86ba.tar.gz opie-37ff6dd391e4b6e0c537cd1d74e226dfdf1b86ba.tar.bz2 |
Use QTextStream and QFile instead of read/sscanf for parsing
ChipSelect3 (cs3). This should make it more robust
-rw-r--r-- | libopie2/opiecore/device/odevice_simpad.cpp | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/libopie2/opiecore/device/odevice_simpad.cpp b/libopie2/opiecore/device/odevice_simpad.cpp index 9fde7f9..2e54216 100644 --- a/libopie2/opiecore/device/odevice_simpad.cpp +++ b/libopie2/opiecore/device/odevice_simpad.cpp @@ -217,28 +217,23 @@ void SIMpad::initButtons() static bool setCS3Bit( bool bitset, int bit ) { - int cs3_fd = ::open( SIMPAD_BOARDCONTROL, O_RDONLY ); - - if ( cs3_fd < 0 ) + QFile file( SIMPAD_BOARDCONTROL ); + if ( !file.open( IO_ReadOnly ) ) return false; - static char line[32]; - int val = 0; + unsigned int val = 0; bool ok = false; + QTextStream stream( &file ); /* - * try to read and parse the Chipselect3 status - * be paranoid and make sure line[31] is null - * terminated + * Use QFile and QTextStream for parsing to be more + * robust */ - while( !ok && ::read(cs3_fd, &line, sizeof(line)) > 0 ) { - line[31] = '\0'; - if (::sscanf(line, "Chipselect3 : %x", &val )) + while ( !stream.atEnd() ) { + QString line = stream.readLine(); + if ( line.startsWith( "Chipselect3 : " ) ) { + val = line.mid( 15 ).toUInt( 0, 16 ); ok = true; + break; + } } - ::close(cs3_fd); - - /* - * we were not able to find the current value - * and as a result we won't set it - */ if ( !ok ) @@ -246,2 +241,4 @@ static bool setCS3Bit( bool bitset, int bit ) { + file.close(); + /* @@ -254,3 +251,3 @@ static bool setCS3Bit( bool bitset, int bit ) { */ - cs3_fd = ::open( SIMPAD_BOARDCONTROL, O_WRONLY ); + int cs3_fd = ::open( SIMPAD_BOARDCONTROL, O_WRONLY ); if ( cs3_fd < 0 ) @@ -258,2 +255,3 @@ static bool setCS3Bit( bool bitset, int bit ) { + char line[32]; ::snprintf(line, sizeof(line), "0x%04x\n", val); |