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/8aamcfy.png Is this some crazy ridiculous attempt at idle disconnections?
  • You've some how managed to add many more locks to an already lock-heavy program, congratulations.
  • http://i.imgur.com/vkY9gEz.png Called your own core shit, nice.
  • 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.

/r/wowservers Thread Parent Link - sunwell.pl