2009/3/18 Tim Ellison <t.p.elli...@gmail.com>:
> Thanks for the review...
>
> Sian January wrote:
>> The patch looks fine for this case, but if I was being picky I would
>> wonder if we also need to throw an IOException for an infinite stream
>> that does contain '\n' characters as well as for one that doesn't?
>
> The RI runs forever on a stream of '\n's and we run up to an
> OutOfMemoryException. It's such a contrived case that I really don't
> think we need to hang forever too to match the behavior.
>
>> Also is '\n' ok to use across all platforms or should it be something
>> like System.getProperty(line.separator)?
>
> The manifest format spec says:
> newline : CR LF | LF | CR (not followed by LF)
>
> so I'll change the containsLine test to be
> if (buffer[i] == 0x0A || buffer[i] == 0x0D) {
>
> Sounds reasonable?
Ok - sounds fine
+1 for applying
>
> Thanks again,
> Tim
>
>
>> 2009/3/18 Tim Ellison <t.p.elli...@gmail.com>:
>>> Sian January wrote:
>>>> 2. org.apache.harmony.archive.tests.java.util.jar.ManifestTest fails
>>>> on Windows XP [Tim is working on this]
>>> I have a proposed patch ready for this problem, see
>>>
>>> https://issues.apache.org/jira/browse/HARMONY-6121
>>>
>>> it's not a one-liner, so I'd appreciate another set of eyeballs on it
>>> before nominating it as a commit into M9.
>>>
>>> Thanks,
>>> Tim
>>>
>>>
>>
>>
>>
>
--
Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU
