Page MenuHomeWildfire Games

Stan (Stanislas Daniel Claude Dolcini)
Butler

User Details

User Since
Jan 6 2017, 3:41 PM (246 w, 4 d)

Recent Activity

Yesterday

Stan added a comment to D2432: Add a -help option.

I mean that debian provides or used to provide a man page for 0 A.D. while some other distributions do not :).

Mon, Sep 27, 8:28 PM
Stan added a comment to D4283: Implement "-help" as a command line option.

I'm not sure debug_printf works at all on Windows. You need to have SysInternals dbgview in order to be able to see such output.

Mon, Sep 27, 6:33 PM
Stan added a comment to D4283: Implement "-help" as a command line option.

I guess the issue was that we didn't want to duplicate the Readme inside the game's code. D4232 used a rather clever trick for that.

Mon, Sep 27, 4:20 PM
Stan added a comment to D4283: Implement "-help" as a command line option.

Did you see https://code.wildfiregames.com/D2432

Mon, Sep 27, 7:49 AM

Fri, Sep 24

Stan added a comment to D4108: Adds FreeType support to the engine.

@s0600204 any idea what we could do for macos and pkg config ?

Fri, Sep 24, 5:41 PM

Thu, Sep 23

Stan added a comment to D2175: Select formations as a whole by default..

Well yeah the logic of control groups is really similar, one might say that control groups and formations are overlapping behaviors, if you take aside the fact that a control group can contain multiple formations.

Thu, Sep 23, 11:30 AM
Stan added inline comments to rP13200: Adds attack move bound to the Ctrl hotkey. Patch from mimo. Refs #1001..
Thu, Sep 23, 8:23 AM

Wed, Sep 22

Stan added inline comments to D4279: Improve minimap viewpoint-polygon placement.
Wed, Sep 22, 8:23 PM
Stan updated subscribers of D4155: Remove code for the lobby bots from SVN.

Indeed it was not discussed publicly I believe, as I had a really hard time reaching to @user1

Wed, Sep 22, 5:23 PM
Stan added a comment to D3611: Use pkg-config even more.

I will run the --force-rebuild if/when it is committed. :)

Wed, Sep 22, 7:16 AM

Tue, Sep 21

Stan updated subscribers of D4278: Moves cursor to VideoMode.

@Langbart can you check cursorbackend="system"

Tue, Sep 21, 3:57 PM
Stan added a comment to D3611: Use pkg-config even more.

Any errors/warnings? Else it's good to go I suppose @s0600204

Tue, Sep 21, 2:09 PM
Stan added a comment to D3611: Use pkg-config even more.

I'm stupid, you need to pass --force-rebuild to build-osx-libs.sh

Tue, Sep 21, 12:11 PM
Stan added a comment to D3611: Use pkg-config even more.

Can you run the script without --preserve libs ;) You need to rebuild the thing for the pc files to be copied.

Tue, Sep 21, 11:23 AM
Stan added a comment to D3611: Use pkg-config even more.

Can you see if you can find the fmt.pc file? It might be name libfmt.pc or something similar.

Tue, Sep 21, 11:05 AM

Mon, Sep 20

Stan updated subscribers of D3611: Use pkg-config even more.

@Langbart can you test this maybe we'll be able to debug what goes wrong on macos :)

Mon, Sep 20, 4:49 PM
Stan added inline comments to D2175: Select formations as a whole by default..
Mon, Sep 20, 11:36 AM
Stan added inline comments to D4277: Rename misnamed lastGatheredType in ResourceGatherer..
Mon, Sep 20, 8:06 AM

Sun, Sep 19

Stan accepted D3646: Debundle Valgrind and make it optional.

Alright then.

Sun, Sep 19, 12:22 PM
Stan updated subscribers of D3646: Debundle Valgrind and make it optional.

According to wraitii valgrind didn't really work for him on macOS

Sun, Sep 19, 12:54 AM

Fri, Sep 17

Stan added inline comments to D4216: [PetraAI] - Reduce duplication when trying to research a specific technology..
Fri, Sep 17, 11:47 AM
Stan updated subscribers of D4273: [gameplay] Briton bonus - faster wood gather rate.

Adding @genava55 as he might have some input.

Fri, Sep 17, 10:07 AM

Thu, Sep 16

Stan added a comment to D4108: Adds FreeType support to the engine.

Yeah it's because freetype ft2build.h is named freenode2/ft2build.h only on macOS

