Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • apk-tools apk-tools
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 93
    • Issues 93
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 19
    • Merge requests 19
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and 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
  • alpinealpine
  • apk-toolsapk-tools
  • Merge requests
  • !15

Improve type discipline and target C18

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Reid Rankin requested to merge reidrankin/apk-tools:reidrankin-c18 into master Jan 25, 2020
  • Overview 10
  • Commits 10
  • Pipelines 0
  • Changes 42

I've gone through the codebase and done some cleanup work:

  • made typecasts, especially to and from signed integers, explicit
  • change GNU-specific language extensions into their annotated forms
  • a few name changes to avoid ambiguity (for example, avoiding 'new' and 'delete' since c++ uses those as reserved words)
  • changing the compilation target from gnu11 to c18 (saves about 5KB off the stripped binary size)

None of this should change existing behavior, but should hopefully help maintainability.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: reidrankin-c18