Speed up Image.resample()#9739
Conversation
Merging this PR will improve performance by 14.75%
|
|
No, they're not, I'm just a dummy and I copy-pasted an |
| for (yy = 0; yy < imOut->ysize; yy++) { | ||
| for (xx = 0; xx < imOut->xsize; xx++) { | ||
| // Verify caller allocated distinct buffers. | ||
| assert(imIn->image8 != imOut->image8); |
There was a problem hiding this comment.
Just for my understanding, why does this need an assert, when similar changes you've made elsewhere don't?
There was a problem hiding this comment.
I added defensive assert()s here since the call chain is more involved than with the other optimizations in this series (where it's plain to see that there's an ImagingNewDirty just nearby).
3 participants
Refs #9649, #9675, #9736, #9737, #9738 (same series, same techniques).
Follows up on #1881.