You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@freemarker.apache.org by dd...@apache.org on 2017/03/14 20:48:26 UTC
[1/3] incubator-freemarker git commit: Added IntelliJ inspections
settings
Repository: incubator-freemarker
Updated Branches:
refs/heads/3 6d9dbff74 -> 755b25a3e
Added IntelliJ inspections settings
Project: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/commit/2ed315e2
Tree: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/tree/2ed315e2
Diff: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/diff/2ed315e2
Branch: refs/heads/3
Commit: 2ed315e2284fe9513516f092c4d60efae6e3911c
Parents: 6d9dbff
Author: ddekany <dd...@apache.org>
Authored: Mon Mar 13 20:33:30 2017 +0100
Committer: ddekany <dd...@apache.org>
Committed: Mon Mar 13 20:33:30 2017 +0100
----------------------------------------------------------------------
.../IntelliJ-IDEA/Editor-Inspections-FreeMarker.xml | 15 +++++++++++++++
1 file changed, 15 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/2ed315e2/src/ide-settings/IntelliJ-IDEA/Editor-Inspections-FreeMarker.xml
----------------------------------------------------------------------
diff --git a/src/ide-settings/IntelliJ-IDEA/Editor-Inspections-FreeMarker.xml b/src/ide-settings/IntelliJ-IDEA/Editor-Inspections-FreeMarker.xml
new file mode 100644
index 0000000..c2e6c17
--- /dev/null
+++ b/src/ide-settings/IntelliJ-IDEA/Editor-Inspections-FreeMarker.xml
@@ -0,0 +1,15 @@
+<component name="InspectionProjectProfileManager">
+ <profile version="1.0">
+ <option name="myName" value="FreeMarker" />
+ <inspection_tool class="LoggerInitializedWithForeignClass" enabled="false" level="WARNING" enabled_by_default="false">
+ <option name="loggerClassName" value="org.apache.log4j.Logger,org.slf4j.LoggerFactory,org.apache.commons.logging.LogFactory,java.util.logging.Logger" />
+ <option name="loggerFactoryMethodName" value="getLogger,getLogger,getLog,getLogger" />
+ </inspection_tool>
+ <inspection_tool class="MissingOverrideAnnotation" enabled="true" level="WARNING" enabled_by_default="true">
+ <option name="ignoreObjectMethods" value="true" />
+ <option name="ignoreAnonymousClassMethods" value="false" />
+ </inspection_tool>
+ <inspection_tool class="RawTypeCanBeGeneric" enabled="true" level="WARNING" enabled_by_default="true" />
+ <inspection_tool class="RawUseOfParameterizedType" enabled="true" level="WARNING" enabled_by_default="true" />
+ </profile>
+</component>
\ No newline at end of file
[3/3] incubator-freemarker git commit: Forward ported from 2.3-gae:
Posted by dd...@apache.org.
Forward ported from 2.3-gae:
Added Configuration.isXxxExplictlySet and Configuration.unsetXxx methods for the Configuration settings: locale, time_zone, default_encoding. (This can be utilized in frameworks to detect if the application has missed setting these. The backward compatible default values are often unwanted, as they are the default locale, time zone and file encoding of the Java environment.)
The locale and default_encoding configuration settings now supports the special "JVM default" value when set from Java .properties file, or via Configuration.setSettings(Properties), or via the #setting directive. Earlier only the time_zone setting has supported this value.
Ensure that the configuration settings don't depend on the machine that runs the test.
Project: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/commit/755b25a3
Tree: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/tree/755b25a3
Diff: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/diff/755b25a3
Branch: refs/heads/3
Commit: 755b25a3e1d6e0c0363609d9a73ba80210df2e81
Parents: 98fd94e
Author: ddekany <dd...@apache.org>
Authored: Tue Mar 14 21:47:29 2017 +0100
Committer: ddekany <dd...@apache.org>
Committed: Tue Mar 14 21:48:17 2017 +0100
----------------------------------------------------------------------
.../apache/freemarker/core/Configurable.java | 22 ++--
.../apache/freemarker/core/Configuration.java | 118 +++++++++++++++++--
.../freemarker/core/ConfigurationTest.java | 65 +++++++++-
.../apache/freemarker/core/SQLTimeZoneTest.java | 2 +
.../apache/freemarker/test/TemplateTest.java | 18 +++
5 files changed, 204 insertions(+), 21 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/755b25a3/src/main/java/org/apache/freemarker/core/Configurable.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/freemarker/core/Configurable.java b/src/main/java/org/apache/freemarker/core/Configurable.java
index 122a3c0..c2aeb85 100644
--- a/src/main/java/org/apache/freemarker/core/Configurable.java
+++ b/src/main/java/org/apache/freemarker/core/Configurable.java
@@ -91,9 +91,9 @@ import org.apache.freemarker.core.valueformat.TemplateNumberFormatFactory;
public class Configurable {
static final String C_TRUE_FALSE = "true,false";
- private static final String NULL = "null";
- private static final String DEFAULT = "default";
- private static final String JVM_DEFAULT = "JVM default";
+ static final String NULL = "null";
+ static final String DEFAULT = "default";
+ static final String JVM_DEFAULT = "JVM default";
/** Legacy, snake case ({@code like_this}) variation of the setting name. @since 2.3.23 */
public static final String LOCALE_KEY_SNAKE_CASE = "locale";
@@ -361,8 +361,8 @@ public class Configurable {
protected Configurable(Version incompatibleImprovements) {
_CoreAPI.checkVersionNotNullAndSupported(incompatibleImprovements);
parent = null;
- locale = Locale.getDefault();
- timeZone = TimeZone.getDefault();
+ locale = Configuration.getDefaultLocale();
+ timeZone = Configuration.getDefaultTimeZone();
sqlDataAndTimeTimeZone = null;
numberFormat = "number";
timeFormat = "";
@@ -1769,7 +1769,8 @@ public class Configurable {
* <ul>
* <li><p>{@code "locale"}:
* See {@link #setLocale(Locale)}.
- * <br>String value: local codes with the usual format in Java, such as {@code "en_US"}.
+ * <br>String value: local codes with the usual format in Java, such as {@code "en_US"}, or
+ * "JVM default" (ignoring case) to use the default locale of the Java environment.
*
* <li><p>{@code "custom_number_formats"}: See {@link #setCustomNumberFormats(Map)}.
* <br>String value: Interpreted as an <a href="#fm_obe">object builder expression</a>.
@@ -1943,7 +1944,8 @@ public class Configurable {
* {@code "disable"} for {@link Configuration#DISABLE_AUTO_ESCAPING_POLICY}.
*
* <li><p>{@code "default_encoding"}:
- * See {@link Configuration#setDefaultEncoding(String)}.
+ * See {@link Configuration#setDefaultEncoding(String)}; since 2.3.26 also accepts value "JVM default"
+ * (not case sensitive) to set the Java environment default value.
* <br>As the default value is the system default, which can change
* from one server to another, <b>you should always set this!</b>
*
@@ -2136,7 +2138,11 @@ public class Configurable {
boolean unknown = false;
try {
if (LOCALE_KEY.equals(name)) {
- setLocale(_StringUtil.deduceLocale(value));
+ if (JVM_DEFAULT.equalsIgnoreCase(value)) {
+ setLocale(Locale.getDefault());
+ } else {
+ setLocale(_StringUtil.deduceLocale(value));
+ }
} else if (NUMBER_FORMAT_KEY_SNAKE_CASE.equals(name) || NUMBER_FORMAT_KEY_CAMEL_CASE.equals(name)) {
setNumberFormat(value);
} else if (CUSTOM_NUMBER_FORMATS_KEY_SNAKE_CASE.equals(name)
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/755b25a3/src/main/java/org/apache/freemarker/core/Configuration.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/freemarker/core/Configuration.java b/src/main/java/org/apache/freemarker/core/Configuration.java
index 4bf2eae..73e85f1 100644
--- a/src/main/java/org/apache/freemarker/core/Configuration.java
+++ b/src/main/java/org/apache/freemarker/core/Configuration.java
@@ -354,9 +354,6 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
/** The default of {@link #getIncompatibleImprovements()}, currently {@link #VERSION_3_0_0}. */
public static final Version DEFAULT_INCOMPATIBLE_IMPROVEMENTS = Configuration.VERSION_3_0_0;
- private static final String NULL = "null";
- private static final String DEFAULT = "default";
-
private static final Version VERSION;
static {
try {
@@ -418,7 +415,10 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
private boolean objectWrapperExplicitlySet;
private boolean templateExceptionHandlerExplicitlySet;
private boolean logTemplateExceptionsExplicitlySet;
-
+ private boolean localeExplicitlySet;
+ private boolean defaultEncodingExplicitlySet;
+ private boolean timeZoneExplicitlySet;
+
private HashMap/*<String, TemplateModel>*/ sharedVariables = new HashMap();
/**
@@ -428,7 +428,7 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
*/
private HashMap<String, Object> rewrappableSharedVariables = null;
- private String defaultEncoding = _SecurityUtil.getSystemProperty("file.encoding", "utf-8");
+ private String defaultEncoding = getDefaultDefaultEncoding();
private ConcurrentMap localeToCharsetMap = new ConcurrentHashMap();
/**
@@ -491,11 +491,6 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
loadBuiltInSharedVariables();
}
- @Override
- public void setTimeZone(TimeZone timeZone) {
- super.setTimeZone(timeZone);
- }
-
private void createTemplateResolver() {
templateResolver = new DefaultTemplateResolver(
null,
@@ -1123,7 +1118,71 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
public boolean isObjectWrapperExplicitlySet() {
return objectWrapperExplicitlySet;
}
-
+
+ @Override
+ public void setLocale(Locale locale) {
+ super.setLocale(locale);
+ localeExplicitlySet = true;
+ }
+
+ /**
+ * Resets the setting to its default, as if it was never set.
+ *
+ * @since 2.3.26
+ */
+ public void unsetLocale() {
+ if (localeExplicitlySet) {
+ setLocale(getDefaultLocale());
+ localeExplicitlySet = false;
+ }
+ }
+
+ /**
+ * Tells if {@link #setLocale(Locale)} (or equivalent) was already called on this instance, or it just holds the
+ * default value.
+ *
+ * @since 2.3.26
+ */
+ public boolean isLocaleExplicitlySet() {
+ return localeExplicitlySet;
+ }
+
+ static Locale getDefaultLocale() {
+ return Locale.getDefault();
+ }
+
+ @Override
+ public void setTimeZone(TimeZone timeZone) {
+ super.setTimeZone(timeZone);
+ timeZoneExplicitlySet = true;
+ }
+
+ /**
+ * Resets the setting to its default, as if it was never set.
+ *
+ * @since 2.3.26
+ */
+ public void unsetTimeZone() {
+ if (timeZoneExplicitlySet) {
+ setTimeZone(getDefaultTimeZone());
+ timeZoneExplicitlySet = false;
+ }
+ }
+
+ /**
+ * Tells if {@link #setTimeZone(TimeZone)} (or equivalent) was already called on this instance, or it just holds the
+ * default value.
+ *
+ * @since 2.3.26
+ */
+ public boolean isTimeZoneExplicitlySet() {
+ return timeZoneExplicitlySet;
+ }
+
+ static TimeZone getDefaultTimeZone() {
+ return TimeZone.getDefault();
+ }
+
@Override
public void setTemplateExceptionHandler(TemplateExceptionHandler templateExceptionHandler) {
super.setTemplateExceptionHandler(templateExceptionHandler);
@@ -2056,6 +2115,7 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
*/
public void setDefaultEncoding(String encoding) {
defaultEncoding = encoding;
+ defaultEncodingExplicitlySet = true;
}
/**
@@ -2068,6 +2128,36 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
}
/**
+ * Resets the setting to its default, as if it was never set.
+ *
+ * @since 2.3.26
+ */
+ public void unsetDefaultEncoding() {
+ if (defaultEncodingExplicitlySet) {
+ setDefaultEncoding(getDefaultDefaultEncoding());
+ defaultEncodingExplicitlySet = false;
+ }
+ }
+
+ /**
+ * Tells if {@link #setDefaultEncoding(String)} (or equivalent) was already called on this instance, or it just holds the
+ * default value.
+ *
+ * @since 2.3.26
+ */
+ public boolean isDefaultEncodingExplicitlySet() {
+ return defaultEncodingExplicitlySet;
+ }
+
+ static private String getDefaultDefaultEncoding() {
+ return getJVMDefaultEncoding();
+ }
+
+ static private String getJVMDefaultEncoding() {
+ return _SecurityUtil.getSystemProperty("file.encoding", "utf-8");
+ }
+
+ /**
* Gets the preferred character encoding for the given locale, or the
* default encoding if no encoding is set explicitly for the specified
* locale. You can associate encodings with locales using
@@ -2366,7 +2456,11 @@ public class Configuration extends Configurable implements Cloneable, ParserConf
}
if (DEFAULT_ENCODING_KEY_SNAKE_CASE.equals(name) || DEFAULT_ENCODING_KEY_CAMEL_CASE.equals(name)) {
- setDefaultEncoding(value);
+ if (JVM_DEFAULT.equalsIgnoreCase(value)) {
+ setDefaultEncoding(getJVMDefaultEncoding());
+ } else {
+ setDefaultEncoding(value);
+ }
} else if (LOCALIZED_LOOKUP_KEY_SNAKE_CASE.equals(name) || LOCALIZED_LOOKUP_KEY_CAMEL_CASE.equals(name)) {
setLocalizedLookup(_StringUtil.getYesNo(value));
} else if (WHITESPACE_STRIPPING_KEY_SNAKE_CASE.equals(name)
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/755b25a3/src/test/java/org/apache/freemarker/core/ConfigurationTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/freemarker/core/ConfigurationTest.java b/src/test/java/org/apache/freemarker/core/ConfigurationTest.java
index c1650ec..0ad4773 100644
--- a/src/test/java/org/apache/freemarker/core/ConfigurationTest.java
+++ b/src/test/java/org/apache/freemarker/core/ConfigurationTest.java
@@ -1460,7 +1460,70 @@ public class ConfigurationTest extends TestCase {
assertNull(cfg.getLazyAutoImports());
assertTrue(cfg.isLazyAutoImportsSet());
}
-
+
+ public void testLocaleSetting() throws TemplateException, ConfigurationException {
+ Configuration cfg = new Configuration(Configuration.VERSION_3_0_0);
+
+ assertEquals(Locale.getDefault(), cfg.getLocale());
+ assertFalse(cfg.isLocaleExplicitlySet());
+
+ Locale nonDefault = Locale.getDefault().equals(Locale.GERMANY) ? Locale.FRANCE : Locale.GERMANY;
+ cfg.setLocale(nonDefault);
+ assertTrue(cfg.isLocaleExplicitlySet());
+ assertEquals(nonDefault, cfg.getLocale());
+
+ cfg.unsetLocale();
+ assertEquals(Locale.getDefault(), cfg.getLocale());
+ assertFalse(cfg.isLocaleExplicitlySet());
+
+ cfg.setSetting(Configuration.LOCALE_KEY, "JVM default");
+ assertEquals(Locale.getDefault(), cfg.getLocale());
+ assertTrue(cfg.isLocaleExplicitlySet());
+ }
+
+ public void testDefaultEncodingSetting() throws TemplateException, ConfigurationException {
+ Configuration cfg = new Configuration(Configuration.VERSION_3_0_0);
+
+ String defaultFileEncoding = System.getProperty("file.encoding");
+ assertNotNull(defaultFileEncoding);
+
+ assertEquals(defaultFileEncoding, cfg.getDefaultEncoding());
+ assertFalse(cfg.isDefaultEncodingExplicitlySet());
+
+ String nonDefault = defaultFileEncoding.equalsIgnoreCase("UTF-8") ? "ISO-8859-1" : "UTF-8";
+ cfg.setDefaultEncoding(nonDefault);
+ assertTrue(cfg.isDefaultEncodingExplicitlySet());
+ assertEquals(nonDefault, cfg.getDefaultEncoding());
+
+ cfg.unsetDefaultEncoding();
+ assertEquals(defaultFileEncoding, cfg.getDefaultEncoding());
+ assertFalse(cfg.isDefaultEncodingExplicitlySet());
+
+ cfg.setSetting(Configuration.DEFAULT_ENCODING_KEY, "JVM default");
+ assertEquals(defaultFileEncoding, cfg.getDefaultEncoding());
+ assertTrue(cfg.isDefaultEncodingExplicitlySet());
+ }
+
+ public void testTimeZoneSetting() throws TemplateException, ConfigurationException {
+ Configuration cfg = new Configuration(Configuration.VERSION_3_0_0);
+
+ assertEquals(TimeZone.getDefault(), cfg.getTimeZone());
+ assertFalse(cfg.isTimeZoneExplicitlySet());
+
+ TimeZone nonDefault = TimeZone.getDefault().equals(_DateUtil.UTC) ? TimeZone.getTimeZone("PST") : _DateUtil.UTC;
+ cfg.setTimeZone(nonDefault);
+ assertTrue(cfg.isTimeZoneExplicitlySet());
+ assertEquals(nonDefault, cfg.getTimeZone());
+
+ cfg.unsetTimeZone();
+ assertEquals(TimeZone.getDefault(), cfg.getTimeZone());
+ assertFalse(cfg.isTimeZoneExplicitlySet());
+
+ cfg.setSetting(Configuration.TIME_ZONE_KEY, "JVM default");
+ assertEquals(TimeZone.getDefault(), cfg.getTimeZone());
+ assertTrue(cfg.isTimeZoneExplicitlySet());
+ }
+
@Test
public void testGetSettingNamesAreSorted() throws Exception {
Configuration cfg = new Configuration(Configuration.VERSION_3_0_0);
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/755b25a3/src/test/java/org/apache/freemarker/core/SQLTimeZoneTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/freemarker/core/SQLTimeZoneTest.java b/src/test/java/org/apache/freemarker/core/SQLTimeZoneTest.java
index 9ade361..aae82f6 100644
--- a/src/test/java/org/apache/freemarker/core/SQLTimeZoneTest.java
+++ b/src/test/java/org/apache/freemarker/core/SQLTimeZoneTest.java
@@ -133,6 +133,7 @@ public class SQLTimeZoneTest extends TemplateTest {
TimeZone.setDefault(GMT_P02);
try {
Configuration cfg = getConfiguration();
+ cfg.unsetTimeZone();
assertNull(cfg.getSQLDateAndTimeTimeZone());
assertEquals(TimeZone.getDefault(), cfg.getTimeZone());
@@ -148,6 +149,7 @@ public class SQLTimeZoneTest extends TemplateTest {
TimeZone.setDefault(GMT_P02);
try {
Configuration cfg = getConfiguration();
+ cfg.unsetTimeZone();
cfg.setSQLDateAndTimeTimeZone(GMT_P02);
assertOutput(FTL, OUTPUT_BEFORE_SETTING_GMT_CFG_GMT2 + OUTPUT_AFTER_SETTING_GMT_CFG_SQL_DIFFERENT);
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/755b25a3/src/test/java/org/apache/freemarker/test/TemplateTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/freemarker/test/TemplateTest.java b/src/test/java/org/apache/freemarker/test/TemplateTest.java
index 769ea16..d2eeb72 100644
--- a/src/test/java/org/apache/freemarker/test/TemplateTest.java
+++ b/src/test/java/org/apache/freemarker/test/TemplateTest.java
@@ -28,7 +28,9 @@ import java.io.StringWriter;
import java.nio.charset.StandardCharsets;
import java.util.Collections;
import java.util.HashMap;
+import java.util.Locale;
import java.util.Map;
+import java.util.TimeZone;
import org.apache.commons.io.IOUtils;
import org.apache.freemarker.core.Configuration;
@@ -60,6 +62,7 @@ public abstract class TemplateTest {
try {
configuration = createConfiguration();
addCommonTemplates();
+ applyEnvironmentIndependentDefaults();
} catch (Exception e) {
throw new RuntimeException("Failed to set up configuration for the test", e);
}
@@ -67,6 +70,21 @@ public abstract class TemplateTest {
return configuration;
}
+ /**
+ * Ensure that the configuration settings don't depend on the machine that runs the test.
+ */
+ private void applyEnvironmentIndependentDefaults() {
+ if (!configuration.isLocaleExplicitlySet()) {
+ configuration.setLocale(Locale.US);
+ }
+ if (!configuration.isDefaultEncodingExplicitlySet()) {
+ configuration.setDefaultEncoding(StandardCharsets.UTF_8.name());
+ }
+ if (!configuration.isTimeZoneExplicitlySet()) {
+ configuration.setTimeZone(TimeZone.getTimeZone("GMT+1"));
+ }
+ }
+
protected final void setConfiguration(Configuration configuration) {
this.configuration = configuration;
}
[2/3] incubator-freemarker git commit: Environment independence of
tests: DecimalFormat symbols
Posted by dd...@apache.org.
Environment independence of tests: DecimalFormat symbols
Project: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/commit/98fd94e5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/tree/98fd94e5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/diff/98fd94e5
Branch: refs/heads/3
Commit: 98fd94e5737ef204df6c85a0d32840d828708b4b
Parents: 2ed315e
Author: ddekany <dd...@apache.org>
Authored: Tue Mar 14 15:26:36 2017 +0100
Committer: ddekany <dd...@apache.org>
Committed: Tue Mar 14 15:26:36 2017 +0100
----------------------------------------------------------------------
.../impl/ExtendedDecimalFormatTest.java | 36 +++++++++++---------
1 file changed, 20 insertions(+), 16 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/98fd94e5/src/test/java/org/apache/freemarker/core/valueformat/impl/ExtendedDecimalFormatTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/freemarker/core/valueformat/impl/ExtendedDecimalFormatTest.java b/src/test/java/org/apache/freemarker/core/valueformat/impl/ExtendedDecimalFormatTest.java
index 1146676..9625a5e 100644
--- a/src/test/java/org/apache/freemarker/core/valueformat/impl/ExtendedDecimalFormatTest.java
+++ b/src/test/java/org/apache/freemarker/core/valueformat/impl/ExtendedDecimalFormatTest.java
@@ -24,6 +24,7 @@ import static org.junit.Assert.*;
import java.io.IOException;
import java.text.DecimalFormat;
+import java.text.DecimalFormatSymbols;
import java.text.ParseException;
import java.util.Locale;
@@ -35,12 +36,13 @@ import org.junit.Test;
public class ExtendedDecimalFormatTest extends TemplateTest {
private static final Locale LOC = Locale.US;
-
+ private static final DecimalFormatSymbols SYMS = DecimalFormatSymbols.getInstance(LOC);
+
@Test
public void testNonExtended() throws ParseException {
for (String fStr : new String[] { "0.00", "0.###", "#,#0.###", "#0.####", "0.0;m", "0.0;",
"0'x'", "0'x';'m'", "0';'", "0';';m", "0';';'#'m';'", "0';;'", "" }) {
- assertFormatsEquivalent(new DecimalFormat(fStr), ExtendedDecimalFormatParser.parse(fStr, LOC));
+ assertFormatsEquivalent(new DecimalFormat(fStr, SYMS), ExtendedDecimalFormatParser.parse(fStr, LOC));
}
try {
@@ -58,18 +60,20 @@ public class ExtendedDecimalFormatTest extends TemplateTest {
@Test
public void testNonExtended2() throws ParseException {
- assertFormatsEquivalent(new DecimalFormat("0.0"), ExtendedDecimalFormatParser.parse("0.0;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0.0"), ExtendedDecimalFormatParser.parse("0.0;;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0.0;m"), ExtendedDecimalFormatParser.parse("0.0;m;", LOC));
- assertFormatsEquivalent(new DecimalFormat(""), ExtendedDecimalFormatParser.parse(";;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0'x'"), ExtendedDecimalFormatParser.parse("0'x';;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0'x';'m'"), ExtendedDecimalFormatParser.parse("0'x';'m';", LOC));
- assertFormatsEquivalent(new DecimalFormat("0';'"), ExtendedDecimalFormatParser.parse("0';';;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0';';m"), ExtendedDecimalFormatParser.parse("0';';m;", LOC));
- assertFormatsEquivalent(new DecimalFormat("0';';'#'m';'"), ExtendedDecimalFormatParser.parse("0';';'#'m';';",
- LOC));
- assertFormatsEquivalent(new DecimalFormat("0';;'"), ExtendedDecimalFormatParser.parse("0';;';;", LOC));
-
+ assertFormatsEquivalent(new DecimalFormat("0.0", SYMS), ExtendedDecimalFormatParser.parse("0.0;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0.0", SYMS), ExtendedDecimalFormatParser.parse("0.0;;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0.0;m", SYMS), ExtendedDecimalFormatParser.parse("0.0;m;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("", SYMS), ExtendedDecimalFormatParser.parse(";;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0'x'", SYMS), ExtendedDecimalFormatParser.parse("0'x';;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0'x';'m'", SYMS),
+ ExtendedDecimalFormatParser.parse("0'x';'m';", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0';'", SYMS), ExtendedDecimalFormatParser.parse("0';';;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0';';m", SYMS), ExtendedDecimalFormatParser.parse("0';';m;", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0';';'#'m';'", SYMS),
+ ExtendedDecimalFormatParser.parse("0';';'#'m';';", LOC));
+ assertFormatsEquivalent(new DecimalFormat("0';;'", SYMS),
+ ExtendedDecimalFormatParser.parse("0';;';;", LOC));
+
try {
new DecimalFormat(";m");
fail();
@@ -320,11 +324,11 @@ public class ExtendedDecimalFormatTest extends TemplateTest {
private void assertFormatsEquivalent(DecimalFormat dfExpected, DecimalFormat dfActual) {
for (int signum : new int[] { 1, -1 }) {
- assertFormatsEquivalent(dfExpected, dfActual, signum * 0);
+ assertFormatsEquivalent(dfExpected, dfActual, 0);
assertFormatsEquivalent(dfExpected, dfActual, signum * 0.5);
assertFormatsEquivalent(dfExpected, dfActual, signum * 0.25);
assertFormatsEquivalent(dfExpected, dfActual, signum * 0.125);
- assertFormatsEquivalent(dfExpected, dfActual, signum * 1);
+ assertFormatsEquivalent(dfExpected, dfActual, signum);
assertFormatsEquivalent(dfExpected, dfActual, signum * 10);
assertFormatsEquivalent(dfExpected, dfActual, signum * 100);
assertFormatsEquivalent(dfExpected, dfActual, signum * 1000);