Re: [mediacapture-main] Should we use [EnforceRange] on min/max in constraints?

I agree that `[EnforceRange]` would work better; alternatively, 
`[Clamp]`would not throw, but instead clamp down to the closest 
acceptable value.

(whichever we choose, we would have to apply it to:
* `min`, `max` in `LongRange`
*`ideal`, `exact` in `ConstrainLongRage`
* the `long` type in the `ConstrainLong` typedef (but I'm not sure if 
it's syntactically valid — the WebIDL specs doesn't talk about 
applying it to union types)

I noticed that all the current usages of ConstrainLong are for 
positive values only; I'll raise a separate issue to discuss with we 
should be using `unsigned long` instead.

-- 
GitHub Notification of comment by dontcallmedom
Please view or discuss this issue at 
https://github.com/w3c/mediacapture-main/issues/384#issuecomment-241361095
 using your GitHub account

Received on Monday, 22 August 2016 09:35:10 UTC