From: Timo Heister Date: Tue, 14 Feb 2023 22:03:23 +0000 (-0500) Subject: fix wrong sign of char in MF X-Git-Tag: v9.5.0-rc1~548^2 X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9e0434d38d7ca475a87cfb08136896d4d88d8927;p=dealii.git fix wrong sign of char in MF On ARM char defaults to unsigned char, which will break the matrix free face loop. Fix this by being explicit. Fixes: /home/ubuntu/dealii-9.4.2/include/deal.II/matrix_free/face_setup_internal.h:522:35: warning: comparison is always false due to limited range of data type [-Wtype-limits] --- diff --git a/include/deal.II/matrix_free/face_setup_internal.h b/include/deal.II/matrix_free/face_setup_internal.h index 0c188a6389..727d61a40b 100644 --- a/include/deal.II/matrix_free/face_setup_internal.h +++ b/include/deal.II/matrix_free/face_setup_internal.h @@ -379,8 +379,8 @@ namespace internal // those faces, a value -1 means that the process with the lower // rank gets it, whereas a value 0 means that the decision can // be made in an arbitrary way. - unsigned int n_faces_lower_proc = 0, n_faces_higher_proc = 0; - std::vector assignment(other_range.size(), 0); + unsigned int n_faces_lower_proc = 0, n_faces_higher_proc = 0; + std::vector assignment(other_range.size(), 0); if (inner_face.second.shared_faces.size() > 0) { // identify faces that go to the processor with the higher