Changes between Version 7 and Version 8 of SCENE_COMMONS_R1

Timestamp:

06/15/09 16:50:47 (16 years ago)

Author:

deni

Comment:

--

SCENE_COMMONS_R1

@@ -89 +89 @@
  * A new private field will be added in ImmImage to store the image multiplied by recently used colours - HashMap<ImmColor, ImmImage> colorImgCache.
  * A new method for multiplying the image by given color will be added in ImmImage. It will check if the image multiplied by the given color is already cached. If it is it will return the cached value. If not - it will compute it and put it in the cache. 
+There is a method that multiplies the image pixels by a color in ElementHelper, it will be copied here and removed from ElementHelper.
+
+'''Caching recently loaded ImmImages''' will prevent us from loading the same image more than once.
+ * All constructors except the one with BufferedImage will be deleted or made private.
+ * A new method will be created instead of them:
+
+{{{
+        /**
+         * Loads an ImmImage from the given URL.
+         * @param url
+         *              The URL from which to load the image.
+         * @return
+         *              The loaded image
+         */
+        public static ImmImage loadCached(URL url) {
+                if (!loadedImages.containsKey(url)) {
+                        try {
+                                InputStream is = null;
+                                try {
+                                        is = url.openStream();
+                                        loadedImages.put(url, new ImmImage(is));
+                                } finally {
+                                        if (is != null) {
+                                                is.close();
+                                        }
+                                }
+                        } catch (IOException e) {
+                                throw new RuntimeException(e);
+                        }
+                }
+                return loadedImages.get(url);
+        }
+}}}
+
+It is a good idea to create a new class representing a map which keeps weak/soft references to its values. Otherwise we could run out of memory. [[BR]]
+However most probably this won't be done now.