Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • O openapi-generator
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,476
    • Issues 3,476
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 402
    • Merge requests 402
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package 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
  • OpenAPI Tools
  • openapi-generator
  • Merge requests
  • !14434

Fix a few issues with the C generator (part 1 version 2)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Administrator requested to merge github/fork/eafer/c_api_fixes_1_v2 into master Jan 11, 2023
  • Overview 1
  • Commits 5
  • Pipelines 1
  • Changes 9

Created by: eafer

This is a second version of the pull request from https://github.com/OpenAPITools/openapi-generator/pull/14388. The differences with the first version are:

  • this version uses the existing implementation of object_t, as requested by @ityuhui
  • the Config.cmake.in file was cleaned up (I think it showed up in the first place because I forgot to rebuild before running the script)
  • returnTypeIsEnum was abandoned and replaced with returnProperty, following the instructions by @wing328
  • the fourth patch from https://github.com/OpenAPITools/openapi-generator/pull/14379 is also included now, because it got fixed

These patches fix a few compilation errors for this api's generated code, but they are not enough by themselves to get it to build. More patches will be submitted later.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/eafer/c_api_fixes_1_v2