Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • libhandy libhandy
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 131
    • Issues 131
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 19
    • Merge requests 19
  • 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

Due to an influx of spam, we have had to impose restrictions on new accounts. Please see this wiki page for instructions on how to get full permissions. Sorry for the inconvenience.

  • Librem5
  • libhandylibhandy
  • Merge requests
  • !200

WIP: Fail on clang's static analyzer errors

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Guido Gunther requested to merge guido.gunther/libhandy:meson-clang into master Jan 16, 2019
  • Overview 2
  • Commits 4
  • Pipelines 1
  • Changes 2

While we run the static analyzer warnings weren't fatal. Fix this.

We can't merge it yet since it fails like

[16/58] Compiling C object 'src/25a6634@@handy-0.0@sha/hdy-combo-row.c.o'.
../../../src/hdy-combo-row.c:447:1: warning: Potential leak of memory pointed to by 'data'
}
^
1 warning generated.
[18/58] Compiling C object 'src/25a6634@@handy-0.0@sha/hdy-dialer-button.c.o'.
../../../src/hdy-dialer-button.c:57:3: warning: Potential leak of memory pointed to by 'text'
  gtk_label_set_label (priv->secondary_label, secondary_text);
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

Which are false positives AFAIKT. Seems Qt has similar issues and we need to teach the analyzer about g_autofree and gtk_list_box_bind_model.

Maybe someone with a crush on clang wants to pick this up?

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: meson-clang