kaylus / myanmud

Automatically exported from code.google.com/p/myanmud
GNU Affero General Public License v3.0
0 stars 0 forks source link

Change this_player over to thread local storage #6

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.
2.
3.

What is the expected output? What do you see instead?

Please use labels and text to provide additional information.

Original issue reported on code.google.com by hambon...@gmail.com on 5 Feb 2010 at 10:46

GoogleCodeExporter commented 9 years ago
We deliberately rejected this_player(), though. As I recall, half of the 
  code in the old LPC driver is concerned with making sure this_player() 
has the right value at all times. It needs to be switched in and out 
constantly, depending on which function is being called where.

Instead, we simply pass along an explicit 'actor' variable.

Original comment by hambon...@gmail.com on 7 Feb 2010 at 10:01

GoogleCodeExporter commented 9 years ago
Do this in driver if it needs to be done.

Original comment by hambon...@gmail.com on 11 Feb 2010 at 2:06

GoogleCodeExporter commented 9 years ago
There's now tls storage of this player, although it's debatable if this should 
be done.

Original comment by hambon...@gmail.com on 27 Mar 2012 at 2:08