Skip to content
GitLab
Closed
Issue created by Neil Brown@nccbMaintainer

[GREENFOOT-837] Add image sharing to GifImage helper class

The GifImage class in Greenfoot does two things: (1) it loads a GIF and splits it into an array of images and (2) it allows the GIF to be run and tells you which image is the valid one at any point in time. The first part is expensive, especially on the gallery where Javascript image creation can be slow. It would be good for GIF files that are used by multiple actors to be able to share that part, but then animate separately. I think the two obvious solutions are either (A) a hidden internal static cache of loaded GIF images or (B) a copy constructor for GifImage that lets it share the loaded images but run at its own rate. A is easier for beginners, but also can cause correctness problems (e.g. if the file is modified between loads) so I think B may be best.

Issue metadata

  • Issue type: Bug
  • Priority: Medium
  • Fix versions: 3.7.0