git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 1/4] Consistently use the term "builtin" instead of "internal command"

From
Sebastian Schuberth <sschuberth@gmail.com>
Date
Jan 2, 2014, 16:15 UTC
Message-ID
<52C590B0.1020702@gmail.com>
In-Reply-To
<52C58FD7.6010608@gmail.com>
Signed-off-by: Sebastian Schuberth <sschuberth@gmail.com>
---
 Documentation/technical/api-builtin.txt |  2 +-
 git.c                                   | 14 +++++++-------
 2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/Documentation/technical/api-builtin.txt b/Documentation/technical/api-builtin.txt
index f3c1357..150a02a 100644
--- a/Documentation/technical/api-builtin.txt
+++ b/Documentation/technical/api-builtin.txt
@@ -14,7 +14,7 @@ Git:
 
 . Add the external declaration for the function to `builtin.h`.
 
-. Add the command to `commands[]` table in `handle_internal_command()`,
+. Add the command to `commands[]` table in `handle_builtin()`,
   defined in `git.c`.  The entry should look like:
 
 	{ "foo", cmd_foo, <options> },
diff --git a/git.c b/git.c
index 3799514..89ab5d7 100644
--- a/git.c
+++ b/git.c
@@ -332,7 +332,7 @@ static int run_builtin(struct cmd_struct *p, int argc, const char **argv)
 	return 0;
 }
 
-static void handle_internal_command(int argc, const char **argv)
+static void handle_builtin(int argc, const char **argv)
 {
 	const char *cmd = argv[0];
 	static struct cmd_struct commands[] = {
@@ -517,8 +517,8 @@ static int run_argv(int *argcp, const char ***argv)
 	int done_alias = 0;
 
 	while (1) {
-		/* See if it's an internal command */
-		handle_internal_command(*argcp, *argv);
+		/* See if it's a builtin */
+		handle_builtin(*argcp, *argv);
 
 		/* .. then try the external ones */
 		execv_dashed_external(*argv);
@@ -563,14 +563,14 @@ int main(int argc, char **av)
 	 *  - cannot execute it externally (since it would just do
 	 *    the same thing over again)
 	 *
-	 * So we just directly call the internal command handler, and
-	 * die if that one cannot handle it.
+	 * So we just directly call the builtin handler, and die if
+	 * that one cannot handle it.
 	 */
 	if (starts_with(cmd, "git-")) {
 		cmd += 4;
 		argv[0] = cmd;
-		handle_internal_command(argc, argv);
-		die("cannot handle %s internally", cmd);
+		handle_builtin(argc, argv);
+		die("cannot handle %s as a builtin", cmd);
 	}
 
 	/* Look for flags.. */
-- 
1.8.3-mingw-1
Previous: Sebastian SchuberthNext: Jonathan Nieder
Message 2 of 16 in “[PATCH v2 0/4]”
  1. 0/4 Sebastian Schuberth, Jan 2, 2014
  2. 1/4 Consistently use the term "builtin" instead of "internal command"Sebastian Schuberth, Jan 2, 2014
  3. Jonathan NiederJan 2, 2014
  4. Sebastian SchuberthJan 2, 2014
  5. Sebastian SchuberthJan 22, 2014
  6. 2/4 Call load_command_list() only when it is neededSebastian Schuberth, Jan 2, 2014
  7. 3/4 Speed up is_git_command() by checking early for internal commandsSebastian Schuberth, Jan 2, 2014
  8. Junio C HamanoJan 2, 2014
  9. Jeff KingJan 3, 2014
  10. Junio C HamanoJan 3, 2014
  11. Kent R. SpillnerJan 3, 2014
  12. Sebastian SchuberthJan 5, 2014
  13. Sebastian SchuberthJan 22, 2014
  14. 4/4 Move builtin-related implementations to a new builtin.c fileSebastian Schuberth, Jan 2, 2014
  15. Junio C HamanoJan 2, 2014
  16. Sebastian SchuberthJan 2, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.