[Gimp-developer] scale-region.c

[Geert Jordaens]

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 at lists.XCF.Berkeley.EDU
> https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer
I see no problem with that,it should be safe.