On 2014/12/08 15:04, David Higgs wrote: > As per an earlier thread on misc@, this fixes sensorsd.conf(5) > parsing of SENSOR_INDICATOR values. Since parsing as integers was both > undocumented and confusing, it is no longer supported. Also, bail on > error if the high/low values don’t create a valid range.
Low/high transitions don't make sense for these types of sensor anyway. That was a quick hack because the indicator sensors don't function as "status" sensors (but I didn't get as far as working out why). > This mimics existing behavior, but still isn't very intuitive. I.e. > “low=Off:high=On” will always be “within” user limits. Should > indicators limits behave differently? > > Feedback is welcome. Specific feedback on the diff, I'm not keen on the case sensitive comparison (so at least use strcasecmp). But it feels to me like if we're changing things in this area we should fix it properly rather than continue the hack but using strings instead of integers. In my opinion the proper fix would be to treat these sensors as "status" sensors.
