From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bo Yang Subject: [PATCH v4 05/18] Parse the -L options Date: Fri, 6 Aug 2010 00:11:44 +0800 Message-ID: <1281024717-7855-6-git-send-email-struggleyb.nku@gmail.com> References: <1281024717-7855-1-git-send-email-struggleyb.nku@gmail.com> Cc: trast@student.ethz.ch, Jens.Lehmann@web.de To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Thu Aug 05 18:13:09 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Oh347-0000oB-LD for gcvg-git-2@lo.gmane.org; Thu, 05 Aug 2010 18:13:07 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933448Ab0HEQMz (ORCPT ); Thu, 5 Aug 2010 12:12:55 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:57614 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759244Ab0HEQMv (ORCPT ); Thu, 5 Aug 2010 12:12:51 -0400 Received: by mail-pz0-f46.google.com with SMTP id 26so2533376pzk.19 for ; Thu, 05 Aug 2010 09:12:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:from:to:cc:subject:date :message-id:x-mailer:in-reply-to:references; bh=kRpMpoNyQSCGgU+ZRcAdunZeNdpPbA3iS41enmc/yXc=; b=EoOTSkYX/FrYdo+bqhperv+L3LvNOBFX1s9Ti/E8vMpb2D3LM2tL/TV2ON7WGt5GRx EapTmD4pleSXNspnXxtotGPk2gBl2jV/u+sdreU7t7xZWrtJ0EI0PxuB2+5nfU7NL/9H 5qtHYP54DSQZ7Jxe2RgYhZUIPjv0k7OX7lVdc= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; b=SvyU02rycqr3Ib0MRBRzanwSctKphZDIlXNUKVH678QSUI6gGiy+Hndqz+0Rug3zfg wH2Du3zuz5NlCxIb0kPux5wezwddDpGE4bbJEkVXQM/NBrW85CfD1bStkQSZqfxz6/d2 +Ud1LdMLHDUVxIH4udIS6veC+Ld5NcianDQbc= Received: by 10.142.156.16 with SMTP id d16mr9252295wfe.283.1281024770966; Thu, 05 Aug 2010 09:12:50 -0700 (PDT) Received: from localhost.localdomain ([222.35.120.192]) by mx.google.com with ESMTPS id w8sm343653wfd.19.2010.08.05.09.12.47 (version=TLSv1/SSLv3 cipher=RC4-MD5); Thu, 05 Aug 2010 09:12:50 -0700 (PDT) X-Mailer: git-send-email 1.7.2.20.g388bbb In-Reply-To: <1281024717-7855-1-git-send-email-struggleyb.nku@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: With the two new APIs of parse options added in the previous commit, we parse the multiple '-L n,m ' syntax. Notice that users can give more than one '-L n,m' for each pathspec. And a pathspec with all its '-L' options maps to a single diff_line_range structure. This has the exactly the same semantics as 'git blame -L n,m ' because we refactored and reused the blame code. Signed-off-by: Bo Yang --- builtin/log.c | 103 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 102 insertions(+), 1 deletions(-) diff --git a/builtin/log.c b/builtin/log.c index 08b8722..84d781e 100644 --- a/builtin/log.c +++ b/builtin/log.c @@ -19,6 +19,7 @@ #include "remote.h" #include "string-list.h" #include "parse-options.h" +#include "line.h" /* Set a default date-time format for git log ("log.date" config variable) */ static const char *default_date_mode = NULL; @@ -27,11 +28,24 @@ static int default_show_root = 1; static int decoration_style; static const char *fmt_patch_subject_prefix = "PATCH"; static const char *fmt_pretty; +static const char *dashdash = "--"; -static const char * const builtin_log_usage = +static char builtin_log_usage[] = "git log [] [..] [[--] ...]\n" + "git log [] -L n,m \n" " or: git show [options] ..."; +static const char *log_opt_usage[] = { + builtin_log_usage, + NULL +}; + +struct line_opt_callback_data { + struct diff_line_range **range; + struct parse_opt_ctx_t *ctx; + struct rev_info *rev; +}; + static int parse_decoration_style(const char *var, const char *value) { switch (git_config_maybe_bool(var, value)) { @@ -49,12 +63,42 @@ static int parse_decoration_style(const char *var, const char *value) return -1; } +static int log_line_range_callback(const struct option *option, const char *arg, int unset) +{ + struct line_opt_callback_data *data = option->value; + struct diff_line_range *r = *data->range; + struct parse_opt_ctx_t *ctx = data->ctx; + + if (!arg) + return -1; + + if (r->nr == 0 && r->next == NULL) { + ctx->out[ctx->cpidx++] = dashdash; + } + + diff_line_range_append(r, arg); + data->rev->line_level_traverse = 1; + return 0; +} + static void cmd_log_init(int argc, const char **argv, const char *prefix, struct rev_info *rev, struct setup_revision_opt *opt) { int i; int decoration_given = 0; struct userformat_want w; + const char *path = NULL, *pathspec = NULL; + static struct diff_line_range *range = NULL, *r = NULL; + static struct parse_opt_ctx_t ctx; + static struct line_opt_callback_data line_cb = {&range, &ctx, NULL}; + static const struct option options[] = { + OPT_CALLBACK('L', NULL, &line_cb, "n,m", "Process only line range n,m, counting from 1", log_line_range_callback), + OPT_END() + }; + + line_cb.rev = rev; + range = xmalloc(sizeof(*range)); + DIFF_LINE_RANGE_INIT(range); rev->abbrev = DEFAULT_ABBREV; rev->commit_format = CMIT_FMT_DEFAULT; @@ -75,6 +119,58 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix, */ if (argc == 2 && !strcmp(argv[1], "-h")) usage(builtin_log_usage); + + parse_options_start(&ctx, argc, argv, prefix, PARSE_OPT_KEEP_DASHDASH | + PARSE_OPT_KEEP_ARGV0 | PARSE_OPT_STOP_AT_NON_OPTION); + for (;;) { + switch (parse_options_step(&ctx, options, log_opt_usage)) { + case PARSE_OPT_HELP: + exit(129); + case PARSE_OPT_DONE: + goto parse_done; + case PARSE_OPT_NON_OPTION: + path = parse_options_current(&ctx); + pathspec = prefix_path(prefix, prefix ? strlen(prefix) : 0, path); + range->spec = alloc_filespec(pathspec); + free((void *)pathspec); + if (range->nr == 0) { + if(range->next) { + die("Path %s need a -L option\n" + "If you want follow the history of the whole file " + "whether to using 'git log' without -L or using " + "'git log -L 1,$ '", range->spec->path); + } else { + parse_options_next(&ctx, 1); + continue; + } + } + r = xmalloc(sizeof(*r)); + DIFF_LINE_RANGE_INIT(r); + r->next = range; + range = r; + parse_options_next(&ctx, 1); + continue; + case PARSE_OPT_UNKNOWN: + parse_options_next(&ctx, 1); + continue; + } + + parse_revision_opt(rev, &ctx, options, log_opt_usage); + } +parse_done: + argc = parse_options_end(&ctx); + + /* die if '-L ' with no pathspec follow */ + if (range->nr > 0 && range->spec == NULL) { + die("Each -L should follow a pathspec"); + } + /* clear up the last range */ + if (range->nr == 0) { + struct diff_line_range *r = range->next; + DIFF_LINE_RANGE_CLEAR(range); + range = r; + } + argc = setup_revisions(argc, argv, rev, opt); memset(&w, 0, sizeof(w)); @@ -125,6 +221,11 @@ static void cmd_log_init(int argc, const char **argv, const char *prefix, rev->show_decorations = 1; load_ref_decorations(decoration_style); } + + /* Test whether line level history is asked for */ + if (range && range->nr > 0) { + setup_line(rev, range); + } } /* -- 1.7.2.20.g388bbb