Cleanup

src/engine/main.cpp

Index: src/engine/main.cpp
===================================================================
--- a/src/engine/main.cpp
+++ b/src/engine/main.cpp
@@ -143,6 +143,31 @@
         filterEngine->GetFilter(text)->RemoveFromList();
         break;
       }
+      case Communication::PROC_SET_PREF:
+      {
+        std::string prefName = "";
+        std::string prefValue = "";

[Wladimir Palant, 2013/07/11 12:53:10]

Please don't initialize these variables, their initial value is ignored and
overwritten below.

+        request >> prefName;
+        request >> prefValue;

[Wladimir Palant, 2013/07/11 12:53:10]

These lines can be merged:

request >> prefName >> prefValue;

+        filterEngine->SetPref(prefName, filterEngine->GetJsEngine()->NewValue(prefValue));

[Wladimir Palant, 2013/07/11 12:53:10]

What about preferences with non-string values? This needs to support strings,
numbers (int64) and booleans to ensure type safety. The preference you are
actually setting is a boolean.

How about sending Communication::ValueType::TYPE_STRING,
Communication::ValueType::TYPE_INT64 or Communication::ValueType::TYPE_BOOL to
indicate the value type? Same in PROC_GET_PREF of course.

Unfortunately, CAdblockPlusClient will need multiple versions of GetPref/SetPref
functions then.

+        break;
+      }
+      case Communication::PROC_GET_PREF:
+      {
+        std::string name;
+        request >> name;
+
+        AdblockPlus::JsValuePtr valuePtr = filterEngine->GetPref(name);
+        if ((valuePtr->IsNull()) || (!valuePtr->IsString()))

[Wladimir Palant, 2013/07/11 12:53:10]

I think checking valuePtr->IsString() is sufficient - if it is a string then it
definitely isn't null. See also UndefinedValue and NullValue tests for the
JsValue class.

+          response << 0;

[Wladimir Palant, 2013/07/11 12:53:10]

Please don't use an integer as a boolean, this should be << false (and << true
below of course).

+        else
+        {
+          response << 1;
+          response << valuePtr->AsString();

[Wladimir Palant, 2013/07/11 12:53:10]

These two lines can be merged:

  response << true << valuePtr->AsString();

+        }
+        break;
+      }
+
     }
     return response;
   }