Fredrick Brennan

Results 619 comments of Fredrick Brennan

And then there were two! :smile: https://github.com/ctrlcctrlv/Noto-Serif-Tagalog

Hey Ben, thanks for all these notes. This one especially seems to affect FontForge. Do you mind if I mention these issues in FontForge issues? Don't worry, I won't block...

Then again, perhaps I'm "holding it wrong" by using the `Coord2` type to represent vectors and not just coordinates? :thinking:

Sorry, I don't quite understand. I have support for both your modes in MFEKpathops. `path_remove_overlapped_points` results in this for example: ![image](https://user-images.githubusercontent.com/838783/152641653-6dfaf003-b4c4-416e-b1c4-ca5c8f8272a3.png) It results in buggy output across the font: ![image](https://user-images.githubusercontent.com/838783/152641759-908c31e1-5b09-4fd4-9a6f-43504154c937.png)...

If you want to see exactly what I see, MFEKpathops is the project. https://github.com/MFEK/pathops/blob/227b9663984b728e4ad241980545eb498ae52832/src/boolean.rs#L123 `apply_flo` is my attempt to use your modes… ```rust let out = match pathop { FloPathOp::RemoveInterior...

Huh, looking at it again has given me something of a “eureka”. I think my accuracy was way too tolerant. I lowered it by around ten orders of magnitude and...

Yeah unfortunately your `RemoveInteriorPoints` code just hates certain glyphs though. `g.low` especially. ![image](https://user-images.githubusercontent.com/838783/152642438-27bc7936-1074-4d64-9938-3da94d961b6c.png) ![image](https://user-images.githubusercontent.com/838783/152642487-75a4fdda-8033-421c-aae3-7ed3ffc11e3a.png) Sigh…don't worry, you're not the only one whose code I've managed to break, cf. https://github.com/adobe-type-tools/afdko/issues/1463.