Patch coming.
Created attachment 24656 [details] patch
This looks fine to me, but you'll need to find someone legit to agree – unofficial r+! :o) My only concerns would be in Platform.h – it looks like the patch attached would turn on sampling as default, though I'm assuming this is not your intention when you land it. Also, the way you have structured the ifdefs it looks like ENABLE_OPCODE_SAMPLING can be defined twice – some compilers will warn on this, so I don't know if you want to restructure the ifdefs to avoid this. G.
> My only concerns would be in Platform.h – it looks like the patch attached > would turn on sampling as default, though I'm assuming this is not your > intention when you land it. Oops! Fixed. > Also, the way you have structured the ifdefs it > looks like ENABLE_OPCODE_SAMPLING can be defined twice – some compilers will > warn on this, so I don't know if you want to restructure the ifdefs to avoid > this. Restructured. Thanks!
Committed revision 37891.