chore: fix compilation pre-C++17 (#5)

addaleax · web-flow · commit b8ad898250a1 · 2024-10-30T11:33:42.000+01:00
`.data()` returns `const char*` rather than `char*` before C++17.
It's also not needed in this particular case.
diff --git a/src/certs.cc b/src/certs.cc
@@ -53,7 +53,7 @@ void failOnError(OSStatus status, const char* error) {
           CFStringGetLength(str.get()),
 	        kCFStringEncodingUTF8) +
       1);
-    CFStringGetCString(str.get(), &msg.data()[offset], msg.size() - offset, kCFStringEncodingUTF8);
+    CFStringGetCString(str.get(), &msg[offset], msg.size() - offset, kCFStringEncodingUTF8);
     msg.resize(strlen(msg.data()));
     throw std::runtime_error(msg);
   }

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ void failOnError(OSStatus status, const char* error) {`
`53`	`53`	`CFStringGetLength(str.get()),`
`54`	`54`	`kCFStringEncodingUTF8) +`
`55`	`55`	`1);`
`56`		`- CFStringGetCString(str.get(), &msg.data()[offset], msg.size() - offset, kCFStringEncodingUTF8);`
	`56`	`+ CFStringGetCString(str.get(), &msg[offset], msg.size() - offset, kCFStringEncodingUTF8);`
`57`	`57`	`msg.resize(strlen(msg.data()));`
`58`	`58`	`throw std::runtime_error(msg);`
`59`	`59`	`}`