Merge pull request #5299 from cherusker/cherusker-2017-08-01-next-generic-inst-id
authormonojenkins <jo.shields+jenkins@xamarin.com>
Wed, 2 Aug 2017 23:10:32 +0000 (01:10 +0200)
committerGitHub <noreply@github.com>
Wed, 2 Aug 2017 23:10:32 +0000 (01:10 +0200)
commit8bbca2bada9398b3a5ddfec0c3c890054c1e8bdc
tree1eb93eb3c2d0331c6b32ee32b51891fe5b94fd33
parent266c991d6f9ebc9b2b096f10a3b7a0f9899f3fd3
parent47535f557d18cf771436ac55578dfb165da0e399
Merge pull request #5299 from cherusker/cherusker-2017-08-01-next-generic-inst-id

[metadata] Fix racy next_generic_inst_id

Like suggested by @luhenry, I would like to propose this fix which was discussed in https://bugzilla.xamarin.com/show_bug.cgi?id=58423. As `next_generic_inst_id` is `int`, `InterlockedIncrement64 ()` seems to be the right choice.

Also, I cannot add any reviewers or assignees, which is why I would kindly ask @lambdageek to review this as well :)