-
Joined on
2025年04月29日
Squashed and clang-formatted ✅
yeah I saw that too after pushing. I was meaning to do a clang anyways before squashing ^^
Reproduced and fixed, just to be save I changed everything related to the original country polygons to be exactly the same as before. Will regenerate and squash later
Agree, changed this and also the later if (!reader.IsExist)... to CHECKs
Okay so this cannot happen at all with the current code, simplifiedPolygons will always have at least one entry with the current code. I turned it into a CHECK same as the others, to prevent any...
What do you mean by this exactly? Raise an exception? Yes would be better, will fix later along with the remaining thing above
As it has been a while, I will need to look at this again more in detail when I find some time
Good point, changed those to use CHECK
Made this line LINFO and added another LINFO "[BORDERS_GENERATOR] done" log
LGTM, but maybe should wait for at least one more person to look at this in case there are strong opinions on the icons