| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Maybe global `-Wcomma' is too harsh.
There should be no functional changes whatsoever.
|
|
No functional changes.
- add `override' everywhere where missing
- almost pass clang's `-Wweak-vtables'
- avoid some float/double conversions
- remove unused private members
- make signedness conversions explicit
- put stuff in right namespaces to aid analysis
|
|
|
|
|
|
|
|
|
|
No functional changes.
|
|
|
|
No visible changes (hopefully).
|
|
I forgot who reported this, sorry.
|
|
Found-by: @Len62
cf. https://github.com/opentrack/opentrack/issues/688#issuecomment-34063145
|
|
Short keyboard and mouse events won't get dropped. We
can now decrease the poll interval from 250 Hz to 10Hz.
|
|
|
|
We heavily used "volatile bool" to check if the thread
loop should stop. But this functionality is already
provided by Qt5's QThread::requestInterruption.
In other cases, "volatile" is wonderfully
underspecified so it's better to ditch its usage in
favor of std::atomic<t>. At the time we don't appear to
be using the "volatile" keyword except when calling
win32's Interlocked*() family of functions as
necessary.
In freetrackclient's header the "volatile" qualifier
was used as part of a typedef. This doesn't work. Use
it as part of data declaration.
|
|
|
|
|
|
It was easiest to reproduce when a protocol failed to load. As usual, win32
debugging without MSVC++ is nearly useless.
|
|
Adjust usages.
|
