]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Reword a comment in make_array_view. 6147/head
authorDavid Wells <wellsd2@rpi.edu>
Tue, 3 Apr 2018 11:20:17 +0000 (07:20 -0400)
committerDavid Wells <wellsd2@rpi.edu>
Tue, 3 Apr 2018 11:20:17 +0000 (07:20 -0400)
The term 'undefined' is not accurate; the order *is* defined, but it is an
implementation detail.

From this stack overflow post:
https://stackoverflow.com/questions/49071748/data-alignment-and-ordering-in-recursively-defined-array/49627551#49627551

include/deal.II/base/array_view.h

index 50ffacbb00818c5e5cd8aca06c0f83418cc72052..60f836de87bae3ad0593d95f7bb3383be2c72776 100644 (file)
@@ -575,8 +575,8 @@ make_array_view (ArrayView<Number> &array_view)
  *
  * @param[in] vector The Tensor for which we want to have an array view
  * object. The array view corresponds to the <em>entire</em> object but the
- * order in which the entries are presented in the array is undefined and can
- * not be relied upon.
+ * order in which the entries are presented in the array is an implementation
+ * detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -601,8 +601,8 @@ make_array_view (const Tensor<rank, dim, Number> &tensor)
  *
  * @param[in] vector The Tensor for which we want to have an array view
  * object. The array view corresponds to the <em>entire</em> object but the
- * order in which the entries are presented in the array is undefined and can
- * not be relied upon.
+ * order in which the entries are presented in the array is an implementation
+ * detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -627,8 +627,8 @@ make_array_view (Tensor<rank, dim, Number> &tensor)
  *
  * @param[in] vector The SymmetricTensor for which we want to have an array
  * view object. The array view corresponds to the <em>entire</em> object but
- * the order in which the entries are presented in the array is undefined and
- * can not be relied upon.
+ * the order in which the entries are presented in the array is an
+ * implementation detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -652,10 +652,10 @@ make_array_view (const SymmetricTensor<rank, dim, Number> &tensor)
  * Consequently, the return type of this function is a view to a set of writable
  * objects.
  *
- * @param[in] vector The SymmetricTensor for which we want to have an array view
- * object. The array view corresponds to the <em>entire</em> object but the
- * order in which the entries are presented in the array is undefined and can
- * not be relied upon.
+ * @param[in] vector The SymmetricTensor for which we want to have an array
+ * view object. The array view corresponds to the <em>entire</em> object but
+ * the order in which the entries are presented in the array is an
+ * implementation detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -897,8 +897,8 @@ make_array_view (Table<2,ElementType>                           &table,
  *
  * @param[in] table The Table for which we want to have an array view object.
  * The array view corresponds to the <em>entire</em> table but the order in
- * which the entries are presented in the array is undefined and can not be
- * relied upon.
+ * which the entries are presented in the array is an implementation detail
+ * and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -923,8 +923,8 @@ make_array_view (Table<2,ElementType> &table)
  *
  * @param[in] table The Table for which we want to have an array view object.
  * The array view corresponds to the <em>entire</em> table but the order in
- * which the entries are presented in the array is undefined and can not be
- * relied upon.
+ * which the entries are presented in the array is an implementation detail
+ * and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -947,10 +947,10 @@ make_array_view (const Table<2,ElementType> &table)
  * Consequently, the return type of this function is a view to a set of
  * @p non-const objects.
  *
- * @param[in] table The LAPACKFullMatrix for which we want to have an array view
- * object. The array view corresponds to the <em>entire</em> object but the
- * order in which the entries are presented in the array is undefined and can
- * not be relied upon.
+ * @param[in] table The LAPACKFullMatrix for which we want to have an array
+ * view object. The array view corresponds to the <em>entire</em> object but
+ * the order in which the entries are presented in the array is an
+ * implementation detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */
@@ -973,10 +973,10 @@ make_array_view (LAPACKFullMatrix<ElementType> &matrix)
  * type because they contain immutable elements. Consequently, the return type
  * of this function is a view to a set of @p const objects.
  *
- * @param[in] table The LAPACKFullMatrix for which we want to have an array view
- * object. The array view corresponds to the <em>entire</em> object but the
- * order in which the entries are presented in the array is undefined and can
- * not be relied upon.
+ * @param[in] table The LAPACKFullMatrix for which we want to have an array
+ * view object. The array view corresponds to the <em>entire</em> object but
+ * the order in which the entries are presented in the array is an
+ * implementation detail and should not be relied upon.
  *
  * @relatesalso ArrayView
  */

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.