Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • L lurch
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Librem5
  • lurch
  • Merge requests
  • !4

api: Fix number of signals

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mohammed Sadiq requested to merge mohammed.sadiq/lurch:wip/sadiq/fix-warnings into librem5 Apr 14, 2020
  • Overview 5
  • Commits 1
  • Pipelines 1
  • Changes 1

We have 10 signals, not 11. Setting 11 results in reading beyond the signals defined and trying to create signals with NULL values.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: wip/sadiq/fix-warnings