The nasty part about the code (not necessarily the patch itself) is that 'false' will be returned from release() in cases of both file missing *and* the release failing.
Maybe it would be easiest to reverse the logic? "Failed to remove lock file" is printed out if the operation fails but *no* output is printed in the two other cases.
The nasty part about the code (not necessarily the patch itself) is that 'false' will be returned from release() in cases of both file missing *and* the release failing.
Maybe it would be easiest to reverse the logic? "Failed to remove lock file" is printed out if the operation fails but *no* output is printed in the two other cases.