Restrict min + max to signed.

asm-js / validator

A reference validator for asm.js.

Apache License 2.0

1.78k stars 154 forks source link

Restrict min + max to signed. #120

Open flagxor opened 8 years ago

flagxor commented 8 years ago

Current firefox seems to restrict these to signed. Allowing any int would seem to be problematic, unless all the args are required to be all unsigned or all signed.

kripken commented 8 years ago

lgtm

kripken commented 8 years ago

@lukewagner, thoughts?

lukewagner commented 8 years ago

That's correct. I've actually kept a list of errata that we've been meaning to apply. wrt the PR, I'm not sure if the spec HTML is updated directly or generated. @dherman?