Browse Source

Adding a comment about the strange shared fields in BitmapTextPage
that I'm not going to touch with a ten foot pole.

Paul Speed 9 years ago
parent
commit
3f1c696e26
1 changed files with 7 additions and 0 deletions
  1. 7 0
      jme3-core/src/main/java/com/jme3/font/BitmapTextPage.java

+ 7 - 0
jme3-core/src/main/java/com/jme3/font/BitmapTextPage.java

@@ -123,6 +123,13 @@ class BitmapTextPage extends Geometry {
         return clone;
     }
 
+    // Here is where one might add JmeCloneable related stuff except
+    // the old clone() method doesn't actually bother to clone anything.
+    // The arrays and the pageQuads are shared across all BitmapTextPage
+    // clones and it doesn't seem to bother anything.  That means the
+    // fields could probably just as well be static... but this code is
+    // all very fragile.  I'm not tipping that particular boat today. -pspeed
+
     void assemble(Letters quads) {
         pageQuads.clear();
         quads.rewind();