Thu, Sep 16, 4:27 PM
Stan added a comment to D4108: Adds FreeType support to the engine.

Interesting so it picked the homebrew version instead of the local one. It's a bit strange the game compiled with no header change though.

Thu, Sep 16, 4:12 PM
Stan added a comment to D4108: Adds FreeType support to the engine.

You probably have a version somewhere @Langbart see if you can find it in /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/

Thu, Sep 16, 4:05 PM

Tue, Sep 14

Stan added a comment to D4241: Allow to push items to the front of the ProductionQueue..

I could probably pull up something with the training wheel like for autotrain, but I'm not sure where it would be used, since it's a hotkey only?

Tue, Sep 14, 7:52 PM
Stan added inline comments to D2175: Select formations as a whole by default..
Tue, Sep 14, 7:49 PM
Stan added inline comments to D3037: Add gui scale to options.
Tue, Sep 14, 2:17 PM
Stan added a comment to D2175: Select formations as a whole by default..

Code seems much easier now XD.I suppose wow will want to have only battalions all the time, is that possible?

Tue, Sep 14, 8:50 AM

Mon, Sep 13

Stan added a comment to D2175: Select formations as a whole by default..

The more I play with this the more I think this should just be the default for formations.

Mon, Sep 13, 6:22 PM
Stan added a comment to D4270: improve player_water material .

You can use github.com or github1s.com's search features

Mon, Sep 13, 2:04 PM
Stan added a comment to D4271: [WIP] Civ backgrounds for minimap..

Don't think that's possible.

Mon, Sep 13, 12:02 PM
Stan updated the diff for D4271: [WIP] Civ backgrounds for minimap..

Linter

Mon, Sep 13, 11:10 AM
Stan added inline comments to D4269: Removes duplication of terrain alpha map creation.
Mon, Sep 13, 11:07 AM
Stan requested review of D4271: [WIP] Civ backgrounds for minimap..
Mon, Sep 13, 11:05 AM
Stan added a comment to D4270: improve player_water material .

Nice patch. How does it look for other objects using that shader?
Just for the record how bad does it look for ARB since there is no shader ?

Mon, Sep 13, 11:03 AM
Stan committed rP25918: Improve desert_small's texture look using the new textures. Rename them to….
Improve desert_small's texture look using the new textures. Rename them to…
Mon, Sep 13, 12:38 AM

Sun, Sep 12

Stan added a comment to D4223: Drop custom utf16 string implementation.

@vladislavbelov can you commit this now that you cleaned up CStr?

Sun, Sep 12, 11:06 PM
Stan added a comment to D4032: Increase the spacing in the top panel and synchronize the color in the tooltip for gatherers..

Maybe the conventions should say modern English.
https://prowritingaid.com/grammar/1000188/Fro-vs-from%E2%80%94what-is-the-difference

Sun, Sep 12, 6:43 PM

Sat, Sep 11

Stan updated subscribers of rP25916: Fix translation of "%(unit)s can't be controlled'.
In D4254#180813, @bb wrote:

Don't forget to add yourself to the programming.json credits

Sat, Sep 11, 4:46 PM

Fri, Sep 10

Stan added a comment to D4108: Adds FreeType support to the engine.

Another solution would be to #ifdef the usages of freetype. Up to you;

Fri, Sep 10, 3:33 PM
Stan added a comment to D4108: Adds FreeType support to the engine.

Okay so as @wraitii said build fails on macos because path is

Fri, Sep 10, 12:35 PM
Stan added a comment to D4108: Adds FreeType support to the engine.

Forgot the build-osx-libs.sh file :)

Fri, Sep 10, 9:56 AM
Stan resigned from rP25101: GameSetup GUI cleanup - Renaming..

Fixed in rP25851

Fri, Sep 10, 8:47 AM

Thu, Sep 9

