Memory Leak

[Hanuwa]

I noticed while invoking the MessageBox, that every time a new one opens up it will add about 1mb of memory continuously to a program and it never gets released, it will just keep stacking up, I looked at the code and found the culprit.

The images, these are loaded into memory each time they are called and from my testing they consume quite a bit of resources, I have a proposition, load the images once they are used and stored them in a dictionary so we can just reuse them across different instances. Here is my example of how I kinda did this:

class CTkMessagebox(customtkinter.CTkToplevel):
    ICONS = {
        "check": None,
        "cancel": None,
        "info": None,
        "question": None,
        "warning": None
    }

    def __init__()
        # ...
        self.icon = self.load_icon(icon, icon_size) if icon else None
    
    def load_icon(self, icon, icon_size):
        if icon not in self.ICONS or self.ICONS[icon] is None:
            image_path = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'icons', icon + '.png')
            if icon_size:
                size_height = icon_size[1] if icon_size[1] <= self.height - 100 else self.height - 100
                size = (icon_size[0], size_height)
            else:
                size = (self.height / 4, self.height / 4)
            self.ICONS[icon] = customtkinter.CTkImage(Image.open(image_path), size=size)
        return self.ICONS[icon] 

With this I only notice a slight amount of memory increase and only when a new icon was invoke. This is only a suggestion but I think it could be an useful one, I also want to thank you for how big of a contribution you have provided to the customtkinter library, you have provided tremendous help for many of us.

If you can please test this to see if you can also see this continuous memory increase, it will be very appreciated :)

[Akascape]

@Hanuwa Thanks a lot for pointing this bug. The solution works perfectly, now maximum memory increase will be 5 mb (for default icons) which is pretty fine.