Closed kejxu closed 5 years ago
thanks for approving the change @sloretz! is there anything else we would need before merging?
@sloretz Ping! I saw this got approved but it is still open (#27, #28 and #29). Let @kejxu or me know anything from us to make them merged back to the upstream. Thanks!
Looks good, merging.
update deprecated macro to
__declspec(deprecated(msg))
for MSVC