util: Add Clang thread safety annotations for variables guarded by cs_args #13126

 849 | @@ -848,7 +850,12 @@ bool ArgsManager::ReadConfigFiles(std::string& error, bool ignore_invalid_keys)
 850 |          }
 851 |          // if there is an -includeconf in the override args, but it is empty, that means the user
 852 |          // passed '-noincludeconf' on the command line, in which case we should not include anything
 853 | -        if (m_override_args.count("-includeconf") == 0) {
 854 | +        bool emptyIncludeConf;
 855 | +        {
 856 | +            LOCK(cs_args);

---

 387 | @@ -387,6 +388,7 @@ void ArgsManager::WarnForSectionOnlyArgs()
 388 |      // if it's okay to use the default section for this network, don't worry
 389 |      if (m_network == CBaseChainParams::MAIN) return;
 390 |  
 391 | +    LOCK(cs_args);

---

 303 | @@ -304,6 +304,7 @@ class ArgsManagerHelper {
 304 |      static inline bool GetNetBoolArg(const ArgsManager &am, const std::string& net_arg)
 305 |      {
 306 |          std::pair<bool,std::string> found_result(false,std::string());
 307 | +        LOCK(am.cs_args);

---

 890 | @@ -886,7 +891,8 @@ bool ArgsManager::ReadConfigFiles(std::string& error, bool ignore_invalid_keys)
 891 |          }
 892 |          // if there is an -includeconf in the override args, but it is empty, that means the user
 893 |          // passed '-noincludeconf' on the command line, in which case we should not include anything
 894 | -        if (m_override_args.count("-includeconf") == 0) {
 895 | +        bool emptyIncludeConf = m_override_args.count("-includeconf") == 0;
 896 | +        if (emptyIncludeConf) {

---

 870 | @@ -871,8 +871,10 @@ bool ArgsManager::ReadConfigStream(std::istream& stream, std::string& error, boo
 871 |  
 872 |  bool ArgsManager::ReadConfigFiles(std::string& error, bool ignore_invalid_keys)
 873 |  {
 874 | -    LOCK(cs_args);
 875 | -    m_config_args.clear();
 876 | +    {
 877 | +        LOCK(cs_args);
 878 | +        m_config_args.clear();

---