Make call param hints more configurable & support more param types for hint deduplication

Author: emesare

basicblock.cpp

@@ -113,6 +113,17 @@ void DisassemblySettings::SetAddressBaseOffset(uint64_t addressBaseOffset)
 	BNSetDisassemblyAddressBaseOffset(m_object, addressBaseOffset);
 }
 
+BNDisassemblyCallParameterHints DisassemblySettings::GetCallParameterHints() const
+{
+	return BNGetDisassemblyCallParameterHints(m_object);
+}
+
+
+void DisassemblySettings::SetCallParameterHints(BNDisassemblyCallParameterHints hints)
+{
+	BNSetDisassemblyCallParameterHints(m_object, hints);
+}
+
 
 DisassemblyTextLine::DisassemblyTextLine()
 {

binaryninjaapi.h

@@ -9824,6 +9824,8 @@ namespace BinaryNinja {
 		void SetAddressMode(BNDisassemblyAddressMode mode);
 		uint64_t GetAddressBaseOffset() const;
 		void SetAddressBaseOffset(uint64_t addressBaseOffset);
+		BNDisassemblyCallParameterHints GetCallParameterHints() const;
+		void SetCallParameterHints(BNDisassemblyCallParameterHints hints);
 	};
 
 	/*!

binaryninjacore.h

@@ -697,7 +697,6 @@ extern "C"
 		ExpandLongOpcode = 2,
 		ShowVariablesAtTopOfGraph = 3,
 		ShowVariableTypesWhenAssigned = 4,
-		ShowCallParameterNames = 6,
 		ShowRegisterHighlight = 7,
 		ShowFunctionAddress = 8,
 		ShowFunctionHeader = 9,
@@ -731,6 +730,13 @@ extern "C"
 		DisassemblyAddressModeFlagsMask = 0xFFFF0000,
 	} BNDisassemblyAddressMode;
 
+	typedef enum BNDisassemblyCallParameterHints
+	{
+		NeverShowMatchingParameterHints,
+		AlwaysShowParameterHints,
+		NeverShowParameterHints,
+	} BNDisassemblyCallParameterHints;
+
 	typedef enum BNTypeClass
 	{
 		VoidTypeClass = 0,
@@ -5134,6 +5140,8 @@ extern "C"
 	BINARYNINJACOREAPI void BNSetDisassemblyAddressMode(BNDisassemblySettings* settings, BNDisassemblyAddressMode mode);
 	BINARYNINJACOREAPI uint64_t BNGetDisassemblyAddressBaseOffset(BNDisassemblySettings* settings);
 	BINARYNINJACOREAPI void BNSetDisassemblyAddressBaseOffset(BNDisassemblySettings* settings, uint64_t addressBaseOffset);
+	BINARYNINJACOREAPI BNDisassemblyCallParameterHints BNGetDisassemblyCallParameterHints(BNDisassemblySettings* settings);
+	BINARYNINJACOREAPI void BNSetDisassemblyCallParameterHints(BNDisassemblySettings* settings, BNDisassemblyCallParameterHints hints);
 
 	// Flow graphs
 	BINARYNINJACOREAPI BNFlowGraph* BNCreateFlowGraph(void);

python/binaryview.py

@@ -8780,7 +8780,6 @@ def find_all_text(
 			settings.set_option(DisassemblyOption.ShowAddress, False)
 			settings.set_option(DisassemblyOption.ShowOpcode, False)
 			settings.set_option(DisassemblyOption.ShowVariableTypesWhenAssigned, True)
-			settings.set_option(DisassemblyOption.ShowCallParameterNames, True)
 			settings.set_option(DisassemblyOption.WaitForIL, True)
 		if not isinstance(settings, _function.DisassemblySettings):
 			raise TypeError("settings parameter is not DisassemblySettings type")
@@ -8870,7 +8869,6 @@ def find_all_constant(
 			settings.set_option(DisassemblyOption.ShowAddress, False)
 			settings.set_option(DisassemblyOption.ShowOpcode, False)
 			settings.set_option(DisassemblyOption.ShowVariableTypesWhenAssigned, True)
-			settings.set_option(DisassemblyOption.ShowCallParameterNames, True)
 			settings.set_option(DisassemblyOption.WaitForIL, True)
 		if not isinstance(settings, _function.DisassemblySettings):
 			raise TypeError("settings parameter is not DisassemblySettings type")

ui/disassemblyview.h

@@ -77,6 +77,7 @@ class BINARYNINJAUIAPI DisassemblyView : public FlowGraphWidget
 	void setOption(BNDisassemblyOption option, bool state = true);
 	void toggleOption(BNDisassemblyOption option);
 	void setAddressMode(std::optional<BNDisassemblyAddressMode> mode, std::optional<bool> hex, std::optional<bool> withName);
+	void setCallParamHints(BNDisassemblyCallParameterHints hints);
 	void setDisplayedFileName();
 	void setAddressBaseOffset(bool toHere);
 
@@ -134,6 +135,7 @@ class BINARYNINJAUIAPI DisassemblyView : public FlowGraphWidget
 	BNFunctionGraphType m_ilViewType;
 	std::set<BNDisassemblyOption> m_options;
 	BNDisassemblyAddressMode m_addressMode;
+	BNDisassemblyCallParameterHints m_callParamHints;
 	DisassemblyContainer* m_container;
 	SettingsRef m_settings;
 

ui/linearview.h

@@ -481,6 +481,7 @@ private Q_SLOTS:
 
 	void toggleOption(BNDisassemblyOption option);
 	void setAddressMode(std::optional<BNDisassemblyAddressMode> mode, std::optional<bool> hex, std::optional<bool> withName);
+	void setCallParamHints(BNDisassemblyCallParameterHints hints);
 	void setDisplayedFileName();
 	void setAddressBaseOffset(bool toHere);