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, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1571#issuecomment-992333053", "url": "https://github.com/xrootd/xrootd/pull/1571#issuecomment-992333053", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

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