Re: [mediacapture-image] Add pan and tilt constraints

Yea, so my main concern is that rounding issues with the floats might cause simple equality to not suffice, which would be an unfortunate side effect of using degrees.

Case in point, say the step value is 1000 arc-seconds. When dealing with degrees, that's:

```
step = 1000/3600;  // 0.2777777777777778
a = 45 + step*5;  // 46.388888888888886
b = 45 + step + step + step + step + step;  // 46.38888888888889

if (a != b) {
    alert("Here be dragons");
}
```

I guarantee that's going to mess with people's code. Anyone not familiar with the behavior of floats wouldn't have that kind of situation occur to them, and so they wouldn't check.

-- 
GitHub Notification of comment by dsanders11
Please view or discuss this issue at https://github.com/w3c/mediacapture-image/pull/182#issuecomment-308886949 using your GitHub account

Received on Thursday, 15 June 2017 22:52:09 UTC