Skip to content
Snippets Groups Projects

demux: subtitle: optimize to avoid strcat, strlen, allocation when appending

Merged Steve Lhomme requested to merge robUx4/vlc:demux-subs-append into master
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Denis Charmet resolved all threads

    resolved all threads

  • MR Acceptance result

    :tada: This MergeRequest has been Accepted! Congratulations.

    MR acceptance checks details:

    • :white_check_mark: MR should be considered mergeable by Gitlab
    • :white_check_mark: Last pipeline should be successful
    • :white_check_mark: MergeRequest should have at least one external review and/or vote
    • :white_check_mark: All threads should be resolved, and score >= 0
    • :white_check_mark: MergeRequest should have no activity (threads/votes) for (24h/24h)

    This message was automatically generated by homer-bot.

  • Steve Lhomme added 41 commits

    added 41 commits

    • 25a077d7...46cacbe0 - 36 commits from branch videolan:master
    • 3f110c95 - demux: subtitle: only allocate internal psz_text when needed
    • e8f7ac27 - demux: subtitle: avoid using strcat when we know the string sizes
    • 1e03b716 - demux: subtitle: keep the size of the concatenated string
    • de7e5074 - demux: subtitle: avoid allocating a dummy string
    • 1f9b8484 - demux: subtitle: only set the string terminator on loop exit

    Compare with previous version

  • Steve Lhomme enabled an automatic merge when all merge checks for 1f9b8484 pass

    enabled an automatic merge when all merge checks for 1f9b8484 pass

  • merged

  • Steve Lhomme mentioned in merge request !7012 (merged)

    mentioned in merge request !7012 (merged)

  • Please register or sign in to reply
    Loading