scalar: mark unused parameters in dummy function

peff · gitster · commit 5e75e503c4f7 · 2024-08-17T09:46:24.000-07:00
We have a dummy load_builtin_commands() function to satisfy the linker,
but which we never expect to be called. Mark its parameters to avoid
complaints from -Wunused-parameter.

Signed-off-by: Jeff King &lt;peff@peff.net&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/scalar.c b/scalar.c
@@ -400,7 +400,8 @@ static int delete_enlistment(struct strbuf *enlistment)
  * Dummy implementation; Using `get_version_info()` would cause a link error
  * without this.
  */
-void load_builtin_commands(const char *prefix, struct cmdnames *cmds)
+void load_builtin_commands(const char *prefix UNUSED,
+			   struct cmdnames *cmds UNUSED)
 {
 	die("not implemented");
 }

Original file line number	Diff line number	Diff line change
`@@ -400,7 +400,8 @@ static int delete_enlistment(struct strbuf *enlistment)`
`400`	`400`	* Dummy implementation; Using `get_version_info()` would cause a link error
`401`	`401`	`* without this.`
`402`	`402`	`*/`
`403`		`-void load_builtin_commands(const char prefix, struct cmdnames cmds)`
	`403`	`+void load_builtin_commands(const char *prefix UNUSED,`
	`404`	`+ struct cmdnames *cmds UNUSED)`
`404`	`405`	`{`
`405`	`406`	`die("not implemented");`
`406`	`407`	`}`