Oops, memrchr is a GNU extension.

14486ce6 · Kenton Varda · 4a14f8e1 · 14486ce6
Commit 14486ce6 authored Aug 09, 2013 by Kenton Varda
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

string.h c++/src/kj/string.h +5 -5

No files found.
--- a/c++/src/kj/string.h
+++ b/c++/src/kj/string.h
@@ -402,12 +402,12 @@ inline Maybe<size_t> StringPtr::findFirst(char c) const {
 }
 inline Maybe<size_t> StringPtr::findLast(char c) const {
-  const char* pos = reinterpret_cast<const char*>(memrchr(content.begin(), c, size()));
+  for (size_t i = size(); i > 0; --i) {
-  if (pos == nullptr) {
+    if (content[i-1] == c) {
-    return nullptr;
+      return i-1;
-  } else {
+    }
-    return pos - content.begin();
  }
+  return nullptr;
 }
 inline String::operator ArrayPtr<char>() {