> -----Original Message-----
> From: Remy Maucherat [mailto:[EMAIL PROTECTED]
> Sent: Tuesday, January 24, 2006 12:52 AM
> To: Tomcat Developers List
> Subject: Re: svn commit: r371765 -
> /tomcat/container/tc5.5.x/catalina/src/share/org/apache/catali
> na/connector/Response.java
>
> Bill Barker wrote:
> >> Author: remm
> >> Date: Mon Jan 23 17:13:19 2006
> >> New Revision: 371765
> >>
> >> URL: http://svn.apache.org/viewcvs?rev=371765&view=rev
> >> Log:
> >> - Remove nonsensical systematic inclusion on ISO-8859-1
> >> charset in the content type, which is noth
> >> useless and inefficient.
> >>
> >
> > -1
> > Sending the charset used by the Writer is very clearly
> required by the
> > servlet spec.
>
> Thanks, I expected no less coming from you :) I will revert my patch.
>
> A couple questions for your enjoyment:
> 1) Is this relevant or irrelevant from the HTTP specification
> perspective ?
It's relevant to the browser trying to display the code. If you've
configured your browser's default encoding to EUC-JP, without the charset
you'll see a big mess when you hit a latin-1 page ;-).
> 2) Does this mean we're running the following ultra efficient code (I
> don't even know why I accepted this stuff back then. It must
> have been
> that this has been done gradually through many many commits) for each
> request that uses a writer ?
>
Yup, that's what it means :). I'm sure you've played the blame-game by now,
and I'm not interested enough to do it myself. It looks like it's trying to
avoid computing the entire header value each time the characterEncoding
changes.
> public String getContentType() {
>
> String ret = contentType;
>
> if (ret != null
> && characterEncoding != null
> && charsetSet) {
> ret = ret + ";charset=" + characterEncoding;
> }
>
> return ret;
> }
>
> Rémy
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
>
This message is intended only for the use of the person(s) listed above as the
intended recipient(s), and may contain information that is PRIVILEGED and
CONFIDENTIAL. If you are not an intended recipient, you may not read, copy, or
distribute this message or any attachment. If you received this communication
in error, please notify us immediately by e-mail and then delete all copies of
this message and any attachments.
In addition you should be aware that ordinary (unencrypted) e-mail sent through
the Internet is not secure. Do not send confidential or sensitive information,
such as social security numbers, account numbers, personal identification
numbers and passwords, to us via ordinary (unencrypted) e-mail.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
