RjY - 2008-11-27

Fixed in 2.5.0.

Pointless rambling follows, stop reading here.

I wondered for ages whether or not this was worth it, I mean, a single convex sector, who needs support for that trivial a map, right? BSP, Killough's node builder, doesn't even produce such builds, it always generates at least one node and two subsectors. So I let the report sit here for nearly a year, gnawing at the back of my mind.

Eventually it gnawed hard enough that I tried to compare our code with Chocolate Doom's, and it turned out it was only a minor thing, a couple of lines removed from R_PointInSubsector in Boom (I guess as an optimisation made by Killough, knowing his nodebuilder never produced maps that would need the special case) I figured, might as well add them back in...

...and the instant CIA eructates the commit message into #doom-tech, somebody (I think it was TheGreenHerring) pipes up with "what, someone's made that simple a map?"

Oh well, such is life. :)