Issue with coding

[Z-Man]

Ooh, well spotted. Yes, that is indeed likely to cause a crash, because it copies the spawn point list, which is actually supposed to be a non-copyable object.

Using

Code: Select all

gArena & arena = Arena; 

instead should fix the problem.

[Alpha Centauri]

Ooh, well spotted. Yes, that is indeed likely to cause a crash, because it copies the spawn point list, which is actually supposed to be a non-copyable object.

Sounds like a good case for a private copy constructor. :P

[ivantis]

see? i knew z-man would get this one. how was i supposed to know? i didnt write the game...

[hoop]

see? i knew z-man would get this one. how was i supposed to know? i didnt write the game...

You weren't supposed to know that that object should not be copied, but anyway the statement:

Code: Select all

gArena arena = Arena; 

would be totally useless, since you could have just used the global object., it doesn't matter if you're using it inside your fucntion.
Even Z-Man's suggestion

Code: Select all

gArena & arena = Arena; 

doesnt' look useful for me: you are just making an alias for 'arena' ,called 'Arena'

Last thing: if the RESPAWN_PLAYER code is placed in gCycle.cpp, you should put your code there too.
IMO gWinzone.cpp should be modified for other purposes.

[Z-Man]

Even Z-Man's suggestion

Code: Select all

gArena & arena = Arena; 

doesnt' look useful for me: you are just making an alias for 'arena' ,called 'Arena'

Yes It was less to type for me, however.

[Alpha Centauri]

Even Z-Man's suggestion

Code: Select all

gArena & arena = Arena; 

doesnt' look useful for me: you are just making an alias for 'arena' ,called 'Arena' :?:

Other way around and you have it essentially right.

[ivantis]

its in gWinZone.cpp for a purpose, its for the teleporting zones.

[hoop]

wow, ok