Skip to content

Other minor fixes to remove unused headers and correct some comments#108

Merged
alan-george-lk merged 1 commit intolivekit:mainfrom
siddimore:other-minor-typo-fixes
Apr 23, 2026
Merged

Other minor fixes to remove unused headers and correct some comments#108
alan-george-lk merged 1 commit intolivekit:mainfrom
siddimore:other-minor-typo-fixes

Conversation

@siddimore
Copy link
Copy Markdown
Contributor

@siddimore siddimore commented Apr 23, 2026

Summary

This PR applies a small set of non-invasive fixes in the C++ SDK.
All changes are documentation/comment corrections or include hygiene improvements, with no intended runtime behavior changes.

Changes

  • Changed RemoteTrackPublication to LocalTrackPublication in the constructor note.
  • Removed empty private section in remote_video_track.h
  • Removed unused iostream include from track.h
    This avoids unnecessarily pulling a heavy standard header through a widely included public header.
  • Fixed VideoStream usage example in video_stream.h
    Updated sample snippet to use VideoStream types and symbols instead of AudioStream names.
  • Removed unused include in remote_audio_track.cpp
  • Removed unused include in remote_video_track.cpp

Notes

No functional logic was changed.
No API signatures were changed.
Changes are limited to comments, examples, access-section cleanup, and unused includes.

@siddimore siddimore marked this pull request as ready for review April 23, 2026 00:24
@alan-george-lk alan-george-lk merged commit 1bc08ab into livekit:main Apr 23, 2026
13 checks passed
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