Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upMoving spawn point (Simple zombie hordes emulation) #6611
Conversation
added some commits
Mar 12, 2014
This comment has been minimized.
This comment has been minimized.
|
I really like the sound of this one a lot! |
This comment has been minimized.
This comment has been minimized.
|
^^ Me too! |
KA101
self-assigned this
Mar 13, 2014
This comment has been minimized.
This comment has been minimized.
|
Biggest problem I'm seeing is a few typos in the descriptive text. ;-) Queueing. |
This comment has been minimized.
This comment has been minimized.
|
Should probably see a bit more zeds spawn at once (possibly 3x the character's highest combat skill??) but this looks very nice so far. I'll bring it aboard. |
KA101
merged commit b3db1ed
into
CleverRaven:master
Mar 13, 2014
1 check was pending
This comment has been minimized.
This comment has been minimized.
|
"group+=" in game::spawn_mon Nothing will break? |
This comment has been minimized.
This comment has been minimized.
|
Kinda depends on where you want it. If you hit the "files changed" tab, you can then click on the left of a line to insert a note on a particular line. (Needs javascript, in case you're a NoScript user.) |
BevapDin
reviewed
Mar 14, 2014
| @@ -638,7 +641,8 @@ void overmap::save() | |||
| for (int i = 0; i < zg.size(); i++) | |||
| fout << "Z " << zg[i].type << " " << zg[i].posx << " " << zg[i].posy << " " << | |||
| zg[i].posz << " " << int(zg[i].radius) << " " << zg[i].population << " " << | |||
| zg[i].diffuse << " " << zg[i].dying << std::endl; | |||
| zg[i].diffuse << " " << zg[i].dying << " " << | |||
| zg[i].horde << " " << zg[i].tx << " " << zg[i].ty << " " << zg[i].interest << std::endl; | |||
This comment has been minimized.
This comment has been minimized.
BevapDin
Mar 14, 2014
Contributor
This breaks save compatibility. The operator<< will read data from the next line, which will screw up all entries below that: cities, roads, radios, npcs.
ghost
referenced this pull request
Reaper42 commentedMar 13, 2014
It turn off by default in options.