bug in fscd.d

Help
Jim
2011-04-14
2013-05-08
  • Jim
    Jim
    2011-04-14

    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.

     
  • Jim
    Jim
    2011-04-14

    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

     
  • Robert Gerlach
    Robert Gerlach
    2011-11-19

    Should be fixed now. Thanks again.