Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
  • eigen eigen
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 649
    • Issues 649
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Custom issue tracker
    • Custom issue tracker
  • Merge requests 22
    • Merge requests 22
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • libeigen
  • eigeneigen
  • Issues
  • #417
Closed
Open
Created Dec 04, 2019 by Eigen Bugzilla@eigenbzReporter

Map should be nested by value, right?

Submitted by Gael Guennebaud @ggael

Assigned to Nobody

Link to original bugzilla bug (#417)
Version: 3.0

Description

Created attachment 251
patch to nest Map by value

Currently Map objects are nested by reference while it seems to me that they should be nested by value just like Block<> are currently nested by value.

The attached patch fix that. Please tell me if I'm overseeing something.

On a side note, using HereditaryBits instead of ~NestedByRefBit for the mask generates many compilation errors... That's why I currently propose to only remove that bit for now.

Attachment 251, "patch to nest Map by value":
MapShouldBeNestedByValue.diff

Edited Dec 05, 2019 by Eigen Bugzilla
Assignee
Assign to
Time tracking