Set default to false, fix setting in Open with Options, minor tweaks to setting language

Author: 0cyn

Plugin.cpp

@@ -42,11 +42,11 @@ BINARYNINJAPLUGIN bool CorePluginInit()
     BinaryNinja::LogRegistry::CreateLogger(PluginLoggerName);
 
     auto settings = BinaryNinja::Settings::Instance();
-    settings->RegisterSetting("core.function.objectiveC.assumeMessageSendTarget",
+    settings->RegisterSetting("core.function.objectiveC.rewriteMessageSendTarget",
         R"({
-		"title" : "Rewrite objc_msgSend calls to first visible implementation",
+		"title" : "Rewrite objc_msgSend calls in IL",
 		"type" : "boolean",
-		"default" : true,
+		"default" : false,
 		"description" : "Message sends of selectors with any visible implementation are replaced with a direct call to the first visible implementation. Note that this can produce false positives if the selector is implemented by more than one class, or shares a name with a method from a system framework."
 		})");
 

Workflow.cpp

@@ -147,7 +147,7 @@ bool Workflow::rewriteMethodCall(LLILFunctionRef ssa, size_t insnIndex)
     ssa->GetFunction()->SetAutoCallTypeAdjustment(ssa->GetFunction()->GetArchitecture(), insn.address, {funcType, BN_DEFAULT_CONFIDENCE});
     // --
 
-    if (!BinaryNinja::Settings::Instance()->Get<bool>("core.function.objectiveC.assumeMessageSendTarget"))
+    if (!BinaryNinja::Settings::Instance()->Get<bool>("core.function.objectiveC.rewriteMessageSendTarget", bv))
         return false;
 
     // Check the analysis info for a selector reference corresponding to the