Stan added inline comments to D3475: General improvements to 0ad.desktop.
Thu, Sep 9, 8:42 PM
Stan abandoned D4127: Move test maps and some demo maps out of the public mod..
Thu, Sep 9, 8:00 PM
Stan committed rP25908: Fix false positive; undefined variable in NetworkClient.cpp.
Fix false positive; undefined variable in NetworkClient.cpp
Thu, Sep 9, 8:00 PM
Stan closed D4258: Fix undefined variable in NetworkClient.cpp and save one check.
Thu, Sep 9, 8:00 PM
Stan committed rP25907: Remove Custom implementation of wrename on windows. Phab:rP25854 removed the….
Remove Custom implementation of wrename on windows. Phab:rP25854 removed the…
Thu, Sep 9, 7:53 PM
Stan closed D4225: Remove Custom implemenation of wrename on windows..
Thu, Sep 9, 7:53 PM
Stan committed rP25906: Disable the false positive "mod by 0" warning on Windows..
Disable the false positive "mod by 0" warning on Windows.
Thu, Sep 9, 7:49 PM
Stan closed D4259: Disable the mod by 0 warning on windows.
Thu, Sep 9, 7:49 PM
Stan accepted D4265: Adds header for forward declarations of CStr.
Thu, Sep 9, 7:43 PM
Stan added a comment to D4265: Adds header for forward declarations of CStr.

Patch looks okay, compiles and seems like a move in the right direction.

Thu, Sep 9, 7:43 PM
Stan added inline comments to D3475: General improvements to 0ad.desktop.
Thu, Sep 9, 5:25 PM
Stan added a comment to D4266: Bring Walruss and Muskox back to the arctic biome.

Maybe they were removed due to a lack of animation? The link in the summary points more toward a will to get A23 back than an actual need for those animals.

Thu, Sep 9, 2:35 PM
Stan added a comment to D4265: Adds header for forward declarations of CStr.

I see thanks. Does it make compilation slower?

Thu, Sep 9, 12:48 PM
Stan added a comment to D4265: Adds header for forward declarations of CStr.

What's the benefit of having an extra file to load for fowarding?

Thu, Sep 9, 10:56 AM

Wed, Sep 8

Stan added inline comments to D4253: Fix PetraAI after the baseManager split..
Wed, Sep 8, 7:33 PM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….

I asked just in case but yeah looks like we will need to teach the script that pulls translations how to clean the files.

Wed, Sep 8, 3:48 PM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….
Hello Stanislas,
Wed, Sep 8, 3:07 PM
Stan added inline comments to D4086: Texture update/ general overhaul for Gulf of Bothnia biomes.
Wed, Sep 8, 10:45 AM
Stan added inline comments to D4086: Texture update/ general overhaul for Gulf of Bothnia biomes.
Wed, Sep 8, 9:37 AM

Tue, Sep 7

Stan updated subscribers of D4106: Adds config settings for borderless fullscreen and window.

IMHO you should commit this too, it doesn't hurt to have it, give us a workaround for the green screen, and keeps default behavior.
@Freagarach and @Langbart might want to try it. I suppose you tested it on Windows?

Tue, Sep 7, 11:22 PM
Stan awarded rP25900: Moves macro-defined methods of CStr to templates. a Love token.
Tue, Sep 7, 11:20 PM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….

Sent them an e-mail

Tue, Sep 7, 4:03 PM
Stan added a comment to D4260: Update translation files on translator change.

How about using the regex we use for translators instead ? if it matches, then the line can be updated, if not it's something else.
Else it looks like a necessary evil.

Tue, Sep 7, 3:53 PM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….

Well it would be a mess because we'd then get extra translations changes after TF we'd have to get rid of.

Tue, Sep 7, 9:52 AM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….

We actually need the translator's names to be able to update the credits.

Tue, Sep 7, 9:37 AM
Stan added a comment to rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….

Given that each user has to do it manually in their user settings and I can't find anything related to that I suppose we'll have to do it manually.

Tue, Sep 7, 9:19 AM

Mon, Sep 6

Stan added inline comments to D4251: Replaces CStr macro by templates.
Mon, Sep 6, 7:38 PM
Stan added inline comments to D4258: Fix undefined variable in NetworkClient.cpp and save one check.
Mon, Sep 6, 5:22 PM
Stan added a comment to D4258: Fix undefined variable in NetworkClient.cpp and save one check.
bool FindPublicIP(ENetHost& transactionHost, CStr& ip, u16& port)
{
	if (!STUNRequestAndResponse(transactionHost))
		return false;
Mon, Sep 6, 5:20 PM
Stan requested review of D4259: Disable the mod by 0 warning on windows.
Mon, Sep 6, 5:08 PM
Stan requested review of D4258: Fix undefined variable in NetworkClient.cpp and save one check.
Mon, Sep 6, 4:57 PM
Stan added a comment to D3646: Debundle Valgrind and make it optional.

Mmmh there is no #ifdef in

Mon, Sep 6, 4:55 PM
Stan added a comment to D3646: Debundle Valgrind and make it optional.

Looks alright. Is mac able to use pkgconfig too or should we make it linux only /add more stuff to premake ?

Mon, Sep 6, 3:00 PM
Stan added inline comments to D4251: Replaces CStr macro by templates.
Mon, Sep 6, 2:39 PM
Stan added a comment to D4251: Replaces CStr macro by templates.

Does that mean mac and Linux can no longer deserialize CStr?

Mon, Sep 6, 11:55 AM

Sun, Sep 5

Stan committed rP25897: Make translation pulling parallel to reduce update time..
Make translation pulling parallel to reduce update time.
Sun, Sep 5, 7:40 PM
Stan committed rP25896: Fixes an email that should be deleted by the i18n script but isn't because of….
Fixes an email that should be deleted by the i18n script but isn't because of…
Sun, Sep 5, 7:38 PM
Stan added a comment to D4255: Typo correction for gui/modmod/validatemod.js.

Refs D1093 #4427 rP20552

Sun, Sep 5, 5:58 PM
Stan added a comment to D4254: Added translatability of message "%(unit)s can't be controlled.".

It's only ps trunk when committing :)
That is because it becomes part of svn which is actually three repos

Sun, Sep 5, 3:10 PM
Stan added a comment to D3646: Debundle Valgrind and make it optional.

Agree with @sera then I think it's good to go.

Sun, Sep 5, 1:24 PM
Stan added a comment to D3611: Use pkg-config even more.

Anything else needed ? Maybe D3646 should go first.

Sun, Sep 5, 1:24 PM

Sat, Sep 4

Stan added a comment to D4251: Replaces CStr macro by templates.

What's the advantage of templates here? Is it faster, safer, better C++ (vs C style code) ?

Sat, Sep 4, 3:41 PM
Stan added a comment to D4252: GUI Enabled Mods - short explanatory note added about later mods overwriting earlier installed mods.

Congrats on your first patch!

Sat, Sep 4, 3:33 PM

Fri, Sep 3

Stan accepted D4247: A little optimization for model loading.

Seems to give small boost 1.5% when loading meshes (conversion to PMD excluded)

Fri, Sep 3, 6:32 PM
Stan added inline comments to D4247: A little optimization for model loading.
Fri, Sep 3, 12:49 PM
Stan accepted D4245: First iteration of CStr cleanup.

Looks alright, doesn't seem to break asian mods.

Fri, Sep 3, 10:55 AM

Thu, Sep 2

Stan added a comment to D3200: Stop dodging arrows by spamclicking or patrol: Combine rotations with acceleration..

I guess we'll need a bundle to get feedback from players.

Thu, Sep 2, 10:48 PM

Wed, Sep 1

Stan awarded rP25885: Removes unused g_GameRestarted forgotten in rP25628. a Love token.
Wed, Sep 1, 10:23 PM
Stan added a comment to D4245: First iteration of CStr cleanup.

Ok. We should find a way to test this patch with the _UNICODE defined like on OpenSuse.

Wed, Sep 1, 3:06 PM
Stan added a comment to D4245: First iteration of CStr cleanup.

Would be nice to commit D4223 first if it's correct, because this patch might make rebasing complicated.

Wed, Sep 1, 2:42 PM

Tue, Aug 31

Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.
In D4244#180458, @Stan wrote:

For OGL 3.3 deprecation it needs to work on macOS first. :)

So, no artist for now.

Tue, Aug 31, 6:34 PM
Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.

For OGL 3.3 deprecation it needs to work on macOS first. :)

Tue, Aug 31, 6:14 PM
Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.

Okay son on first look it would seem only vertex groups are counted. But somehow though not all of them.

Tue, Aug 31, 5:54 PM
Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.
In D4244#180436, @Stan wrote:

Well it matters a lot for artists since we basically have to duplicate bones to have objects like swords helmets etc.

GPU doesn't care about artists. It just have a limited amount of memory for uniforms.

Tue, Aug 31, 4:33 PM
Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.

Well it matters a lot for artists since we basically have to duplicate bones to have objects like swords helmets etc.

Tue, Aug 31, 4:27 PM
Stan added a comment to D4244: Forbids using more than 64 bones for GPU skinning.

Does that count prop bones ?

Tue, Aug 31, 3:53 PM