fix types

Author: stephen-derosa

README.md

@@ -232,8 +232,11 @@ with:
 See [docs/tools.md](docs/tools.md).
 
 # Deprecation
-> NOTE: With the official 1.0.0 release we have introduced breaking changes to previous unofficial versions in order
-to align with other LiveKit client SDKs. See [PR #143](https://github.com/livekit/client-sdk-cpp/pull/143) for the source code changes.
+Future deprecations and deprecation dates will be listed here for the next major release.
+
+#### `v1.0.0`
+>NOTE: With the official 1.0.0 release we have introduced breaking changes to previous unofficial versions in order
+to align with other LiveKit client SDKs. See the `v1.0.0` release notes for the full list of changes.
 
 ## Contributing
 PRs welcome. Issues: <https://github.com/livekit/client-sdk-cpp/issues>.

src/room.cpp

@@ -217,9 +217,9 @@ bool Room::disconnect(DisconnectReason reason) {
 
   std::shared_ptr<FfiHandle> handle;
   RoomDelegate* delegate_snapshot = nullptr;
-  std::unique_ptr<LocalParticipant> local_participant_to_cleanup;
+  std::shared_ptr<LocalParticipant> local_participant_to_cleanup;
   std::unordered_map<std::string, std::shared_ptr<RemoteParticipant>> remote_participants_to_clear;
-  std::unique_ptr<E2EEManager> e2ee_manager_to_clear;
+  std::shared_ptr<E2EEManager> e2ee_manager_to_clear;
   std::unordered_map<std::string, std::shared_ptr<TextStreamReader>> text_stream_readers_to_clear;
   std::unordered_map<std::string, std::shared_ptr<ByteStreamReader>> byte_stream_readers_to_clear;
   int listener_to_remove = 0;

src/tests/integration/test_room.cpp

@@ -100,11 +100,11 @@ TEST_F(RoomTest, UserDisconnect) {
   RoomOptions options;
   ASSERT_TRUE(room.connect(server_url_, token_, options)) << "connect failed";
   ASSERT_EQ(room.connectionState(), ConnectionState::Connected);
-  ASSERT_NE(room.localParticipant(), nullptr);
+  ASSERT_NE(room.localParticipant().lock(), nullptr);
 
   EXPECT_NO_THROW(room.disconnect()) << "disconnect should not throw on a connected room";
   EXPECT_EQ(room.connectionState(), ConnectionState::Disconnected);
-  EXPECT_EQ(room.localParticipant(), nullptr) << "local participant should be cleared after disconnect";
+  EXPECT_EQ(room.localParticipant().lock(), nullptr) << "local participant should be cleared after disconnect";
   EXPECT_EQ(delegate.count.load(), 1) << "onDisconnected should fire exactly once";
   EXPECT_EQ(delegate.last_reason, DisconnectReason::ClientInitiated);