#3900 closed enhancement (fixed)
[PATCH] Seperate _kalis RMGen library
Reported by: | FeXoR | Owned by: | FeXoR |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 21 |
Component: | Maps | Keywords: | patch |
Cc: | _kali | Patch: |
Description (last modified by )
Since most maps don't use the libs gaia.js
and setup.js
and to destiguish them more as a specific approach for random map generation I propose seperating them into another directory. That also avoids unneeded namespace flooding.
They are quite closely tied up so they will likely be needed both or none anyways.
Since some of the functions and the maps using them focus on multiplayer teams I propose the name mp-team
.
We might want to do this for other libs as well.
Attachments (1)
Change History (9)
by , 8 years ago
Attachment: | seperate_rm-team.patch added |
---|
comment:1 by , 8 years ago
Uh, I can't find the deletion of the two moved files from rmgen
so I'm not sure if that patch is correct...
comment:3 by , 8 years ago
Cc: | added |
---|---|
Milestone: | Backlog → Alpha 21 |
Summary: | [PATCH] Seperate RMGen libraries to directory mp-team → [PATCH] Seperate _kalis RMGen library |
Just loading those libs in that case is a simple but effective improvement.
The name team isn't ideal though as it also contains functions like addBluffs which is a new way of modifying terrain (see the ambush map, for example in the alpha 20 trailer).
cc'ing _kali, maybe he has a better ieda on the naming.
comment:4 by , 8 years ago
Replying to FeXoR:
Sugestion from mimo:
mpteam
.
It is rather unpleasant to be quoted like that, misrepresenting what i said in IRC. So please don't do it again. My comment on IRC was: I don't like hyphen on directory name, so IF YOU PROPOSE rm-team, then rmteam would be better. But I also said that I didn't know enough the code to propose a name, and certainly i have never mentionned mpteam.
comment:5 by , 8 years ago
@mimio: You are right, sorry. I didn't think this would keep someone to suggest a different name and it was not intended to do so.
sanderd17 suggested rmgen2 (just to summ up what names popped up in IRC, I don't know anything about how serious he was)
comment:6 by , 8 years ago
IMO rmgen2 would fit better. The person committing it should make sure to svn mv
the files (to ensure that it detects that the files were moved, rather than a file being deleted and a new one added).
Since the code of getTeams
is duplicated by many old maps, the function should remain in rmgen
and other maps should be told to use them. But as that conflicts with the svn mv
command, it is material for another commit.
comment:8 by , 8 years ago
Keywords: | review removed |
---|
Patch against r18022