(09-29-2016, 01:05 PM)Wexrio Wrote: But is there a cmake option to disable it? Maybe I could make one...
Yes, there's ENABLE_ANALYTICS, but it's actually completely broken right now. In the rush to get analytics finished before the 5.0 release, no code got added to make it do anything. If you want to fix that, pull requests are welcome.
(09-29-2016, 01:05 PM)Wexrio Wrote: Also why is a constexpr used and then defined as a u32 on line 138 of Core/Common/Analytics.cpp ? Excuse my noobness in programming but wouldn't that make the buffer size 800 bits instead of dependent on compile time options? Seems like a small buffer for all the data you are grabbing up.
I don't know why you're talking about "instead of dependent on compile time options", but anyway... The u32 means that the value 25 is stored in a u32 – it doesn't mean that the size limit is 25 u32s. m_reports_queue contains strings, so the size limit is 25 strings.