Improve formatting and comments

Author: akukanov

include/oneapi/dpl/pstl/glue_algorithm_ranges_impl.h

@@ -410,7 +410,7 @@ struct __contains_fn
     bool
     operator()(_ExecutionPolicy&& __exec, _R&& __r, const _T& __value, _Proj __proj = {}) const
     {
-        // To ensure no dangling pointer is returned, __r may not be forwarded
+        // To ensure no dangling iterator is returned, __r may not be forwarded
         return oneapi::dpl::ranges::find(std::forward<_ExecutionPolicy>(__exec), __r, __value, __proj)
                != std::ranges::end(__r);
     }
@@ -429,8 +429,8 @@ struct __contains_subrange_fn
                _Proj2 __proj2 = {}) const
     {
         // To ensure no dangling subrange is returned, __r1 may not be forwarded
-        return __r2.size() == 0 || ! oneapi::dpl::ranges::search(std::forward<_ExecutionPolicy>(__exec), __r1, __r2,
-                                                                 __pred, __proj1, __proj2).empty();
+        return __r2.size() == 0 || !(oneapi::dpl::ranges::search(std::forward<_ExecutionPolicy>(__exec), __r1, __r2,
+                                                                 __pred, __proj1, __proj2).empty());
     }
 }; // __contains_subrange_fn
 } // __internal

test/parallel_api/ranges/std_ranges_contains_subrange.pass.cpp

@@ -17,7 +17,6 @@
     auto checker = TEST_PREPARE_CALLABLE(std::ranges::contains_subrange);
 #else
     struct {
-        
         template<std::ranges::forward_range R1, std::ranges::forward_range R2, typename Pred = std::ranges::equal_to,
                  typename Proj1 = std::identity, typename Proj2 = std::identity>
         bool operator()(R1&& r1, R2&& r2, Pred pred = {}, Proj1 proj1 = {}, Proj2 proj2 = {})
@@ -38,10 +37,10 @@ main()
     using namespace test_std_ranges;
     namespace dpl_ranges = oneapi::dpl::ranges;
 
-    auto lam = [](auto i) { return i + 371; };
-    using data_gen_shifted = decltype(lam);
+    auto shift = [](auto i) { return i + 371; };
+    using data_gen_shifted = decltype(shift);
 
-    // Sizes of both sequences will vary in the test, so each call might test both successful and unsuccessful searches
+    // Sizes of both sequences vary in the test, so each call might test both successful and unsuccessful searches
     launcher<0, int>{big_sz}(dpl_ranges::contains_subrange, checker, binary_pred_const);
     launcher<1, int>{}(dpl_ranges::contains_subrange, checker);
     launcher<2, int>{}(dpl_ranges::contains_subrange, checker, binary_pred, dpl::identity{});