Commit cd8af6a

[Hugo Trippaers]

Min,

I’m checking some findbugs reports and found the following piece of code in 
VolumeDataStoreVO (line 352):

    public void setRefCnt(Long refCnt) {
        if (refCnt > 0) {
            refCnt--;
        }
        else {
            s_logger.warn("We should not try to decrement a zero reference 
count even though our code has guarded");
        }
    }


This doesn’t seem to make a lot of sense to me as the refCnt field doesn’t get 
set in this function. Could you have a look as you committed this and might 
know what the intended behavior is?

Cheers,

Hugo