Inline IndexedViewMethods.inc into DenseBase.h

Pavel Guzenfeldrequested to merge
PavelGuzenfeld/eigen:refactor/inline-indexed-view-methods into master

Summary

Inline the contents of Eigen/src/plugins/IndexedViewMethods.inc directly into DenseBase.h and remove the .inc file.

  • Makes the indexed view operator() overloads discoverable directly in DenseBase.h rather than hidden behind a plugin include
  • The file was only included from DenseBase.h (confirmed by grep)
  • All code, SFINAE style (return-type SFINAE), EIGEN_PARSED_BY_DOXYGEN guards, and Doxygen documentation preserved exactly as-is
  • Net change: -193 lines (deleted .inc) / +185 lines (inlined into DenseBase.h) = -8 lines total

Test plan

Local Docker (GCC 15, CMake 3.31, Debian trixie, C++17):

Test suite Tests Result
indexed_view 1 variant ✅ pass
stl_iterators 1 variant ✅ pass
block 12 variants + 10 fail tests ✅ 22/22 pass
reshape 1 variant ✅ pass
mapped_matrix 4 variants ✅ 4/4 pass
redux 4 variants ✅ 4/4 pass
vectorwiseop 7 variants ✅ 7/7 pass
array_cwise 31 variants ✅ 31/31 pass
product_small 43 variants ✅ 43/43 pass
product_large 9 variants ✅ 9/9 pass
basicstuff 8 variants ✅ 7/8 pass (1 pre-existing flaky)
Total 139 tests 138/139 pass

The single failure (basicstuff_8) is a known pre-existing flaky test (random-seed-dependent casting comparison) unrelated to this change.

Closes #2766

Edited by Pavel Guzenfeld

Merge request reports