Begin forwarded message:
> From: Marc Hadley <Marc.Hadley_at_Sun.COM>
> Date: May 30, 2008 10:18:32 AM EDT
> To: Mikael Ståldal <mikael.staldal_at_univits.com>
> Subject: Re: JSR 311 Comments
>
> Thanks for the comments, disposition below.
>
> On May 30, 2008, at 8:53 AM, Mikael Ståldal wrote:
>
>> * In javax.ws.rs.ext.MessageBodyWriter#getSize(), the parameter is
>> specified as:
>> "t - the type"
>> shouldn't this instead be:
>> "t - the instance to write."
>>
> Good catch, I'll fix this.
>
>>
>> * Setion 4.3.4 says:
>> An implementation MUST include pre-packaged MessageBodyReader and
>> MessageBodyWriter im-
>> plementations for the following Java and media type combinations:
>> java.lang.String All text media types (text/*).
>> java.io.InputStream All media types (*/*).
>> java.io.Reader All media types (*/*).
>> java.io.File All media types (*/*).
>>
>> How is MessageBodyReader to java.io.File supposed to work? Read to
>> temporary file?
>>
> Exactly.
>
>> Why is java.lang.String only supported for text/* when
>> java.io.Reader is supported for */*? It would be more consistent if
>> both java.lang.String and java.io.Reader was supported for either
>> text/* or */*.
>>
> Good question, I've created issue 41 to track this:
>
> https://jsr311.dev.java.net/issues/show_bug.cgi?id=41
>
>>
>> * Wouldn't it make send for HttpHeaders#getLanguage() to return
>> java.util.Locale?
>>
> Good idea, I've created issue 42 to track this:
>
> https://jsr311.dev.java.net/issues/show_bug.cgi?id=42
>
>> * The specification is quite vauge about what a matrix parameter
>> actually is.
>
> I'll add references to the javadoc.
>
> Thanks again,
> Marc.
>
> ---
> Marc Hadley <marc.hadley at sun.com>
> CTO Office, Sun Microsystems.
>
>
---
Marc Hadley <marc.hadley at sun.com>
CTO Office, Sun Microsystems.