Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-12 Thread Konstantin Kolinko
2009/6/12 Mark Thomas ma...@apache.org: Konstantin Kolinko wrote: Thank you for detailed explanation. My analysis is the following: hres.setLocale(locale); call -  o.a.c.Response.setLocale() - o.a.c.connector.ResponseBase.setLocale() In o.a.c.connector.ResponseBase.setLocale() it calls

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-12 Thread Mark Thomas
Konstantin Kolinko wrote: I do not like that your patch changes behavior where it was not broken previously. To be honest, I don't like it either. The fact that we have to provide workarounds for broken browsers that can't follow a spec that couldn't be clearer if it was written in 6 foot high

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-11 Thread Mark Thomas
fha...@apache.org wrote: Modified: tomcat/current/tc4.1.x/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/current/tc4.1.x/STATUS.txt?rev=783570r1=783569r2=783570view=diff == --- tomcat/current/tc4.1.x/STATUS.txt

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-11 Thread Konstantin Kolinko
2009/6/11 Mark Thomas ma...@apache.org: fha...@apache.org wrote: Modified: tomcat/current/tc4.1.x/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/current/tc4.1.x/STATUS.txt?rev=783570r1=783569r2=783570view=diff

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-11 Thread Mark Thomas
Konstantin Kolinko wrote: 2009/6/11 Mark Thomas ma...@apache.org: fha...@apache.org wrote: Modified: tomcat/current/tc4.1.x/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/current/tc4.1.x/STATUS.txt?rev=783570r1=783569r2=783570view=diff

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-11 Thread Konstantin Kolinko
Thank you for detailed explanation. My analysis is the following: hres.setLocale(locale); call - o.a.c.Response.setLocale() - o.a.c.connector.ResponseBase.setLocale() In o.a.c.connector.ResponseBase.setLocale() it calls CharsetMapper.getCharset(locale) and updates the contentType header. The

Re: svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-11 Thread Mark Thomas
Konstantin Kolinko wrote: Thank you for detailed explanation. My analysis is the following: hres.setLocale(locale); call - o.a.c.Response.setLocale() - o.a.c.connector.ResponseBase.setLocale() In o.a.c.connector.ResponseBase.setLocale() it calls CharsetMapper.getCharset(locale) and

svn commit: r783570 - /tomcat/current/tc4.1.x/STATUS.txt

2009-06-10 Thread fhanik
Author: fhanik Date: Wed Jun 10 23:34:57 2009 New Revision: 783570 URL: http://svn.apache.org/viewvc?rev=783570view=rev Log: vote Modified: tomcat/current/tc4.1.x/STATUS.txt Modified: tomcat/current/tc4.1.x/STATUS.txt URL: