fix

Author: Alex-PLACET

test/test_xoperation.cpp

@@ -858,6 +858,29 @@ namespace xt
             EXPECT_EQ(expected1, res3);
             EXPECT_EQ(expected2, res4);
         }
+
+        TEST_CASE("divide_4d")
+        {
+            using T4 = xt::xtensor<float, 4, xt::layout_type::row_major>;
+            using shape_type = typename T4::shape_type;
+
+            shape_type shape = {2, 3, 2, 2};
+            T4 a(shape, 4.5f);
+            T4 b(shape, 1.3f);
+
+            EXPECT_EQ((a / b)(0, 0, 0, 0), a(0, 0, 0, 0) / b(0, 0, 0, 0));
+
+            float sb = 1.2f;
+            EXPECT_EQ((a / sb)(0, 0, 0, 0), a(0, 0, 0, 0) / sb);
+
+            float sa = 4.6f;
+            EXPECT_EQ((sa / b)(0, 0, 0, 0), sa / b(0, 0, 0, 0));
+
+            // self-divide assignment: a = a / b (no zeros in b)
+            auto a_before = a(1, 2, 1, 1);
+            a = a / b;
+            EXPECT_EQ(a(1, 2, 1, 1), a_before / b(1, 2, 1, 1));
+        }
     }
 
 #undef XOPERATION_TEST_TYPES

test/test_xview.cpp

@@ -1864,13 +1864,20 @@ namespace xt
 
     TEST(xview, drop_on_1dim_array)
     {
-        const auto my_array = xt::xtensor<double, 1>({1, 2, 3});
+        auto my_array = xt::xtensor<double, 1>({1, 2, 3});
 
-        xt::view(my_array, xt::drop(1));
-        EXPECT_EQ(my_array, (xt::xtensor<double, 1>{1, 3}));
+        // drop(1) creates a view excluding index 1
+        auto v1 = xt::view(my_array, xt::drop(1));
+        EXPECT_EQ(v1, (xt::xtensor<double, 1>{1, 3}));
 
+        // Assign through the drop view
+        xt::view(my_array, xt::drop(1)) = 0.;
+        EXPECT_EQ(my_array, (xt::xtensor<double, 1>{0, 2, 0}));
+
+        // Reset, then test drop with a variable (the original compilation issue)
+        my_array = xt::xtensor<double, 1>({1, 2, 3});
         const size_t index = 1;
-        xt::view(my_array, xt::drop(index));
-        EXPECT_EQ(my_array, (xt::xtensor<double, 1>{1}));
+        auto v2 = xt::view(my_array, xt::drop(index));
+        EXPECT_EQ(v2, (xt::xtensor<double, 1>{1, 3}));
     }
 }