More readable line.

Add slider for sharpness factor.

Successful build - Chance fights ever on the side of the prudent.

Fix missing this.

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.

Remove usage of this.GetRate()

Missed some copyrights

I used svn mv to move and rename the files. This way allows comparison of the files.
I also updated the copyright year.

Successful build - Chance fights ever on the side of the prudent.

Remove message code.

can you request review ?

Successful build - Chance fights ever on the side of the prudent.

Remove more noise and coupling.

Successful build - Chance fights ever on the side of the prudent.

Move the code to the Autobuildable component to decouple it as much as possible.

@elexis

If its easier, one could also just upload replays and perhaps a link to the mod.

windows build was fixed

Successful build - Chance fights ever on the side of the prudent.

Add missing SDL_clipboard.h include for Windows.

In D2492#110943, @Freagarach wrote:

In D2492#110907, @Stan wrote:

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)

D2492.diff10 KBDownload

(You'd need to update the Foundation test though.)

In D2492#110907, @Stan wrote:

Stan` shoots himself in the foot: Shouldn't this use Status Effects? Could be done in V2.

I'm not sure how one could use Status Effects for this?

In D2492#110907, @Stan wrote:

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)

In D2643#110822, @Stan wrote:

Do you need an icon with a red cross?

In D2642#110937, @gameboy wrote:

I use Intel CPU, and the graphics card USES NVIDIA GTX850M. The operating system is Windows10 64bit. Do you support Intel CPU and NVIDIA series of graphics CARDS in this patch

I use Intel CPU, and the graphics card USES NVIDIA GTX850M. The operating system is Windows10 64bit. Do you support Intel CPU and NVIDIA series of graphics CARDS in this patch

Successful build - Chance fights ever on the side of the prudent.

remove switch

It might be better to store the data in UnitAI as datapairs of class and variation.

Successful build - Chance fights ever on the side of the prudent.

Fix small style issues

In D2642#110911, @Imarok wrote:

I got

ERROR: Failed to compile shader 'shaders/glsl/cas.fs':
0(64) : error C1008: undefined variable "rcp"
0(70) : error C1008: undefined variable "rcp"
0(73) : error C1008: undefined variable "rcp"

Edit: Tested on Ubuntu 18.04 with GTX 960

Successful build - Chance fights ever on the side of the prudent.

Replace rcp(x) to 1/x.
rcp seems to be a performance optimized function for AMD GPUs. As strange, rsqrt is also a special function, which doesn't work with my GPU. I had it replaced bevor my first upload.
@vladislavbelov do you know more about such things? Is it possible to implement a parallel path for AMD hardware?

In D2642#110914, @Stan wrote:

AFAIK @gameboy uses Windows.

AFAIK @gameboy uses Windows.

In D2642#110901, @gameboy wrote:

@OptimusShepard I tested your patch and when I select CAS in the options, the error appears：

ERROR: Failed to compile shader 'shaders/glsl/cas.fs': 0(64) : warning C1503: undefined variable "rcp" 0(70) : warning C1503: undefined variable "rcp" 0(73) : warning C1503: undefined variable "rcp" 0(64) : error C1008: undefined variable "rcp" 0(70) : error C1008: undefined variable "rcp" 0(73) : error C1008: undefined variable "rcp"

I got

ERROR: Failed to compile shader 'shaders/glsl/cas.fs':
0(64) : error C1008: undefined variable "rcp"
0(70) : error C1008: undefined variable "rcp"
0(73) : error C1008: undefined variable "rcp"

Edit: Tested on Ubuntu 18.04 with GTX 960

@Freagarach, can you try your solution with the latest version of the patch? (To move the three functions)

Structure with autobuildable component builds without unit building it.
Modifications from technologies and auras apply correctly.

@OptimusShepard I tested your patch and when I select CAS in the options, the error appears：

Have to use both. Events are not supported in class syntax.

Failing to build on windows.

@elexis
He meant the button on crash window, not that it creates crash.

Random thought for the if else repeat mess: put all strings in a global with MarkForTranslation and get the correct on by constructing it using the template properties. Also translators might like some context

In D1492#69823, @elexis wrote:

setCenterPosition(undefined) still unhealthy, ain't?

part of autobuildable has class syntax and part prototype, should use one

Build failure - The Moirai have given mortals hearts that can endure.

Forgot one line.

Build failure - The Moirai have given mortals hearts that can endure.

Tooltip.

Successful build - Chance fights ever on the side of the prudent.

Use std::vector.

Successful build - Chance fights ever on the side of the prudent.

Translation.

Successful build - Chance fights ever on the side of the prudent.

Remove cancelling

Successful build - Chance fights ever on the side of the prudent.

Fix broken check.

you can move this.reachedtarget section to function and pass it computed distance

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

Successful build - Chance fights ever on the side of the prudent.

comment

• Cancel timers when rate is 0

• Rebased.
• Add test for FF == true for splash.

Successful build - Chance fights ever on the side of the prudent.

But I sent you a paste last week which had the timer in the Autobuildable cmp. and that worked?