summaryrefslogtreecommitdiff
Unidiff
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--libopie2/opiecore/device/odevice_palm.cpp24
1 files changed, 14 insertions, 10 deletions
diff --git a/libopie2/opiecore/device/odevice_palm.cpp b/libopie2/opiecore/device/odevice_palm.cpp
index 399c08c..d40dfce 100644
--- a/libopie2/opiecore/device/odevice_palm.cpp
+++ b/libopie2/opiecore/device/odevice_palm.cpp
@@ -185,3 +185,3 @@ bool Palm::filter ( int /*unicode*/, int keycode, int modifiers, bool isPress, b
185 } 185 }
186 186
187 if (newkeycode!=keycode) { 187 if (newkeycode!=keycode) {
@@ -194,3 +194,3 @@ bool Palm::filter ( int /*unicode*/, int keycode, int modifiers, bool isPress, b
194 } 194 }
195 195
196 return false; 196 return false;
@@ -231,2 +231,3 @@ int Palm::displayBrightnessResolution() const
231 int res = 1; 231 int res = 1;
232 int fd = -1;
232 233
@@ -237,7 +238,8 @@ int Palm::displayBrightnessResolution() const
237 case Model_Palm_Z72: 238 case Model_Palm_Z72:
238 int fd = ::open( m_backlightdev + "max_brightness", O_RDONLY|O_NONBLOCK ); 239 fd = ::open( m_backlightdev + "max_brightness", O_RDONLY|O_NONBLOCK );
239 if ( fd ) 240 if ( fd != -1)
240 { 241 {
241 char buf[100]; 242 char buf[100];
242 if ( ::read( fd, &buf[0], sizeof buf ) ) ::sscanf( &buf[0], "%d", &res ); 243 if ( ::read( fd, &buf[0], sizeof buf ) > 0 )
244 ::sscanf( &buf[0], "%d", &res );
243 ::close( fd ); 245 ::close( fd );
@@ -249,3 +251,3 @@ int Palm::displayBrightnessResolution() const
249 } 251 }
250 252
251 return res; 253 return res;
@@ -263,2 +265,3 @@ bool Palm::setDisplayBrightness( int bright )
263 int val = ( bright == 1 ) ? 1 : ( bright * numberOfSteps ) / 255; 265 int val = ( bright == 1 ) ? 1 : ( bright * numberOfSteps ) / 255;
266 int fd = -1;
264 267
@@ -270,4 +273,4 @@ bool Palm::setDisplayBrightness( int bright )
270 case Model_Palm_Z72: 273 case Model_Palm_Z72:
271 int fd = ::open( m_backlightdev + "brightness", O_WRONLY|O_NONBLOCK ); 274 fd = ::open( m_backlightdev + "brightness", O_WRONLY|O_NONBLOCK );
272 if ( fd ) 275 if ( fd != -1 )
273 { 276 {
@@ -279,4 +282,5 @@ bool Palm::setDisplayBrightness( int bright )
279 break; 282 break;
280 283
281 default: res = false; 284 default:
285 res = false;
282 } 286 }