Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
  • Sign in
A
aml
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 27
    • Issues 27
    • List
    • Boards
    • Labels
    • Milestones
  • Merge Requests 6
    • Merge Requests 6
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • argo
  • aml
  • Issues
  • #27

Closed
Open
Opened Mar 13, 2019 by Swann Perarnau@perarnau
  • Report abuse
  • New issue
Report abuse New issue

Finish splitting the headers

Now that we have reorganized the aml headers, make sure that we actually split all the building blocks into separate headers. We should end up with:

  • include/aml.h for the core
  • include/aml/buildingblock.h for each main abstraction
  • include/aml/buildingblock/specific_implementation.h for each implementation of a building block

At this point, I think having aml.h include the main abstraction building block would be good, but maybe not include all the implementations inside each buildingblock.h

Fixed in !44 (merged).

Edited Mar 29, 2019 by Swann Perarnau
Assignee
Assign to
First Official Release
Milestone
First Official Release
Assign milestone
Time tracking
None
Due date
None
0
Labels
None
Assign labels
  • View project labels
Reference: argo/aml#27