Comment by pdimitar

3 months ago

Feast your eyes: https://nee.lv/2021/02/28/How-I-cut-GTA-Online-loading-times...

There were HN discussions at the time as well.

4 comments

pdimitar

Reply
zahlman  3 months ago

Amazing. The hashing thing is definitely on R*, but I wouldn't have expected an sscanf implementation to call strlen either. ("Ah, but of course they expect this kind of incremental parsing/tokenization to be done using the totally legit and un-problematic strtok instead!")

  • pdimitar  3 months ago

    Well I gave up trying to understand C and C++ ages ago; they always felt like they sabotaged me for trying to get better at them. Moved on to many other programming languages.

    In this case I believe they should have just vendored a small JSON parser in the game. It's a big blob of .exe and .dll files, why not? Found it weird but then again, we have all made mistakes that we later cringed at.

    • zahlman  3 months ago

      They could have just as easily vendored something with the same problem, and still not noticed. And if they cared on that level, they could also have switched to BSON, msgpack etc.

      1 reply →