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

Re: [PATCH] branch: reject -D/-d without branch name

From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date
Jan 25, 2013, 08:45 UTC
Message-ID
<vpqmwvxhcra.fsf@grenoble-inp.fr>
In-Reply-To
<1359102416-1240-1-git-send-email-pclouds@gmail.com>
Nguyễn Thái Ngọc Duy <pclouds@gmail.com> writes:
Show 13 quoted lines
> diff --git a/builtin/branch.c b/builtin/branch.c
> index 873f624..1d3e842 100644
> --- a/builtin/branch.c
> +++ b/builtin/branch.c
> @@ -837,7 +837,7 @@ int cmd_branch(int argc, const char **argv, const char *prefix)
>  		colopts = 0;
>  	}
>  
> -	if (delete)
> +	if (delete && argc)
>  		return delete_branches(argc, argv, delete > 1, kinds, quiet);
>  	else if (list) {
>  		int ret = print_ref_list(kinds, detached, verbose, abbrev,

Shouldn't this error out with a clean error message ("branch name expected" or so)?

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Previous: Nguyễn Thái Ngọc DuyNext: Duy Nguyen
Message 2 of 14 in “branch: reject -D/-d without branch name”
  1. branch: reject -D/-d without branch nameNguyễn Thái Ngọc Duy, Jan 25, 2013
  2. Matthieu MoyJan 25, 2013
  3. Duy NguyenJan 25, 2013
  4. 1/3 branch: reject -D/-d without branch nameNguyễn Thái Ngọc Duy, Jan 25, 2013
  5. 2/3 branch: give a more helpful message on redundant argumentsNguyễn Thái Ngọc Duy, Jan 25, 2013
  6. Jonathan NiederJan 27, 2013
  7. 3/3 branch: mark more strings for translationNguyễn Thái Ngọc Duy, Jan 25, 2013
  8. Jonathan NiederJan 27, 2013
  9. Junio C HamanoJan 25, 2013
  10. Duy NguyenJan 26, 2013
  11. 1/4 branch: no detached HEAD check when editing another branch's descriptionNguyễn Thái Ngọc Duy, Jan 28, 2013
  12. 2/4 branch: reject -D/-d without branch nameNguyễn Thái Ngọc Duy, Jan 28, 2013
  13. 3/4 branch: give a more helpful message on redundant argumentsNguyễn Thái Ngọc Duy, Jan 28, 2013
  14. 4/4 branch: mark more strings for translationNguyễn Thái Ngọc Duy, Jan 28, 2013

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.