HRCPP-278 added missing method removeWithVersionAsync

[rigazilla]

added method and a little fix

[mgencur]

Can you add a simple test, please?

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL Build 637 for 8113c4d
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:05:41

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 Build 498 for 8113c4d
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:07:29

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL Build 641 for 8113c4d
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:05:45

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 Build 504 for 8113c4d
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:07:32

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL Build 648 for 6182f02
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:05:47

[infinispanrelease]

SUCCESS - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 Build 513 for 6182f02
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:09:04

[rvansa]

No objections, seems like just following the principle set up in the original implementation. Is there a test for goAsync?

[rigazilla]

@rvansa In the Simple.cpp file there's a test for the get/put Async that generates the code from the goAsync template.

@mgencur with the last commit I'm counting 19 Async methods. If we want a test for each ones I think it's better to track this as a separate issue.

[rvansa]

OK, I am content with this PR.

[slaskawi]

Looks good to me!

[tristantarrant]

Pushed to master, thanks