Add infinite perspective matrix

[d07RiV]

Support for zFar = infinity for projection matrix.

Dunno why dist build ended up so different.

[stefnotch]

@d07RiV Wouldn't it be cleaner to simply modify

gl-matrix/src/gl-matrix/mat4.js

Line 1268 in f1871a7

export function perspective(out, fovy, aspect, near, far) {

to support setting the far value to Math.Infinity?

[d07RiV]

That seems less intuitive to me (Infinity is not a valid value in any API, and anywhere else in the library), but if you think that's better then maybe.

Null/undefined is probably a better choice, especially since it's the last argument.

[stefnotch]

@d07RiV I'd prefer to not add a ton of functions that might not be used as often.
less intuitive: Well, other than adding another function, how could you make it more intuitive?

[d07RiV]

You think null value for far is better?

[stefnotch]

@d07RiV That seems equally good/bad. I'd say, let's just go with Math.Infinity and add a note in the documentation. Something like @param {number} far Far bound of the frustum, supports Math.Infinity

[stefnotch]

That's also fine. 🙂 If anyone has an issue with it, they can just open an issue.

[d07RiV]

Why not both?

[d07RiV]

Also I'm not sure why build was changed so much, and I didn't generate docs because it would put my time/timezone in every file.

[stefnotch]

@d07RiV Well, the checks passed and the PR adds some useful functionality, so, merging this!
Thank you! 👍