Changes

Jump to: navigation, search

Kraken Info

91 bytes added, 05:13, 22 October 2010
imaging-desaturate
It iterates through each pixel, setting the R/G/B values all to the the same value, which is a function of the prior R/G/B values.
This desaturation loop is repeated 200 times. '''Mozilla-specific things.''' The first time the desaturation loop runs, the (R, G, B) values are converted from integers to doubles. The remaining 199 times, the values are doubles to begin with. The trace code generated assumes the array elements are integers, because that's what is seen at record-time, but in the end 99.5% of the time it's not, so we end up taking a particular side-exit every time to a second trace fragment.
The program is kind of stupid for (a) not truncating the desaturated R/G/B values and (b) desaturating the image 200 times in a row. Indeed, the benchmarks admits as much for (b):
Pixastic.Actions.desaturate.process(params);
But we have [https://bugzilla.mozilla.org/show_bug.cgi?id=603841 Bug 603841] is open to deal with it as change this behaviour. '''Mozilla-specific things.''' The trace code generated assumes the array elements are integers, because that's what isseen at record-time, but in the end 99.5% of the time it's not, so we end up taking a particular side-exit every time to a second trace fragment..  If bug 603841 isn't fixed, it would be good if we could somehow (using the oracle?) detect that the side-exit is hot and recompile the fragment.
== json-parse-financial ==
Confirm
1,345
edits

Navigation menu