]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Improve grammar in error messages. 14655/head
authorWolfgang Bangerth <bangerth@colostate.edu>
Tue, 10 Jan 2023 04:14:05 +0000 (21:14 -0700)
committerWolfgang Bangerth <bangerth@colostate.edu>
Tue, 10 Jan 2023 04:14:05 +0000 (21:14 -0700)
include/deal.II/base/array_view.h
tests/base/array_view_09.cc
tests/base/array_view_09.output

index 168310404e1716c1c36679d98f43775334e1b14b..deb64f89139b7fa4f3566e3f3f6825136c586bf7 100644 (file)
@@ -655,10 +655,10 @@ make_array_view(const Iterator begin, const Iterator end)
   static_assert(
     std::is_same<typename std::iterator_traits<Iterator>::iterator_category,
                  typename std::random_access_iterator_tag>::value,
-    "The provided iterator should be a random access iterator.");
+    "The provided iterator needs to be a random access iterator.");
   Assert(begin <= end,
          ExcMessage(
-           "The beginning of the array view should be before the end."));
+           "The beginning of the array view needs to be before the end."));
   Assert(internal::ArrayViewHelper::is_contiguous(begin, end),
          ExcMessage("The provided range isn't contiguous in memory!"));
   // the reference type, not the value type, knows the constness of the iterator
@@ -684,7 +684,7 @@ make_array_view(ElementType *const begin, ElementType *const end)
 {
   Assert(begin <= end,
          ExcMessage(
-           "The beginning of the array view should be before the end."));
+           "The beginning of the array view needs to be before the end."));
   return ArrayView<ElementType, MemorySpaceType>(begin, end - begin);
 }
 
index 36bee2e25c061b0871b89f22652952b57eea6d1f..102ce8ac46ff5aeea62abecde86a43b477843a1e 100644 (file)
@@ -63,9 +63,9 @@ test()
     const std::array<double, 10> &v2 = v;
     const auto                    a  = make_array_view(v2.begin(), v2.end());
     static_assert(is_const_reference<decltype(*a.begin())>(),
-                  "type should be const");
+                  "type needs to be const");
     static_assert(is_const_reference<decltype(*a.end())>(),
-                  "type should be const");
+                  "type needs to be const");
 
     v[5] = 10;
     AssertThrow(a[5] == 10, ExcInternalError());
@@ -76,15 +76,15 @@ test()
     std::vector<double> v(10);
     std::fill(v.begin(), v.end(), 42.0);
     auto a = make_array_view(v.cbegin() + 2, v.cend());
-    // a should be ArrayView<const double>
+    // a needs to be ArrayView<const double>
     static_assert(!std::is_const<decltype(a)>::value,
                   "a should not be const (but has const value)");
     static_assert(std::is_const<decltype(a)::value_type>::value,
-                  "a::value_type should be const");
+                  "a::value_type needs to be const");
     static_assert(is_const_reference<decltype(*a.begin())>(),
-                  "type should be const");
+                  "type needs to be const");
     static_assert(is_const_reference<decltype(*a.end())>(),
-                  "type should be const");
+                  "type needs to be const");
     v[2] = 10.0;
     AssertThrow(a[0] == v[2], ExcInternalError());
   }
@@ -100,15 +100,15 @@ test()
     // static_vector::cbegin() and static_vector::cend() correctly, so ignore
     // the type checking in that case
 #if BOOST_VERSION >= 106200
-    // a should be const ArrayView<const double>
+    // a needs to be const ArrayView<const double>
     static_assert(std::is_const<decltype(a)>::value,
                   "a should not be const (but has const value)");
     static_assert(std::is_const<decltype(a)::value_type>::value,
-                  "a::value_type should be const");
+                  "a::value_type needs to be const");
     static_assert(is_const_reference<decltype(*a.begin())>(),
-                  "type should be const");
+                  "type needs to be const");
     static_assert(is_const_reference<decltype(*a.end())>(),
-                  "type should be const");
+                  "type needs to be const");
 #endif
     v[2] = 10.0;
     AssertThrow(a[0] == v[2], ExcInternalError());
@@ -127,10 +127,10 @@ test()
       // to make CTest happy
 #ifndef DEBUG
     deallog
-      << "ExcMessage(\"The beginning of the array view should be before the end.\")"
+      << "ExcMessage(\"The beginning of the array view needs to be before the end.\")"
       << std::endl;
     deallog
-      << "ExcMessage(\"The beginning of the array view should be before the end.\")"
+      << "ExcMessage(\"The beginning of the array view needs to be before the end.\")"
       << std::endl;
 #endif
   }
index 78b054254ca0cf320865b8808f69081ce5626c53..c5e1b02eb29f14aec78281f5568d45098285e0bb 100644 (file)
@@ -1,4 +1,4 @@
 
-DEAL::ExcMessage("The beginning of the array view should be before the end.")
-DEAL::ExcMessage("The beginning of the array view should be before the end.")
+DEAL::ExcMessage("The beginning of the array view needs to be before the end.")
+DEAL::ExcMessage("The beginning of the array view needs to be before the end.")
 DEAL::OK

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.