Fix use init list instead of constructor to trigger cppcheck

Fixes a simple warning to correctly trigger cppcheck

Framework/Kernel/src/DataItem.cpp

@@ -14,14 +14,12 @@ namespace Mantid::Kernel {
 
 /** Default constructor
  */
-DataItem::DataItem() { m_lock = std::make_unique<Poco::RWLock>(); }
+DataItem::DataItem() : m_lock(std::make_unique<Poco::RWLock>()) {}
 
 /** Copy constructor
+ * Always makes a unique lock
  */
-DataItem::DataItem(const DataItem & /*other*/) {
-  // Always make a unique lock!
-  m_lock = std::make_unique<Poco::RWLock>();
-}
+DataItem::DataItem(const DataItem & /*other*/) : m_lock(std::make_unique<Poco::RWLock>()) {}
 
 /**
  * Destructor. Required in cpp do avoid linker errors when other projects try to

buildconfig/Jenkins/cppcheck.sh

@@ -4,7 +4,7 @@ SCRIPT_DIR=$(dirname "$0")
 # If errors slip through to master this can be used to set a non-zero
 # allowed count while those errors are dealt with. This avoids breaking all
 # builds for all developers
-ALLOWED_ERRORS_COUNT=1069
+ALLOWED_ERRORS_COUNT=1067
 
 if [[ ${JOB_NAME} == *pull_requests* ]]; then
     # This relies on the fact pull requests use pull/$PR-NAME