Hello,
I have run cppcheck on Workrave and it reported following errors:

$ cppcheck backend/ common/ frontend/ -q --error-exitcode=1
[backend/src/DistributionSocketLink.cc:1112]: (error) Possible null pointer dereference: client - otherwise it is redundant to check if client is null at line 1122
[common/src/win32/crashlog.c:702]: (error) Uninitialized variable: keyname
[common/src/win32/crashlog.cc:724]: (error) Uninitialized variable: keyname
[frontend/gtkmm/src/GUI.cc:1527]: (error) Possible null pointer dereference: b - otherwise it is redundant to check if b is null at line 1529
[frontend/gtkmm/src/GUI.cc]: (information) Interrupted checking because of too many #ifdef configurations.
[frontend/gtkmm/win32/setup/src/zapper.c:168]: (error) Common realloc mistake: 'pids' nulled but not freed upon failure


More warnings are displayed when --enable=all is given on the command line.