Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
P
poppler
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 613
    • Issues 613
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 40
    • Merge Requests 40
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • poppler
  • poppler
  • Merge Requests
  • !16

The source project of this merge request has been removed.
Merged
Opened Aug 26, 2018 by Adam Reichold@adamreicholdContributor

RFC: Reimplement GooList as a subtype of std::vector

  • Overview 0
  • Changes 23

This reimplements GooList as a subtype of std::vector<void *> to facilitate porting the various places where GooList is used to STL containers. After this change, this can happen mechanically by replacing all of the GooList-specific API by their implementations at the call sites and cleaning up until at some point no more GooList-specific API is left and the type can be removed.

This of course keeps the normally inefficient pattern of storing pointer to objects instead of objects inside these containers which however can then be refactoring in a separate step building on top this and as a pure optimization.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Reference: poppler/poppler!16
Source branch: goo-list-adapter