Issue 4127 - [emscripten] Convert subscription classes to C++ - Part 1

compiled/String.h

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 331 matching lines @@
   {
     if (!sourceLen)
       return;
 
     assert(source, u"Null buffer passed to OwnedString.append()"_str);
     size_t oldLength = length();
     grow(sourceLen);
     std::memcpy(mBuf + oldLength, source, sizeof(value_type) * sourceLen);
   }
 
+  void append(const char* source, size_type sourceLen)
+  {
+    if (!sourceLen)
+      return;
+
+    assert(source, u"Null buffer passed to OwnedString.append()"_str);

[sergei, 2017/04/12 12:04:31]

I think it makes sense to put this assert before the check above.

[hub, 2017/04/12 12:12:07]

Not sure what the intent is but to me it looks reasonable to firsrt check for
sourceLen = 0 like it is done to allow nullptr, 0 arguments.

[sergei, 2017/04/12 12:14:40]

Ah, sorry, misread it as `if (!source)...`. It's correct.

+    size_t oldLength = length();
+    grow(sourceLen);
+    for (size_t i = 0; i < sourceLen; i++)
+      mBuf[oldLength + i] = source[i];
+  }
+
   void append(const String& str)
   {
     append(str.mBuf, str.length());
   }
 
   void append(value_type c)
   {
     append(&c, 1);
   }
 
   template<typename T,
       typename std::enable_if<std::is_integral<T>::value>::type* = nullptr>
   void append(T num)
   {
     bool negative = false;
     if (num < 0)
     {
       negative = true;
       num = -num;
     }
 
     size_type size = 0;
-    for (int i = num; i; i /= 10)
+    for (T i = num; i; i /= 10)
       size++;
     size = (size ? size : 1);
 
     size_type pos = length();
     grow((negative ? 1 : 0) + size);
 
     if (negative)
       mBuf[pos++] = '-';
 
     for (int i = size - 1; i >= 0; i--)
     {
       mBuf[pos + i] = '0' + (num % 10);
       num /= 10;
     }
   }
 };