deac/osd-contiki
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Changed 'data' casting for 'ctk_signal_keypress'.

Browse source 
In these cases data was set as `process_post_synch(window->owner, ctk_signal_keypress, (process_data_t)(size_t)c);` so it seems reasonable to cast "back" to `size_t`. Additionally the comparison can be limited to `char`. See i.e. calcc for reference:
```
if(ev == ctk_signal_keypress) {
  if((char)(size_t)data >= '0' &&
     (char)(size_t)data <= '9') {
```
This commit is contained in:
Oliver Schmidt 2015-10-06 16:25:42 +02:00 committed by Simon Duquennoy
parent 8f954fbab5
commit a29502ed0d
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
core/ctk/ctk-filedialog.c
View file

@ -155,14 +155,14 @@ ctk_filedialog_eventhandler(struct ctk_filedialog_state *s,
} }
return 1; return 1;
} else if(ev == ctk_signal_keypress) { } else if(ev == ctk_signal_keypress) {
if((unsigned long)data == CH_CURS_UP) { if((char)(size_t)data == CH_CURS_UP) {
clearptr(); clearptr();
if(fileptr > 0) { if(fileptr > 0) {
--fileptr; --fileptr;
} }
showptr(); showptr();
return 1; return 1;
} else if((unsigned long)data == CH_CURS_DOWN) { } else if((char)(size_t)data == CH_CURS_DOWN) {
clearptr(); clearptr();
if(fileptr < FILES_HEIGHT - 1) { if(fileptr < FILES_HEIGHT - 1) {
++fileptr; ++fileptr;