Hi,

In the following constructor for GenericLockManager, the LoggerFacade
argument is ignored and a new LoggerFacade is always created.  Thus, the
LoggerFacade that I'm passing in is never used.  This affects the
subclasses of GenericLockManager as well since they all call this
constructor.

    public GenericLockManager(int maxLockLevel, LoggerFacade logger,
long timeoutMSecs,
            long checkThreshholdMSecs) throws IllegalArgumentException {
        if (maxLockLevel < 1)
            throw new IllegalArgumentException("The maximum lock level
must be at least 1 ("
                    + maxLockLevel + " was specified)");
        this.maxLockLevel = maxLockLevel;
        this.logger = logger.createLogger("Locking");
        this.globalTimeoutMSecs = timeoutMSecs;
        this.checkThreshhold = checkThreshholdMSecs;
    }

Is this just an oversight?  Will someone fix it?

Thanks,
Ky



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to