Hi Sergei,

On Sat, Mar 11, 2017 at 9:09 PM, Sergei Golubchik <serg@mariadb.org> wrote:
Marko, when you change the ABI of services you absolutely must update
coresponding service versions in include/service_versions.h.

May be, it's best not to break the ABI for such a minor reason and only
change THD methods, not service functions.

Based on your feedback, I will not push this patch. Instead, I pushed a few other size_t patches to bb-10.2-marko for testing. With those patches, the warning count on 64-bit Windows (where size_t is 64 bits but long is 32 bits) will be a little lower than it is now. The warning count on 32-bit Windows is approaching a bearable number.

Best regards,

Marko
--
DON’T MISS
M|17
April 11 - 12, 2017
The Conrad Hotel
New York City
https://m17.mariadb.com/

Marko Mäkelä, Lead Developer InnoDB
MariaDB Corporation