CompareTo code seems odd, ok?

[Dave Brosius]

This comparator seems odd to me, as it never returns (d) but perhaps i'm 
confused, is it ok?

org.apache.jackrabbit.core.query.lucene.sort.ValueComparableWrapper

 public int compareTo(ValueComparableWrapper o) {
        final int d = compare(v, o.getValue());
        if (d != 0) {
            if (reversed) {
                return -d;
            }
            return -d;
        }
        return 0;
    }