[PATCH] Fix pressure_time calculation for SAC-rate

Linus Torvalds torvalds at linux-foundation.org
Sun Jan 6 17:06:17 PST 2013



On Sun, 6 Jan 2013, Linus Torvalds wrote:
> 
> This fixes the problem I mentioned in the previous commit:
> 
>   "That does make the rounding errors more obvious in the graph, but we 
>    can fix that separately."

Btw, I'm not sure how much of it was the bar/atm confusion, and how much 
of it was due to doing the pressure_time calculation *after* having 
switched 'current' to the next entry. Because the patch had another fix, 
namely doing the calculation with the previous 'current' pressure tracking 
thing, rather than the newly created one. The previous 'current' is the 
correct one to use, because that's the one that is associated with the gap 
from the previous entry to the current entry.

                Linus


More information about the subsurface mailing list