Igor Kushnir authored Feb 12, 2022 and Milian Wolff committed May 28, 2022

ItemRepositoryRegistry::unRegisterRepository() is called only from
within ~ItemRepository(). ~ItemRepository() unconditionally calls
close() right after unRegisterRepository(). There is no need to lock the
repository's mutex in its destructor. So removing the duplicate close()
call along with the repository's mutex lock from unRegisterRepository()
must be safe.