From 6f8b6633be7788d67d09ddb28d0352329d50ddbc Mon Sep 17 00:00:00 2001
From: themancalledjakob <jakob@jrkb.land>
Date: Tue, 19 Sep 2023 15:52:18 +0200
Subject: [PATCH] Revert "more robust outline flipping"

This reverts commit 18909268059dc76911223636b24767c1b8f53181.
---
 src/gpufont/font.hpp | 50 +++++++++++++++++++-------------------------
 1 file changed, 22 insertions(+), 28 deletions(-)

diff --git a/src/gpufont/font.hpp b/src/gpufont/font.hpp
index a241e64..efa7b93 100644
--- a/src/gpufont/font.hpp
+++ b/src/gpufont/font.hpp
@@ -712,7 +712,6 @@ class Font {
                 if(charcode == '\r' || charcode == '\n'){
                     continue;
                 }
-                // if does exist already
                 if(glyphs.count(glyphIdentity) != 0){
                     continue;
                 }
@@ -931,38 +930,33 @@ class Font {
                     convertContour(curves[i], directions[i], bbs[i], &face->glyph->outline, start, face->glyph->outline.contours[i], emSize);
                     start = face->glyph->outline.contours[i] + 1;
                 }
-                // outmost outline must be clock-wise
-                // then flip direction the deeper we go inside
                 for(int i = 0; i < n_contours; i++){
-                    int insideAnother = 0;
-                    for(int j = 0; j < n_contours; j++){
-                        if(i != j){
-                            bool inside = bbs[j].xMin <= bbs[i].xMin
-                                          && bbs[j].xMax >= bbs[i].xMax
-                                          && bbs[j].yMin <= bbs[i].yMin
-                                          && bbs[j].yMax >= bbs[i].yMax;
-                            if(inside){
-                                insideAnother++;
+                    if(directions[i] < 0){
+                        bool flip = true;
+                        for(int j = 0; j < n_contours; j++){
+                            if(i != j){
+                                bool inside = bbs[j].xMin <= bbs[i].xMin
+                                              && bbs[j].xMax >= bbs[i].xMax
+                                              && bbs[j].yMin <= bbs[i].yMin
+                                              && bbs[j].yMax >= bbs[i].yMax;
+                                if(inside){
+                                    flip = false;
+                                    break;
+                                }
                             }
                         }
-                    }
-                    bool flip = false;
-                    if((insideAnother % 2 == 0 && directions[i] < 0)
-                       || (insideAnother % 2 == 1 && directions[i] > 0)){
-                        flip = true;
-                    }
-                    if(flip){
-                        for(BufferCurve & curve : curves[i]){
-                            float tmp_x = curve.x0;
-                            float tmp_y = curve.y0;
-                            curve.x0 = curve.x2;
-                            curve.y0 = curve.y2;
-                            curve.x2 = tmp_x;
-                            curve.y2 = tmp_y;
+                        if(flip){
+                            for(BufferCurve & curve : curves[i]){
+                                float tmp_x = curve.x0;
+                                float tmp_y = curve.y0;
+                                curve.x0 = curve.x2;
+                                curve.y0 = curve.y2;
+                                curve.x2 = tmp_x;
+                                curve.y2 = tmp_y;
+                            }
+                            std::reverse(curves[i].begin(), curves[i].end());
                         }
-                        std::reverse(curves[i].begin(), curves[i].end());
                     }
-                    //}
                     bufferCurves.insert(bufferCurves.end(), curves[i].begin(), curves[i].end());
                 }
             }else{