FFI: implement Room::list_threads [help wanted]

[viktorstrate]

Expose Room::list_threads to the FFI.

I am new to this code base, so I'm trying to understand how all the pieces fit together.
I want to use Room::list_threads from Swift, so this pull request attempts to expose this function through the FFI.

The code currently does not compile, because I don't know how to convert the SDK version of TimelineEvent obtained from the self.inner.list_threads(..), to a compatible FFI version.
Can anyone with more experience guide me on what to do here?

• Public API changes documented in changelogs (optional)

Signed-off-by: Viktor Strate [email protected]

[viktorstrate]

This line fails to compile

[andybalaam]

Welcome! Thanks for working on this. I managed to get it to compile with this change:

$ git diff
diff --git a/bindings/matrix-sdk-ffi/src/room/mod.rs b/bindings/matrix-sdk-ffi/src/room/mod.rs
index 29ca13b5ef..e4daa65e80 100644
--- a/bindings/matrix-sdk-ffi/src/room/mod.rs
+++ b/bindings/matrix-sdk-ffi/src/room/mod.rs
@@ -1243,7 +1243,17 @@ impl Room {
         let roots = self.inner.list_threads(inner_opts).await?;
 
         Ok(ThreadRoots {
-            chunk: roots.chunk.into_iter().map(|x| x.into()).collect(),
+            chunk: roots
+                .chunk
+                .into_iter()
+                .filter_map(|timeline_event| {
+                    timeline_event
+                        .raw()
+                        .deserialize()
+                        .ok()
+                        .map(|any_timeline_event| TimelineEvent(Box::new(any_timeline_event)))
+                })
+                .collect(),
             prev_batch_token: roots.prev_batch_token,
         })
     }

This solves the problem of how to convert from a matrix_sdk_common::deserialized_responses::TimelineEvent into a matrix_sdk_ffi::event::TimelineEvent (oh how I hate things with the same name!) by using the cheat/trick of converting it to JSON (.raw()) and then parsing it (.deserialize). It then ignores any errors in that process by converting errors to None (.ok()) and removing from the list with filter_map.

So it would be worth asking in the SDK development Matrix room whether this is the best way to go about it, and whether it's OK to ignore errors here.

[codspeed-hq]

CodSpeed Performance Report

Merging #5953 will not alter performance

_{Comparing viktorstrate:ffi-list-threads (b2a31df) with main (9a6acd5)}

Summary

✅ 50 untouched