This bugzilla service is closed. All entries have been migrated to https://gitlab.com/libeigen/eigen

Bug 417

Summary: Map should be nested by value, right?
Product: Eigen Reporter: Gael Guennebaud <gael.guennebaud>
Component: Core - generalAssignee: Nobody <eigen.nobody>
Status: RESOLVED FIXED    
Severity: major CC: gael.guennebaud, jacob.benoit.1
Priority: Normal    
Version: 3.0   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
patch to nest Map by value none

Description Gael Guennebaud 2012-02-09 13:47:06 UTC
Created attachment 251 [details]
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.
Comment 1 Gael Guennebaud 2012-02-09 15:27:33 UTC
https://bitbucket.org/eigen/eigen/changeset/8242ba5bc94e/
changeset:   8242ba5bc94e
user:        ggael
date:        2012-02-09 15:25:42
summary:     fix bug 417: Map should be nested by value, not by reference

https://bitbucket.org/eigen/eigen/changeset/14791ccf153b/
changeset:   14791ccf153b
branch:      3.0
user:        ggael
date:        2012-02-09 15:25:42
summary:     fix bug 417: Map should be nested by value, not by reference
Comment 2 Nobody 2019-12-04 11:27:29 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to gitlab.com's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.com/libeigen/eigen/issues/417.