Re: svn commit: r1607081 - in /jackrabbit/oak/trunk: oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/mk/ oak-commons/src/test/java/org/apache/jackrabbit/oak/commons/mk/ oak-core/src/main/java/org/apache/jackrabbit/oak/kernel/ oak-core/src/main/...

[Jukka Zitting]

Hi,

On Tue, Jul 1, 2014 at 9:57 AM,  <ste...@apache.org> wrote:
> -            return rep.getBlobStore().readBlob(blobId, pos, buff, off, 
> length);
> +            int read = rep.getBlobStore().readBlob(blobId, pos, buff, off, 
> length);
> +            return read < 0 ? 0 : read;

Shouldn't this (and all the other similar lines) be:

    return read < 0 ? -1 : read;

? My build gets stuck in an infinite loop because the read() method returns 0.

BR,

Jukka Zitting