winrt-rust icon indicating copy to clipboard operation
winrt-rust copied to clipboard

Fix type for `HttpProgress` struct

Open Boddlnagg opened this issue 5 years ago • 0 comments

I just stumbled upon the fact that HttpProgress seems to be the only struct that contains reference types as members (Windows.Foundation.IReference<u64>).

Currently this is projected as *mut windows::foundation::IReference<u64>, but we probably don't want to have a raw pointer here. I think that ComPtr<windows::foundation::IReference<u64>> would be correct (i.e. when we get an instance of that struct, the members have their reference count incremented, and we have to Release() the interfaces when we drop the struct), but I'm not 100% sure.

Boddlnagg avatar May 20 '19 20:05 Boddlnagg