Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • gst-plugins-base gst-plugins-base
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 644
    • Issues 644
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 88
    • Merge requests 88
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GStreamer
  • gst-plugins-basegst-plugins-base
  • Merge requests
  • !86

video-converter: fix number of allocated lines

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nicolas Dufresne requested to merge ndufresne/gst-plugins-base:350-video-convert-fix-number-of-allocated-lines into master Jan 11, 2019
  • Overview 5
  • Commits 1
  • Pipelines 1
  • Changes 1

We make an allocator for temporary lines and then use this for all the steps in the conversion that can do in-place processing.

Keep track of the number of lines each step needs and use this to allocate the right number of lines.

Previously we would not always allocate enough lines and we would end up with conversion errors as lines would be reused prematurely.

Fixes #350 (closed)

Closes #350 (closed)

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: 350-video-convert-fix-number-of-allocated-lines