On Tue Jun 17 17:40:44 2025 +0000, Rémi Bernon wrote:
Any reason to include the definition for this one which only has deprecated methods, but not for `IGattDeviceService2` which I assumed you skipped for the same reason (versus `IGattDeviceService3` which has non-deprecated methods for instance)? I've reviewed the whole thing and except for minor details above it looks alright, and probably okay to merge at once to avoid the burden of splitting the changes, but again, forward declarations are sufficient in most cases, and it's probably better to only add the interfaces we need to implement, right before implementing them, to avoid writing, reviewing, and committing unnecessary cruft. The WinRT surface and verbosity makes it even more relevant (or we need a better way to do this than rewriting every IDL).
Ah, I skipped the `IGattDeviceService2` commit, as the interface relied on `BluetoothLEDevice *`, requiring a separate commit after the latter was added. Added in the latest revision, thanks.