height=0 not respected on walls

[Luke-Jr]

A wall with a 0 height should be invisible. Instead 0 seems to take on a "default height" meaning.

[Manuel Moos (z-man)]

This was by design :) Apparently, the designer did not think about the benefit of having a default value applied iff height= is omitted. Feel free to fix, but mind backwards compatibility with maps using old dtds that rely on this.

[Luke-Jr (luke-jr)]

What old DTDs are you referring to? I can't find any that default to 0.

[Manuel Moos (z-man)]

Old == all current. Yeah, they don't define a default. However, it's possible that a map designer using those dts would have put height="0" in there to get a default height wall, and that behavior shouldn't be changed. However, It's more likely he'd just have left the height out; the 0 value is then implicitly added by the float getting code in the parser. Dunno. Just grep the resource repository to see whether height="0" is actively used or not. If it's not used too much, don't bother with adding legacy code.