Sven Neumann wrote: > Hi Geert, > > I have a small patch to scale-region.c that I would to have your opinion > on. I noticed that the current code sometimes does an unneeded copy > operation. This happens when the scale factor is 2^n. For example when > an image of 800x600 pixels is scaled to 400x300. The function > determine_scale() then decides that a first scale step to 400x300 needs > to be made and the result is then scaled again with a scale factor of > 1.0. There's even an optimization n the scale() function for this > special case. > > Attached is a patch that changes determine_scale() to avoid this extra > step. Is there anything that I am missing or do you agree that it should > be safe to apply this? > > > Sven > > > > ------------------------------------------------------------------------ > > _______________________________________________ > Gimp-developer mailing list > Gimp-developer@xxxxxxxxxxxxxxxxxxxxxx > https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer I see no problem with that,it should be safe. _______________________________________________ Gimp-developer mailing list Gimp-developer@xxxxxxxxxxxxxxxxxxxxxx https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
