Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
gstreamer
gstreamer
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 409
    • Issues 409
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 83
    • Merge Requests 83
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GStreamer
  • gstreamergstreamer
  • Merge Requests
  • !729

Open
Opened Jan 10, 2021 by Marijn Suijten@MarijnS95🦀Contributor
  • Report abuse
Report abuse

allocator: Make GstAllocationParams function parameter const

  • Overview 19
  • Commits 3
  • Pipelines 8
  • Changes 11

For Rust we'd really like G-IR annotations to be on point to save us from overriding mutability of various parameters, and at the same time enforcing the same rules here in C. All uses of GstAllocationParams in plugins-base have been consistently checked and converted to const, for which the same has to be done here in GstAllocator::alloc and a few other functions.

This was brought up while working on GLFilter bindings. CC @slomo.

Can we get this backported to 1.18? I think @slomo said changing signatures like this is "fine", but it may cause code to emit discarded-qualifiers and incompatible-pointer-types warnings (as it does currently with the gst-plugins-* repos), eventually as errors if enabled. It'd be nice for us to import these changes to gstreamer-rs though.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Reference: gstreamer/gstreamer!729
Source branch: parameter-const-correctness