|Summary:||Make headers self-containing|
|Product:||Eigen||Reporter:||Christoph Hertzberg <chtz>|
|Component:||Core - general||Assignee:||Nobody <eigen.nobody>|
|Severity:||Unknown||CC:||andy.somerville, gael.guennebaud, jacob.benoit.1, jitseniesen|
|Bug Depends on:|
|Bug Blocks:||138, 728, 269|
Description Christoph Hertzberg 2011-08-31 10:59:54 UTC
I would very much appreciate if Eigen-headers would be self containing, i.e. * Have each of them declare the appropriate namespaces. * Make each of them include the necessary dependencies. Most important advantage for me would be to simplify browsing the code, currently my IDE (eclipse CDT) is (almost) perfectly capable to do syntax completion with Eigen types in my own code, however as soon as I want to look into the internals of a method, I have to do a lot of guessing in which header the actual implementation might be. The other option to find that out would be debugging in O0-mode, however, at some point you can easily get lost there as well. If at least the namespace declarations could be done globally, I could locally insert the necessary dependencies on a need-to-work basis (this obviously does not work currently, especially not with external libraries). This would also solve bug 269 at almost no cost.
Comment 1 Christoph Hertzberg 2011-09-26 13:05:13 UTC
Added blocking of bug 138 (at least to force a decision until 3.1)
Comment 2 Jitse Niesen 2012-04-15 13:10:38 UTC
The (easy) first part, to have each header file declare the Eigen namespace, is now done (changeset 0cf501b7d3dd). For some reason, doxygen had to be instructed to parse all the .h files so probably there are some changes in the documentation it produces but at a first glance everything still seems to be there.
Comment 3 Jitse Niesen 2012-07-17 19:14:55 UTC
*** Bug 13 has been marked as a duplicate of this bug. ***
Comment 4 Nobody 2019-12-04 11:05:27 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/339.