Welcome to ZDF!

Ah okay. Still, thanks a ton! :mrgreen:

Better, yes. But since P_PointInSector will always return something valid the 'if (sec)' isn't really needed. Good enough, though.

[quote="Graf Zahl"]Why do you use secnum instead of the pointer that was returned by P_PointInSector?[/quote]

Was using the same code as GetSectorFloorZ in ACS. Changed it. Is that better?

Why do you use secnum instead of the pointer that was returned by P_PointInSector?

Imho the name of the enum should be GZAFlags, not GZFlags, with the enum elements having GZAF_ as prefix (the action function is called GetZAt, not GetZat, then the question would be `what's a Zat?`). Also that sounds too familiar with GZDoom :P

Done. I added a couple more flags to disable 3D floor, portal, and midtexture checking in case someone needs it (I can tell this will be handy).

[list][*][b]3DRESTRICT[/b]: Ignore midtextures and 3D floors whose floorz is above the actor's z.
[*][b]NOPORTALS[/b]: Don't check through portals, only the original sector designated.
[*][b]NO3DFLOORS[/b]: Ignore all 3D floors.[/list]

Precisely that. It really only matters if an actor intersects with a 3D floor.

What should I be using for bottomz and topz? Looks like the actor's Z and their Top... Not sure.

I don't think that your submission is really usable, it's missing both 3D floor and portal handling. And that's also the reason why no GetHighestCeiling function exists.
The functions that should be used are:

double HighestCeilingAt(const DVector2 &a, sector_t **resultsec = NULL);
double LowestFloorAt(const DVector2 &a, sector_t **resultsec = NULL);
double NextHighestCeilingAt(double x, double y, double bottomz, double topz, int flags = 0, sector_t **resultsec = NULL, F3DFloor **resultffloor = NULL);
double NextLowestFloorAt(double x, double y, double z, int flags = 0, double steph = 0, sector_t **resultsec = NULL, F3DFloor **resultffloor = NULL);

which are all portal-aware, and the 'Next...' variants also 3D floor aware.

Slightly offtopic, but Graf, is there an equivalent way to get the highest floor or lowest ceiling by chance? I know one exists for a vertex but I'm looking for an opposite of GetLowestFloor function in the source.

Actor Da
{
	+NOINTERACTION
	States
	{
	Spawn:
		TNT1 A 35 NoDelay
		{
			A_Log("My FloorZ Is");
			A_Warp(AAPTR_PLAYER1,x,y,z,0,WARPF_NOCHECKPOSITION|WARPF_ABSOLUTEPOSITION,"Null");
			A_LogFloat(FloorZ);
			A_LogFloat(GetZAt(0,0));
			A_Log("50 Units From Me Is");
			A_LogFloat(GetZAt(50,0));
			
		}
		Stop
	}
}

[url=https://github.com/rheit/zdoom/pull/648]Pull Request[/url]

[b]float GetZAt(x, y, angle, flags, pick_pointer);[/b]
Gets the floor z at x distance ahead and y distance to the side in relative form from the calling actor pointer. Flags are as follows (GZF_ prefix):
[list][*][b]CEILING[/b]: Returns the ceiling z instead of floor.
[*][b]ABSOLUTEPOS[/b]: x and y are absolute positions.
[*][b]ABSOLUTEANG[/b]: angle parameter does not add the pointer's angle to the angle parameter.
[*][b]3DRESTRICT[/b]: Ignore midtextures and 3D floors whose floorz is above the actor's z.
[*][b]NOPORTALS[/b]: Don't check through portals.
[*][b]NO3DFLOORS[/b]: Ignore all 3D floors.[/list]

[b][size=150]Why?[/size][/b]

As it stands, I have in one of my projects a VERY unwieldy system that calls A_Warp to move an actor towards a location, grab the floorz, and head back to the origin. After a while, mounting the A_Warp calls slowly becomes a BIT expensive when relying on multiple actors to get the job done.

Cutting out the need to move at all can enhance this performance by removing the unneeded overhead. So I came up with this. It's basically GetSectorFloor/CeilingZ for DECORATE, but better and enhanced with the same genious relativity of A_SpawnItemEx.

A test sample:

[code]Actor Da
{
+NOINTERACTION
States
{
Spawn:
TNT1 A 35 NoDelay
{
A_Log("My FloorZ Is");
A_Warp(AAPTR_PLAYER1,x,y,z,0,WARPF_NOCHECKPOSITION|WARPF_ABSOLUTEPOSITION,"Null");
A_LogFloat(FloorZ);
A_LogFloat(GetZAt(0,0));
A_Log("50 Units From Me Is");
A_LogFloat(GetZAt(50,0));

}
Stop
}
}[/code]