floodfill wraps around

[snoopyjc]

The ImageDraw.floodfill code assumes a negative index will raise an IndexError, but python wraps that around to the last element, which is not the desired effect. This causes a floodfill that hits the left side or the top of the image to wrap around to the other side. Code:

                    try:
                        p = pixel[s, t]
                    except (ValueError, IndexError):
                        pass
                    else:

Fix:

                    try:
                        if s < 0 or t < 0:
                            raise IndexError
                        p = pixel[s, t]
                    except (ValueError, IndexError):
                        pass
                    else:

[radarhere]

Thanks. I've created PR #4017 to resolve this, adjusting your solution slightly.

[hugovk]

Thanks both, merged!

[snoopyjc]

Great - thanks!!

…

-joe Sent from my AT&T iPhone

On Sep 5, 2019, at 4:01 PM, Hugo van Kemenade ***@***.***> wrote: Thanks both, merged! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.