In fscd.c current_time is a clock_t, but mode_configure and mode_brightness are ints.
On my 64-bit system this causes the timeout feature to be completely broken.
Changing the definition of mode_configure and mode_brightness to clock_t fixes
things. I think this should be OK on 32-bit systems as well.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Along those lines, the debug formats in handle_x11_event() and handle_xinput_event(), and the declarations of key_fn and key_alt in handle_xinput_event() also should be updated (to clock_t).
The format should use a long type. Right now the code has %lu for current_time, but on my system clock_t is a long int, so it should really be %ld. But maybe on other systems unsigned is the way to go.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
In fscd.c current_time is a clock_t, but mode_configure and mode_brightness are ints.
On my 64-bit system this causes the timeout feature to be completely broken.
Changing the definition of mode_configure and mode_brightness to clock_t fixes
things. I think this should be OK on 32-bit systems as well.
Along those lines, the debug formats in handle_x11_event() and handle_xinput_event(), and the declarations of key_fn and key_alt in handle_xinput_event() also should be updated (to clock_t).
The format should use a long type. Right now the code has %lu for current_time, but on my system clock_t is a long int, so it should really be %ld. But maybe on other systems unsigned is the way to go.
Many thanks for report, I'll look into this.
Best regards,
Robert
Should be fixed now. Thanks again.