Skip to content

GitLab

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

Merged
Created Oct 22, 2020 by Marco Trevisan@3v1n0🎺Developer

Function make the object more C++ friendly

  • Overview 3
  • Commits 3
  • Pipelines 7
  • Changes 3

Coming from !494 (merged). This would be best with !504 (merged), however there's no dependency.

One of the issues here (as discussed in !494 (comment 915310)) was that it was causing a memory error, the reason was that we manually free'd the Function private stuff even after that it was set as private data (in case of error).

But now it's handled properly.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: cpp-function