This comment was posted to reddit on Feb 16, 2016 at 7:23 pm and was deleted within 8 hour(s) and .
Sunwell Core released to the public
Sure, let's begin, shall we?
Here's 30 minutes worth of basic random code review:
WorldSocket::HandleAuthSession Single threaded authentication, welcome to network blocking hell.
Still using ACE for networking, not using boost AT ALL ANYWHERE.
http://i.imgur.com/jWk2Ajv.png Really? Breaking if an opcode took more than 3 ms? I bet the response time is a nightmare there. How many cycles would you skip under worst-case complexity just to cast a single spell? 5? 10? Lol. God forbid if the cpu cache flushed while processing opcodes
http://i.imgur.com/6fpJWOP.png Cute, no warden at all for OSX, Let just send a simple packet edit to bypass your entire anticheat.
http://i.imgur.com/lVpsqJ0.png You have got to be joking me. Hacks, shit readability, shit maintainability. This file contains much more than that as well.
http://i.imgur.com/my26yEy.png So let me get this straight, you insert into a basic stl list with manual locking (hello locked queue, what are you??), then you proceed to iterate that locked list http://i.imgur.com/QrWE8cR.png to insert into another stl list, to then process here http://i.imgur.com/Kfm8Wx9.png, which is not a thread safe operation (because you are still running Map::Update while accessing the player in the AH thread, creating data races).
I have no problem with you or sunwell, but when people give you guys so much praise and i can find numerous problems in 30 minutes it saddens me.