Assuming that the value written into `in` is always -1 we can rid synchronized block in favour of guarding `in = - 1` with writing into volatile `closedByReader `:

public void close() throws IOException {
    closedByReader = true;
    synchronized (this) {
        in = -1;
    }
}