looks good so far.

I just wonder if this:
+      Locale locale = getLocale();
+      if (null == locale)
+      {
+        RequestContext reqContext = RequestContext.getCurrentInstance();
+        if (reqContext != null)
+        {
+          locale = reqContext.getFormattingLocale();
+        }
+        if (locale == null)
+        {
+           locale = context.getViewRoot().getLocale();
+        }
+      }

could be moved to a private function, since some other place is doing
the same, I think

-M

On Tue, Oct 21, 2008 at 9:46 PM, Anita Anandan <[EMAIL PROTECTED]> wrote:
> Hi all -
>
> I was wondering if one of the Trinidad committees could take a look at
> trinidad-1262 (https://issues.apache.org/jira/browse/TRINIDAD-1262)? It
> needs to be reviewed and committed.
>
> Thanks
> Anita
>



-- 
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf

Reply via email to