rtc: don't put ffi_drop_handle side effect inside assert

With python -O asserts are stripped, which would skip the drop and leak
the native handle. Call drop_handle unconditionally and assert the result
separately.

Author: theomonnom

livekit-rtc/livekit/rtc/_ffi_client.py

@@ -85,7 +85,10 @@ def dispose(self) -> None:
             self._disposed = True
             ffi = FfiClient._owned_instance()
             if ffi is not None:
-                assert ffi._ffi_lib.livekit_ffi_drop_handle(ctypes.c_uint64(self.handle))
+                # Keep the drop out of assert(): with `python -O` asserts are
+                # stripped, which would skip the call and leak the handle.
+                dropped = ffi._ffi_lib.livekit_ffi_drop_handle(ctypes.c_uint64(self.handle))
+                assert dropped
 
     def __repr__(self) -> str:
         return f"FfiHandle({self.handle})"