### Summary In the Eigen doc pages and code comments there are frequent references to `DenseBase::operator()(const RowIndices&, const ColIndices&)`. It took me a good minute to find that the method I was looking for was actually in `./plugins/IndexedViewMethods.h` and injected into `DenseBase` via an include (see [here](https://gitlab.com/libeigen/eigen/-/blob/master/Eigen/src/Core/DenseBase.h?ref_type=heads#L604)). Is there a way to make this a little nicer to search for? I see that right now the method is being injected because it uses a macro to write const and non-const versions of all the indexing methods. imo that's fine, but I think adding a comment before the include that says something like "Defines DenseBase::operator()(const RowIndices&, const ColIndices&) and other index views" would make searching a little easier. (Example doc that uses method name) https://eigen.tuxfamily.org/dox/group__TutorialSlicingIndexing.html - [x] Have a plan to fix this issue. - Happy to make a tiny pr to include the comment Also a small side note I can make another issue for if people want. Right now those methods use SFINAE in the return type like the below ```cpp template<typename RowIndices, typename ColIndices> typename internal::enable_if<internal::valid_indexed_view_overload<RowIndices,ColIndices>::value && internal::traits<typename EIGEN_INDEXED_VIEW_METHOD_TYPE<RowIndices,ColIndices>::type>::ReturnAsIndexedView, typename EIGEN_INDEXED_VIEW_METHOD_TYPE<RowIndices,ColIndices>::type >::type operator()(const RowIndices& rowIndices, const ColIndices& colIndices) EIGEN_INDEXED_VIEW_METHOD_CONST ``` It would be a little clearer imo if it used a nontype template parameter for SFINAE like the below so the return type is more clear ```cpp template<typename RowIndices, typename ColIndices, internal::enable_if<internal::valid_indexed_view_overload<RowIndices,ColIndices>::value>::ReturnAsIndexedView>* = nullptr, internal::enable_if<internal::traits<typename EIGEN_INDEXED_VIEW_METHOD_TYPE<RowIndices,ColIndices>::type>* = nullptr> typename EIGEN_INDEXED_VIEW_METHOD_TYPE<RowIndices,ColIndices>::type >::type operator()(const RowIndices& rowIndices, const ColIndices& colIndices) EIGEN_INDEXED_VIEW_METHOD_CONST ``` If the nontype template parameters pass SFINAE they are `void* = nullptr` and are ignored. I'd be happy to do this and a few other cleanups if the Eigen team wants that