Hi Tim, 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?
Also is '\n' ok to use across all platforms or should it be something like System.getProperty(line.separator)? Thanks, Sian 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
