Skip to content

add user_data trailer type#1180

Open
chenosaurus wants to merge 14 commits into
mainfrom
dc/fm_user_data
Open

add user_data trailer type#1180
chenosaurus wants to merge 14 commits into
mainfrom
dc/fm_user_data

Conversation

@chenosaurus

Copy link
Copy Markdown
Contributor

No description provided.

/// (`PTF_USER_TIMESTAMP`, `PTF_FRAME_ID`, `PTF_USER_DATA`), so individual fields
/// are `Option`.
#[derive(Debug, Clone)]
pub struct FrameMetadata {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: Could we define a helper on FrameMetadata so the user can calculate their exact max user_data length based on the other fields they have set?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if FrameMetadata would be able to calculate it since all the fields get set at the same time usually. Otherwise it would need to know about the Features that were enabled on the TrackPublishOptions.

Base automatically changed from dc/frame_metadata_rename to main June 20, 2026 20:17
@github-actions

github-actions Bot commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Changeset

The following package versions will be affected by this PR:

Package Bump
livekit minor
livekit-ffi minor
livekit-protocol minor

@chenosaurus chenosaurus requested a review from pblazej as a code owner June 22, 2026 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants