Dydzio

@DjWarmonger can you also check corner cases like changed conditions in VCAI::decomposeGoal if they won't trigger code execution where they previously did not. For example goar being both abstract and...

> > > Uhh, checking these "corner cases" took a majority of time ever spent on AI. Yes, it may and probably will cause issue with logic. Yep, with my...

In my VCMI fork the branch is rebased, here in pull request it is not refreshed... what next? Got these messages in command line when doing force push: remote: Unexpected...

@ArseniyShestakov When reworking GetObj -> VisitObj I made it intended to be used to visit enemy heroes for now.

Now comment that actually answers SXX questions: *Switching to no space after * and &. -I think I will stay neutral here after all. That means, I think no space...

I think target syntax should be int ** a; if we stay with current spacebar rules. But anyway, using double pointers at this point will most likemy be some code...

There is option that if we decide to unify some stuff uncrustify cant handle, we might use resharper and then decide what to do with these cases for future maintenance...

@ArseniyShestakov I mean to use resharper one time to unify some additional formatting stuff that we will not care about later because our standard tools will not be able check...

No idea anymore, but based on comments I think I changed the design so VisitObj became new intended goal for visiting heroes somewhere along the way so the fix with...

I will look at this after I buy new PC, should happen before version 1.1 stabilization starts Should not be hard to remake from scratch though