You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by so...@apache.org on 2011/09/09 20:21:37 UTC
svn commit: r1167323 - in /myfaces/trinidad/trunk:
trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/
trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/
trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/
trinida...
Author: sobryan
Date: Fri Sep 9 18:21:36 2011
New Revision: 1167323
URL: http://svn.apache.org/viewvc?rev=1167323&view=rev
Log:
Reverting TRINIDAD-2116
Modified:
myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/FastMessageFormat.java
myfaces/trinidad/trunk/trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/FastMessageFormatTest.java
myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java
myfaces/trinidad/trunk/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js
Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/FastMessageFormat.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/FastMessageFormat.java?rev=1167323&r1=1167322&r2=1167323&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/FastMessageFormat.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/util/FastMessageFormat.java Fri Sep 9 18:21:36 2011
@@ -86,27 +86,7 @@ public class FastMessageFormat
for (int i = 0; i < formatLength; i++)
{
char ch = _formatText[i];
-
- // Check for doubled-up quotes
- if (ch == '\'' && (i+1 < formatLength) && (_formatText[i+1] == '\''))
- {
- /*
- The tranlations tool uses special processing for single quotation marks that appear in
- resource strings. In ResourceBundle files, if a message contains an argument placeholder,
- e.g. {0} or {1}, it is assumed to be a format pattern for MessageFormat and the translation
- tool automatically doubles all single quotes inside such a message. If the message contains
- no arguments, it is expected to be used directly after loading from the bundle, without
- processing with MessageFormat. Therefore, no quotes are doubled automatically in such a
- message. This code change accounts for the presence of doubled quotes.
- */
-
- buffer.append(_formatText, lastStart, i - lastStart);
-
- // skip one of the double quotes.
- i++;
- lastStart = i;
- }
- else if (ch == '{')
+ if (ch == '{')
{
// Only check for single digit patterns that have an associated token.
if (i + 2 < formatLength && _formatText[i + 2] == '}')
Modified: myfaces/trinidad/trunk/trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/FastMessageFormatTest.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/FastMessageFormatTest.java?rev=1167323&r1=1167322&r2=1167323&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/FastMessageFormatTest.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/test/java/org/apache/myfaces/trinidad/util/FastMessageFormatTest.java Fri Sep 9 18:21:36 2011
@@ -54,16 +54,6 @@ public class FastMessageFormatTest
String[] params = { "beef", "kosher", null };
String result = fmf.format(params);
assertEquals(result, "beef {{3} isn't {} {a} {12a}kosher {");
-
- // tests doubled single quotes.
- // Same as above, except doubled single quotes are replaced
- // with a single single quote.
- // Expected result: "beef {{3} isn't {} {a} {12a}kosher {"
- FastMessageFormat fmf2 =
- new FastMessageFormat("{0} {{3} isn''t {} {a} {12a}{2}{1} {");
- String[] params2 = { "beef", "kosher", null };
- result = fmf2.format(params2);
- assertEquals(result, "beef {{3} isn't {} {a} {12a}kosher {");
}
}
Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java?rev=1167323&r1=1167322&r2=1167323&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/config/GlobalConfiguratorImpl.java Fri Sep 9 18:21:36 2011
@@ -428,8 +428,8 @@ public final class GlobalConfiguratorImp
}
finally
{
- //Do cleanup of anything which may have use the thread local manager during
- //init.
+ //Do cleanup of anything which may have use the thread local manager during
+ //init.
_releaseManagedThreadLocals();
_initLock.unlock();
}
Modified: myfaces/trinidad/trunk/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js?rev=1167323&r1=1167322&r2=1167323&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js (original)
+++ myfaces/trinidad/trunk/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js Fri Sep 9 18:21:36 2011
@@ -961,7 +961,6 @@ TrFastMessageFormatUtils.format = functi
// arguments[1], ..., arguments[arguments.length-1]
var formatLength = formatString.length;
var tokenCount = arguments.length - 1;
- var substr = null;
// Use the javascript StringBuffer technique.
var buffer = [];
@@ -971,18 +970,7 @@ TrFastMessageFormatUtils.format = functi
{
// IE7 does not support the string[index] syntax, so use string.charAt(index) instead.
var ch = formatString.charAt(i);
-
- // Check for a double quoted (escape) character
- if (ch == '\'' && (i + 1 < formatLength) && (formatString.charAt(i+1) == '\''))
- {
- substr = formatString.substring(lastStart, i - lastStart);
- buffer.push(substr);
-
- // skip one of the double quotes.
- i++;
- lastStart = i;
- }
- else if (ch == '{')
+ if (ch == '{')
{
// Only check for single digit patterns that have an associated token.
if (i + 2 < formatLength && formatString.charAt(i+2) == '}')
@@ -991,7 +979,7 @@ TrFastMessageFormatUtils.format = functi
if (tokenIndex >= 0 && tokenIndex < tokenCount)
{
// Use the javascript StringBuffer technique for append(string)
- substr = formatString.substring(lastStart, i);
+ var substr = formatString.substring(lastStart, i);
buffer.push(substr);
var token = arguments[tokenIndex+1];