Remove the last use of memmove. NFC.

whitequark · whitequark · commit cc107887e727 · 2019-05-23T19:30:33.000Z
Much clearer!
diff --git a/src/mesh.cpp b/src/mesh.cpp
@@ -186,17 +186,15 @@ void SMesh::Simplify(int start) {
                     if(fabs(bDot) < LENGTH_EPS && fabs(dDot) < LENGTH_EPS) {
                         conv[WRAP((j+1), convc)] = c;
                         // and remove the vertex at j, which is a dup
-                        memmove(conv+j, conv+j+1,
-                                          (convc - j - 1)*sizeof(conv[0]));
+                        std::move(conv+j+1, conv+convc, conv+(convc-1));
                         convc--;
                     } else if(fabs(bDot) < LENGTH_EPS && dDot > 0) {
                         conv[j] = c;
                     } else if(fabs(dDot) < LENGTH_EPS && bDot > 0) {
                         conv[WRAP((j+1), convc)] = c;
                     } else if(bDot > 0 && dDot > 0) {
                         // conv[j] is unchanged, conv[j+1] goes to [j+2]
-                        memmove(conv+j+2, conv+j+1,
-                                            (convc - j - 1)*sizeof(conv[0]));
+                        std::move_backward(conv+j+1, conv+convc, conv+j+2);
                         conv[j+1] = c;
                         convc++;
                     } else {
