Tweak comments and variable names, to make the wacky colour stuff as BGR not RGB...

git-svn-id: https://svn.apache.org/repos/asf/jakarta/poi/trunk@496782 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/src/scratchpad/src/org/apache/poi/hslf/usermodel/RichTextRun.java b/src/scratchpad/src/org/apache/poi/hslf/usermodel/RichTextRun.java
index 4954b7e7b2997663bd620d6468538429cc505ccd..3fd9c95259f85d56d0834bb12f4069a5b78340a8 100644 (file)
--- a/src/scratchpad/src/org/apache/poi/hslf/usermodel/RichTextRun.java
+++ b/src/scratchpad/src/org/apache/poi/hslf/usermodel/RichTextRun.java
@@ -347,18 +347,20 @@ public class RichTextRun
        }
 
        /**
-        * Sets color of the text, as a int rgb
+        * Sets color of the text, as a int bgr.
+        * (PowerPoint stores as BlueGreenRed, not the more
+        *  usual RedGreenBlue) 
         * @see java.awt.Color
         */
-       public void setFontColor(int rgb) {
-               setCharTextPropVal("font.color", rgb);
+       public void setFontColor(int bgr) {
+               setCharTextPropVal("font.color", bgr);
        }
        
     /**
      * Sets color of the text, as a java.awt.Color
      */
     public void setFontColor(Color color) {
-        //in PowerPont RGB bytes are swapped,
+        // In PowerPont RGB bytes are swapped, as BGR
         int rgb = new Color(color.getBlue(), color.getGreen(), color.getRed(), 254).getRGB();
         setFontColor(rgb);
     }