Minor comment improvements

arborx · Apr 11, 2024 · 9a32bf4 · 9a32bf4
1 parent 92d5e96
commit 9a32bf4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/ArborX_BruteForce.hpp b/src/ArborX_BruteForce.hpp
@@ -85,7 +85,7 @@ class BruteForce
     // Automatically add LegacyDefaultCallback if
     //   1. A user does not provide a callback
     //   2. The index is constructed on PairValueIndex
-    //   3. The output value_type matches the index_type in the PairValueIndex
+    //   3. The output value_type is an integral type
     constexpr bool use_convenient_shortcut = []() {
       if constexpr (!Kokkos::is_view_v<std::decay_t<CallbackOrView>>)
         return false;
@@ -101,7 +101,7 @@ class BruteForce
       // Simplified way to get APIv1 result using APIv2 interface
       Details::CrsGraphWrapperImpl::queryDispatch(
           Tag{}, *this, space, Predicates{user_predicates},
-          Details::LegacyDefaultCallback{},
+          Details::LegacyDefaultCallback{}, // inject legacy callback arg
           std::forward<CallbackOrView>(callback_or_view),
           std::forward<View>(view), std::forward<Args>(args)...);
       return;

diff --git a/src/ArborX_LinearBVH.hpp b/src/ArborX_LinearBVH.hpp
@@ -109,7 +109,7 @@ class BoundingVolumeHierarchy
     // Automatically add LegacyDefaultCallback if
     //   1. A user does not provide a callback
     //   2. The index is constructed on PairValueIndex
-    //   3. The output value_type matches the index_type in the PairValueIndex
+    //   3. The output value_type is an integral type
     constexpr bool use_convenient_shortcut = []() {
       if constexpr (!Kokkos::is_view_v<std::decay_t<CallbackOrView>>)
         return false;
@@ -125,7 +125,7 @@ class BoundingVolumeHierarchy
       // Simplified way to get APIv1 result using APIv2 interface
       Details::CrsGraphWrapperImpl::queryDispatch(
           Tag{}, *this, space, Predicates{user_predicates},
-          Details::LegacyDefaultCallback{},
+          Details::LegacyDefaultCallback{}, // inject legacy callback arg
           std::forward<CallbackOrView>(callback_or_view),
           std::forward<View>(view), std::forward<Args>(args)...);
       return;