Re: [PATCH v5 0/5] MR8654: windows.devices.bluetooth: Add stubs for BluetoothLEAdvertisementWatcher

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.bluetooth/tests/bluetooth.c:
> - hr = IBluetoothLEAdvertisementWatcher_get_Status( watcher, &status );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> - todo_wine ok( status == BluetoothLEAdvertisementWatcherStatus_Created, "got status %u.\n", status );
> -
> - mode = BluetoothLEScanningMode_None;
> - hr = IBluetoothLEAdvertisementWatcher_get_ScanningMode( watcher, &mode );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> - todo_wine ok( mode == BluetoothLEScanningMode_Passive, "got status %u.\n", status );
> -
> - hr = IBluetoothLEAdvertisementWatcher_get_SignalStrengthFilter( watcher, &sig_filter );
> - todo_wine ok( hr == S_OK, "got hr %#lx.\n", hr );
> - todo_wine ok( !!sig_filter, "got sig_filter %p.\n", sig_filter );
> - if (SUCCEEDED( hr ))
> - {
> -
>    IReference_INT16 *int16;
> -
>    IReference_TimeSpan *span;

I'd suggest to move these to the top, (and change the `span` variable name as it shadows the other one). When the todo_wine is going to be removed it would be nice to remove the if too.