[ 
https://issues.apache.org/jira/browse/HTTPCORE-39?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12469987
 ] 

Roland Weber commented on HTTPCORE-39:
--------------------------------------

Hello Bindul,

The ReasonPhraseCatalog is an interface. The implementation in the first patch 
is hard-coded, as the code is taken from the HttpStatus class.
I'd like to have an implementation based on ResourceBundle, but I see it as a 
low priority. It's also up for discussion whether that kind of class should be 
in HttpCore. We prefer not to have fixed/default locations for parameters, and 
the bundle could be seen as such.

Thanks for the suggestion,
   Roland


> refactor HttpStatus
> -------------------
>
>                 Key: HTTPCORE-39
>                 URL: https://issues.apache.org/jira/browse/HTTPCORE-39
>             Project: HttpComponents Core
>          Issue Type: Improvement
>          Components: HttpCore
>    Affects Versions: 4.0-alpha3
>            Reporter: Roland Weber
>         Assigned To: Roland Weber
>            Priority: Minor
>             Fix For: 4.0-alpha4
>
>         Attachments: 2k7-02-03-HttpStatus.txt
>
>
> The HttpStatus class not only represents a status code with associated 
> message, it also has static factory methods for creating objects with 
> hard-coded english status messages. Suggested improvements:
> - define HttpStatusFactory in the API
> - move the factory method stuff to impl, for example EnglishHttpStatusFactory
> - reduce HttpStatus class to representation only, final and serializable
> patch to follow when I find the time
> cheers,
>   Roland

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


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to