opentrack.git/tracker-s2bot/ftnoir_tracker_s2bot.cpp, branch opentrack-2.3.11-beta1head tracking software for Windows and Linuxhttp://ananke.misaki.pl/cgit/opentrack.git/atom?h=opentrack-2.3.11-beta12018-07-08T21:20:56Zmodernize only2018-07-08T21:20:56ZStanislaw Haliksthalik@misaki.pl2018-07-08T21:20:56Zurn:sha1:90940a774eab876c38d5cef981b4be5bae67a462core, modules: modernize syntax only2018-06-26T21:01:53ZStanislaw Haliksthalik@misaki.pl2018-06-22T10:54:47Zurn:sha1:f50ac3549d6a7f1199fa012e4b03f581bc8d305b
Use more C++17 features where this helps any.
clean up "static" and "constexpr" types2018-02-15T09:23:11ZStanislaw Haliksthalik@misaki.pl2018-02-15T08:06:13Zurn:sha1:0a92bc147f91f3ecacdf66d995f01f9577107a86
- use `static constexpr inline' to avoid requiring
explicit declarations in object code
- use `const Foo* const' to maybe put into readonly
binary segment (at least for ELF DSOs)
- `constexpr' in function scope has storage, avoid
`static'
- don't use `constexpr' where there's no advantage,
like arrays
We'd like to avoid overhead of atomic initialization
for each function call. No idea how `static constexpr'
requiring storage in the standard plays with atomic
initialization requirement. Hearsay points that
`constexpr' without `static' in block scope behaves
more to our liking. It's all hazy though.
I'm not 100% sure if `static inline constexpr' has any
storage. Hopefully none, like a #define, and stuff
bigger than registers gets coalesced within the same
module, with small stuff being immediates.
compat/util: retire2018-01-18T18:35:29ZStanislaw Haliksthalik@misaki.pl2018-01-18T18:35:08Zurn:sha1:d361f57c8064ce13a0478653050b169dc94e2e99
Adjust usages.
api: add status check for modules2017-12-04T11:19:26ZStanislaw Haliksthalik@misaki.pl2017-12-03T21:23:08Zurn:sha1:b256886a5cb9f2ae3ebda70a2045b19ed9f4233euse std::size2017-09-23T17:22:18ZStanislaw Haliksthalik@misaki.pl2017-09-23T17:22:18Zurn:sha1:2658bc66b8b68212594c71f7b51199921c3c1df9tracker/s2bot: fix few bugs2017-07-21T03:42:42ZStanislaw Haliksthalik@misaki.pl2017-07-21T03:42:42Zurn:sha1:45e6b4f1539b4ed68ff1d4136897827111d00396
- Received values got converted to integers. I find
this hard to believe. Even if this is so, converting
to double leaves forward-compatibility if the sender
software gets its act together.
- There was a buffer overflow when indexing the
"indices" array.
- Mutex scope bigger than necessary.
The code is copy-pasted from the FreePIE tracker.
Perhaps if its own code was easier to understand, there
wouldn't be any buffer overflows to begin with.
tracker/s2bot: reformat to indent as 4 spaces2017-07-21T03:25:21ZStanislaw Haliksthalik@misaki.pl2017-07-21T03:23:31Zurn:sha1:4358241517f0c4740a729959855cab8db4c7f4c6tracker/s2bot: fix stopping tracker2017-07-20T21:13:15ZStanislaw Haliksthalik@misaki.pl2017-07-20T21:13:15Zurn:sha1:83b474c4d084c27245a2b1a6790e371193caae3a
- need use QThread::exit to stop the event loop
- QObject::moveToThread works when passed a working
thread
- stop QTimer in its thread
tracker/s2bot: fix -Werror2017-07-18T14:41:58ZStanislaw Haliksthalik@misaki.pl2017-07-18T14:41:58Zurn:sha1:c318a7f682d1d355b834a656ecde8c62ab521338