string_bytes: add StringBytes::IsValidString()

Performs a quick, non-exhaustive check on the input string to see if
it's compatible with the specified string encoding.

Curently it only checks that hex strings have a length that is a
multiple of two.

diff --git a/src/string_bytes.cc b/src/string_bytes.cc
index a2ee589..e4a34fe 100644 (file)
--- a/src/string_bytes.cc
+++ b/src/string_bytes.cc
@@ -257,6 +257,13 @@ size_t StringBytes::Write(char* buf,
 }
 
 
+bool StringBytes::IsValidString(Handle<String> string, enum encoding enc) {
+  if (enc == HEX && string->Length() % 2 != 0) return false;
+  // TODO(bnoordhuis) Add BASE64 check?
+  return true;
+}
+
+
 // Quick and dirty size calculation
 // Will always be at least big enough, but may have some extra
 // UTF8 can be as much as 3x the size, Base64 can have 1-2 extra bytes

diff --git a/src/string_bytes.h b/src/string_bytes.h
index 05d9277..8071a49 100644 (file)
--- a/src/string_bytes.h
+++ b/src/string_bytes.h
@@ -36,6 +36,11 @@ using v8::Value;
 
 class StringBytes {
  public:
+  // Does the string match the encoding? Quick but non-exhaustive.
+  // Example: a HEX string must have a length that's a multiple of two.
+  // FIXME(bnoordhuis) IsMaybeValidString()? Naming things is hard...
+  static bool IsValidString(Handle<String> string, enum encoding enc);
+
   // Fast, but can be 2 bytes oversized for Base64, and
   // as much as triple UTF-8 strings <= 65536 chars in length
   static size_t StorageSize(Handle<Value> val, enum encoding enc);