ActorMover can't make an actor non-solid

According to this here Classes:ActorMover

128: If the thing being moved is normally solid, make it nonsolid so that it can't be blocked.

However I have found out that's actually not the case.
According to this it seems the ActorMover is changing its own flags instead of its tracer's flags. Seems like a typo?
https://github.com/coelckers/gzdoom/blo ... ra.zs#L492

actormover-nonsolidtest.wad

Have a test map, the moving green pillar is supposed to be non-solid, but you can stand in its way and it'll be blocked.
The ActorMover is using the 128 bit.

EDIT: Actually, any flag changes it's supposed to do are done on itself and not the tracer, regardless if 128 is used or not.
This seems wrong?

You do not have the required permissions to view the files attached to this post.

Not sure if this is relevant, but yes, apparently ZDoom's version did actually change the tracer's flags. So it's definitely broken.
https://github.com/rheit/zdoom/blob/mas ... a.cpp#L542

Made a PR: https://github.com/coelckers/gzdoom/pull/1523

Thank you kindly!
Although the fix is incomplete.

Other flags that are being changed is bNoGravity here
And bInvulnerable and bDormant here

The whole issue seems to be contained in the Activate() override which is pretty short.
Just to compare (again), here's ZDoom's version of ActorMover's Activate()

I fixed what you reported and the test wad works.

Another PR: https://github.com/coelckers/gzdoom/pull/1524

Apologies for the inconvenience.
I should have caught the whole thing in the OP but didn't notice it until later.

Thanks again.

No big deal, i didn't catch it either. It was an inconvenience for the ZZDoom release, i could have moved the tag but being myself i had to do a hard reset. xD