tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeanfrancois Arcand <jfarc...@apache.org>
Subject Re: cvs commit: jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat5 CoyoteResponse.java
Date Tue, 03 Dec 2002 16:19:13 GMT


Remy Maucherat wrote:

> jfarcand@apache.org wrote:
>
>> jfarcand    2002/12/03 08:04:02
>>
>>   Modified:    coyote/src/java/org/apache/coyote Response.java
>>                coyote/src/java/org/apache/coyote/tomcat5
>>                         CoyoteResponse.java
>>   Log:
>>   As Bill's recommends, do not set a default locale in Response 
>> directly.
>
>
> That seems like a bad idea. AFAIK, a default locale was set before, 
> and I removed it when I refactored the header handling; I think your 
> patch was fine.

>
> OTOH, calling setLocale does not do the same thing (it ends up setting 
> the content-language header, which may not be what you want to do). 

That's true. What I'm trying to do is to only set the locale attribute 
to confrom with the 2.4 spec. I will take a look at the spec 2.2/2.3. 
I'm surprised there is no local by default.

-- Jeanfrancois

>
>
> Remy
>
>
> -- 
> To unsubscribe, e-mail:   
> <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: 
> <mailto:tomcat-dev-help@jakarta.apache.org>
>
>


--
To unsubscribe, e-mail:   <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:tomcat-dev-help@jakarta.apache.org>


Mime
View raw message