Print

Print


@ellert : thanks for the PR, on which platform / compiler are you getting this problem? Looking at `regex_replace` signatures I see:

```
template <class traits, class charT>
  basic_string<charT> regex_replace (const charT* s,
          const basic_regex<charT,traits>& rgx,
          const charT* fmt,
          regex_constants::match_flag_type flags = regex_constants::match_default);
```

this should much well a `C string`.

Secondly, instead of creating a temporary `std::string("/")` could you please do
```
static const std::string slash( "/" );
path_rule.m_path_prefix = std::regex_replace( path_rule.m_path_prefix, std::regex("//"), slash );
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/1571#issuecomment-992333053

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1