add comments and make the statefulness of localeIsAdapting more obvious.

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

diff --git a/src/java/org/apache/poi/ss/usermodel/DataFormatter.java b/src/java/org/apache/poi/ss/usermodel/DataFormatter.java
index 308445f9ca90588e2c9f1356942573d853397c27..6771207cc592569c2ff4fe6cec6ca11e96ddf13f 100644 (file)
--- a/src/java/org/apache/poi/ss/usermodel/DataFormatter.java
+++ b/src/java/org/apache/poi/ss/usermodel/DataFormatter.java
@@ -207,7 +207,7 @@ public class DataFormatter implements Observer {
     private Locale locale;
     
     /** stores if the locale should change according to {@link LocaleUtil#getUserLocale()} */
-    private boolean localeIsAdapting = true;
+    private boolean localeIsAdapting;
     
     private class LocaleChangeObservable extends Observable {
         void checkForLocaleChange() {
@@ -265,8 +265,13 @@ public class DataFormatter implements Observer {
      * @param  emulateCSV whether to emulate CSV output.
      */
     private DataFormatter(Locale locale, boolean localeIsAdapting, boolean emulateCSV) {
+        this.localeIsAdapting = true;
         localeChangedObservable.addObserver(this);
+        // localeIsAdapting must be true prior to this first checkForLocaleChange call.
         localeChangedObservable.checkForLocaleChange(locale);
+        // set localeIsAdapting so subsequent checks perform correctly
+        // (whether a specific locale was provided to this DataFormatter or DataFormatter should
+        // adapt to the current user locale as the locale changes)
         this.localeIsAdapting = localeIsAdapting;
         this.emulateCSV = emulateCSV;
     }