LGTM
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Sep 26 2022
Sep 25 2022
It does not work for me: as soon as i hit Save as a host of a MP game the game crashes. Same for the Load Game button.
I'm not satisfied with the "selecting priority" inside PopTask. The predicate is not trivial and both mutexes are owned at the same time. Do you think this are problems? Do you have a solution?
My ideas (non of them is perfect and i omited m_Kill):
Sep 24 2022
Make sure to capitalize all words in genericName. :)
Should the minimap have its own sprite.xml file for clarity?
Sep 23 2022
Sep 20 2022
Arch is using it
Sep 19 2022
Also there's some info in the meson FAQ about how to use wildcards
Sep 18 2022
icon ideas:
level 1 have 'I' in the top right, level two have 'II' in the top right.
Sep 16 2022
@vladislavbelov asked me to clarify more things before committing this, because it might lead to a 6 months Feature Freeze or a painful revert.
We need to make sure it doesn't cause more spikes in other cases.
In D4758#203957, @Stan wrote:I the meantime can you keep testing it with different loads and replays to make sure there isn't something unstable ? :)
EG. The custom combat huge demo or a 4v4 with ais on big maps etc.
I the meantime can you keep testing it with different loads and replays to make sure there isn't something unstable ? :)
Sep 15 2022
Nah, this looks alright, will see about committing it after the release comes out.
No worries, do you need me to upload again?
Sep 14 2022
Sorry about that, ZFS keeps failing these days, and the VM gets stuck.
Build was aborted.
Sep 13 2022
Successful build - Chance fights ever on the side of the prudent.
Successful build - Chance fights ever on the side of the prudent.
Looks good to go. Wonder if they should have a copyright header. According to https://trac.wildfiregames.com/browser/ps/trunk/source/tools/LICENSE.txt it's MIT.
Won't commit because we reached CF, but will be nice after the release.
Added time units (seconds, milliseconds) to variable names.
Made Relese NG entity file an optional argument
In D4698#203936, @Stan wrote:Just realised I'm an idiot and you need a way to specify the location because you don't want to put the rng in the mod folder (since you're probably gonna zip it afterwards) and it won't be there on the CI either.
Just realised I'm an idiot and you need a way to specify the location because you don't want to put the rng in the mod folder (since you're probably gonna zip it afterwards) and it won't be there on the CI either.
Sorry, that's what happens when I do too much things at once.
Deleted the middle directory scripts in entity.rng's location. Didn't made it eligible, by argument, to avoid directory creation in case of not existence.
In D4698#203928, @Stan wrote:~~ Sadly I'm not sure I can set you as author. There seem to be a way but I dunno if trac and phab support it https://stackoverflow.com/questions/2107719/how-to-set-author-for-svn#2107898 ~~
EDIT: Seems it's only for files.
~~ Sadly I'm not sure I can set you as author. There seem to be a way but I dunno if trac and phab support it https://stackoverflow.com/questions/2107719/how-to-set-author-for-svn#2107898 ~~
It's been a complete overhaul of the file, but it is working. Now it receives the modules' location and name as parameters.
Sep 12 2022
Sep 11 2022
Added a new diff with the flag -U3000 as suggested on IRC.
As there haven't been any more feedback on this revision and we had various cases where people got confused which code is the current one, I went ahead and committed the changes now.
Fix year and LGTM.
Successful build - Chance fights ever on the side of the prudent.
Successful build - Chance fights ever on the side of the prudent.
- Make config variable naming consistent
- Check whether the passed in upgrade at least exists
That commit leads to an out of bounds access on each match start. I'm not sure there is no other bugs in the code.
Sep 10 2022
Successful build - Chance fights ever on the side of the prudent.
Successful build - Chance fights ever on the side of the prudent.
- Minor lint changes
- Set plan ID
- Move damage priorities to config
Sep 9 2022
Maybe try this flag https://github.com/7wells/0ad/blob/master/binaries/system/readme.txt#L75
Build failure - The Moirai have given mortals hearts that can endure.
Build failure - The Moirai have given mortals hearts that can endure.
Build failure - The Moirai have given mortals hearts that can endure.
Build was aborted.
Sep 8 2022
Successful build - Chance fights ever on the side of the prudent.
Successful build - Chance fights ever on the side of the prudent.
change parameter order of PushTask
add context
In D4425#203777, @Stan wrote:Can't you make it the responsibility of the receiver to destroy objects instead of adding the extra overhead of a unique ptr?
Successful build - Chance fights ever on the side of the prudent.
Successful build - Chance fights ever on the side of the prudent.
Revert to the one that worked
Note to self/anyone else interested: One could also use this in the army manager to upgrade units as well. If the army composition is more melee, the persian champions could instead be switched to archer variants to balance it out and whatnot.
Successful build - Chance fights ever on the side of the prudent.