Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
VTK-m
VTK-m
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 150
    • Issues 150
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 21
    • Merge Requests 21
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI / CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • VTK
  • VTK-mVTK-m
  • Merge Requests
  • !2262

Merged
Opened Sep 14, 2020 by Kenneth Moreland@kmorelMaintainer

Remove ColorTable.hxx

  • Overview 8
  • Commits 2
  • Pipelines 4
  • Changes 10

The ColorTable header was split into two files: the normal ColorTable.h header and a secondary ColorTable.hxx that contains the ColorTable "implementation." This is so you can include ColorTable.h without necessarily having to use a device compiler.

However, this implementation is confusing. So ColorTable.hxx has been deleted and moved elsewhere. This means that the Sample method implementation is moved to ColorTable.cxx. Compiling these methods into the library makes more sense anyway. It also means that the Map methods have been moved to free functions and placed in a ColorTableMap.h header.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Reference: vtk/vtk-m!2262
Source branch: no-colortable-hxx