- User Since
- Dec 27 2016, 8:33 PM (195 w, 1 d)
Aug 2 2017
Aug 1 2017
0ad-project/binaries/... That's not going to apply cleanly when someone tries to test your patch. Please either use arc (described in https://trac.wildfiregames.com/wiki/Phabricator#UsingArcanist), or at the very least ensure that you create the patch using the directory that has binaries/ and source/ as the root.
Jul 22 2017
Jul 14 2017
Jul 12 2017
Jul 11 2017
(IMO it makes things too crowded, but it just comes down to personal taste.)
Jul 10 2017
Fix oversight; the traders should only move if !possibleRoute, as well as !ship.
Jul 9 2017
Okay, will do.
Jul 8 2017
The patch removes the early return and now matches with the trainableEntities() function.
Jul 7 2017
I first had to do a bit more reading on how this function is used, but in fact I am still a bit confused about what this patch achieves (maybe I miss something obvious, or didn't read enough):
- In all 5 uses of researchableTechs(), the civ passed to it will always be the player civ.
- The trainableEntities() function, which is like a sister function, is sometimes passed a civ (which again is the player civ), or sometimes not, and so it is possible that it may return invalid templates.
Jul 6 2017
Jul 5 2017
Alright, the comment was addressed in the new diff, and I ran tests on naval maps with the new diff.
I believe the diff is good, although to play devil's advocate, I'm really not sure about adding support for arbitrary mods which may or may not be modified later at a whim, making the code irrelevant (there is, of course, no comparable quality control nor review process for Delenda Est, and this is typical for most game mods). And in any case, such added code which panders to mods will be irrelevant/useless in the vast majority of cases anyways, as most people do not use mods (so this could be seen as effectively adding dead code), but only add unneeded complexity which will need to be maintained. I know there is some support in f.e. the structure tree, but that is perhaps different.
I don't believe the AI will need adaptation to these changes. The AI never accesses the Builder component (through AIProxy or otherwise); it instead uses its own
buildableEntities() function in entity.js (which is basically the equivalent of cmpBuilder.GetEntitiesList()). And in any case, this change to GetEntitiesList() just adds a template existence check, and the AI already does checks for this in the appropriate places.
Jul 3 2017
Jul 1 2017
Jun 30 2017
Jun 29 2017
Use Vladislav's recommendation and add the function that deletes m_MapReader to the LoadRequests to be called when all map loading tasks are finished, instead of calling it from CGame.
Thank you Vladislav for the comments!
Jun 28 2017
SAFE_DELETE() m_MapReader once all map-loading tasks have been finished.
Jun 27 2017
Jun 26 2017
(Also there is an unused CTerritoryManager member var (another non-existent class) in CWorld with a corresponding getter, a relic of the old simulation (before the big rewrite that replaced it with simulation2).)
Jun 25 2017
If you're not going to use arc, you might want to try creating patches from root directory of your working copy. Using your E drive as the root directory is not helpful.
Jun 24 2017
I don't really know this code; when adding the range visualization I just moved the code that was already in the old UpdateStaticOverlay into the new UpdateTexturedOverlay (I didn't change/look at the math).
Jun 23 2017
Jun 20 2017
Jun 18 2017
Jun 16 2017
Jun 15 2017
Jun 14 2017
Jun 10 2017
Jun 6 2017
I meant I used the exact command ./pyrogenesis -mod=public -replay=commands.txt on the attached commands.txt, but got the errors I posted when doing so. I didn't pass the mod 'mod', but I did pass the mod 'public'. It seems like some sort of bug.
Jun 5 2017
Yes, and it would be preferable to include it for trade routes (eventually, Petra should check its trade routes periodically with this check imo, even if just using a straight line as is done here). But yes, isUnderEnemyFire is fine.
Jun 4 2017
Update SetWallPlacementPreview, remove uneeded QueryOwnerInterface call in GuiInterface.js (just use the player var already passed to the functions), remove old comment in cmpVisual, remove the return in SetActor.
Thanks, this will be a great help for this issue, but still, as alluded to the trac ticket, I believe that HQ.isDangerousLocation() should be used as well when planing trade routes, even more so as this approach is less perfect than doing an actual computePath check.
Jun 1 2017
This commit seems to be the one that broke two hotkeys, see D584.
Okay, this fixes the garrison/ungarrison loop in the described case.
The comments were addressed in the new diff, looks good.
May 31 2017
I really like this texture for the new heal range, thanks for that (preferably just for entities with the heal component, not for healing auras).
Remove typo fix that was committed, add early return if teamsLocked.
Fix typo in rP19194 in chathelper.js that throws errors.
The patch was tested, and it works as advertised.
May 30 2017
Here is an example when enemy structures are close to the AI's territory, maybe there needs to be a check for structure, the units will get stuck in a loop indefinitely; there are three enemy houses below the civic center:
I think this trigger script is getting to the point that some of the logic/functions should be moved to a common place (TriggerHelper) so that other scripts can easily use them to handle spawning gaia units/make them patrol/attack, etc. (likely won't happen in this diff), as the lot of the functions are generic enough that they should be elsewhere.
May 29 2017
I tested this map, and I think it is nice from a gameplay perspective (didn't look at the code).
Update and allow space in the options menu.
This is good to see, it's good to have different textures for different range visualizations (although I'm a bit uncertain if this new texture should also be used for auras that happen to heal, it may be confusing to new players).
May 28 2017
Don't save a another var to the rangevis comp that doesn't need to be saved.
May 27 2017
May 26 2017
On Windows, this diff causes a crash deep in SpiderMonkey when opening the replay menu (not if there is nothing to load in the replay menu though).