Summary: | blueNorm() implementation produces an interesting warning from clang++ | ||
---|---|---|---|
Product: | Eigen | Reporter: | Kyle M. Tarplee <kyle.tarplee> |
Component: | Geometry | Assignee: | Nobody <eigen.nobody> |
Status: | RESOLVED FIXED | ||
Severity: | Unknown | CC: | gael.guennebaud, hauke.heibel, jacob.benoit.1 |
Priority: | Normal | ||
Version: | 3.2 | ||
Hardware: | x86 - 64-bit | ||
OS: | All | ||
Whiteboard: | |||
Bug Depends on: | |||
Bug Blocks: | 387 |
Description
Kyle M. Tarplee
2013-04-08 23:13:14 UTC
The code for 3.2 also has this problem. Indeed, that might look scary, but fortunately this piece of code was actually useless, and that's why this mistake did not show up in our unit test. Thanks for the report. https://bitbucket.org/eigen/eigen/commits/72a7ce1bca6d/ Changeset: 72a7ce1bca6d User: ggael Date: 2013-04-09 09:23:40 Summary: Fix bug 581: remove useless piece of code is blueNorm https://bitbucket.org/eigen/eigen/commits/83ccf6713af3/ Changeset: 83ccf6713af3 Branch: 3.1 User: ggael Date: 2013-04-09 09:23:40 Summary: Fix bug 581: remove useless piece of code is blueNorm (transplanted from 72a7ce1bca6d88fb4bc788be628925bebdf0e037) -- 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/581. |