You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Johan Compagner (JIRA)" <ji...@apache.org> on 2008/03/23 16:35:25 UTC

[jira] Assigned: (WICKET-1444) AbstractDecimalConverter has a numberFormats fields that holds NumberFormats, But those are not thread save...

     [ https://issues.apache.org/jira/browse/WICKET-1444?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Johan Compagner reassigned WICKET-1444:
---------------------------------------

    Assignee: Johan Compagner

> AbstractDecimalConverter has a numberFormats fields that holds NumberFormats, But those are not thread save...
> --------------------------------------------------------------------------------------------------------------
>
>                 Key: WICKET-1444
>                 URL: https://issues.apache.org/jira/browse/WICKET-1444
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.2
>            Reporter: Johan Compagner
>            Assignee: Johan Compagner
>
> We really should remove that map because now they can be used multi threaded because for example the DoubleConverter has a static INSTANCE variable..
> The AbstractIntegerConverter doesnt do that caching for example, that returns a new Format instance everytime.
> The problem is that we can remove that map just fine, expects that we have setNumberFormat(Locale,Format) method
> thats is a method that is very wrong because ot the caching, And they should use/override getNumberFormat when you call setNumberFormat..

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.