Remove no longer relevant comment and refactor a few lines

Author: ewoudwempe

include/xtensor/xassign.hpp

@@ -470,16 +470,13 @@ namespace xt
                 linear_assigner<false>::run(de1, de2);
             }
         }
+        else if (simd_strided_assign)
+        {
+            strided_loop_assigner<simd_strided_assign>::run(de1, de2);
+        }
         else
         {
-            if (simd_strided_assign)
-            {
-                strided_loop_assigner<simd_strided_assign>::run(de1, de2);
-            }
-            else
-            {
-                stepper_assigner<E1, E2, default_assignable_layout(E1::static_layout)>(de1, de2).run();
-            }
+            stepper_assigner<E1, E2, default_assignable_layout(E1::static_layout)>(de1, de2).run();
         }
     }
 

include/xtensor/xcontainer.hpp

@@ -928,11 +928,12 @@ namespace xt
         {
             return i == 0;
         };
-        // This is unsafe if the strides are all 0. Does that happen?
         if (!is_contiguous_container<storage_type>::value)
         {
             return false;
         }
+        // We need to make sure the inner-most non-zero stride is one.
+        // Trailing zero strides are ignored because they indicate bradcasted dimensions.
         if (m_layout == layout_type::row_major)
         {
             auto it = std::find_if_not(m_strides.rbegin(), m_strides.rend(), is_zero);