Repository navigation
Conversation
|
What exactly was wrong with it? The old code I took from glib ( |
|
Well, the existing routine should work perfect with all UTF-8 characters as well. Let me try to reproduce this. |
|
Ok, so I've just created such a constraint using the e129755 build from Appveyor on Windows XP SP3: What configuration are you using exactly? |
|
Intel Core 2 Quad Q6600, Windows 7 Ultimate SP1 x86, Visual Studio Ultimate 2013 Version 12.0.40629.00 Update 5 |
|
Ok, I'll install that. Did you build it in the default configuration, or did you change some CMake parameters? |
|
I just rebuilt using your exact VS version and OS. I cannot reproduce your crash. Does the binary from https://ci.appveyor.com/project/whitequark/solvespace/build/2.1.7/artifacts crash as well? |
|
Sorry for delay here.. So, looks like problem is gone after cloning a repo in new location and rebuilding. Still don't know why it was crashing. Sorry for false alarm and wasting your time! |
|
I suspect the UTF-8 in the source got broken somehow; the UTF-8 iterator is optimized for speed, not robustness, and it's all but supposed to crash that way. This isn't really important because all input data (except that read from savefiles) is already sanitized at boundary, and savefiles will be sanitized soon since I have to rewrite the serialization code anyway. |
Merge from remote.




Old UTF8 conversion works wrong with degree symbol that is showing while using angle constraint.
Calculating of bytes offset was wrong and then we stuck in loop trying to read some thrash in memory.