Make m_Sector variable of SectorEffect readable

Marisa the Magician · Post by **Marisa the Magician** » Thu Jan 25, 2018 7:41 pm

It's just a small nitpick but, if it were at least readable externally I would be able to stop effects by sector directly from ZScript, rather than have to interface with ACS (which has the added problem of being by sector tag).

Edit: Figured the best way to do this is with a getter function, gonna send in a pull request once I verify that I wrote it right.

Edit 2: The pull request is here.

Post by **Major Cooke** » Sun Jan 28, 2018 11:40 am

Considering that it was suggested ACS functionality would be exported at some point for ZScript, this seems like a legitimate way forward. Also not needing to use TIDs all the time, and just the specific sector a thing is interested in, this will be a considerable boon.

Post by **Graf Zahl** » Sun Jan 28, 2018 11:44 am

I'm not convinced this is the right idea. It'd be better to export the links inside the sector struct that point to the active thinkers.

Marisa the Magician · Sun Jan 28, 2018 11:52 am

Damn it, you could have said that before. I'll be right on it.

ZDoom