patches/pretty/format-aliases/v4
tag262f5290be855f3745bb3e5904551ed8482a4175
object 122a1b81d7b54fdb7d134a2ee43fe4131b219e46
authorWill Palmer <wmpalmer@gmail.com>
Sun, 2 May 2010 10:40:37 +0000 (2 11:40 +0100)
From 122a1b81d7b54fdb7d134a2ee43fe4131b219e46 Mon Sep 17 00:00:00 2001
From: Will Palmer <wmpalmer@gmail.com>
Date: Sun, 2 May 2010 11:40:37 +0100
Subject: [PATCH v4 0/3] pretty: format aliases

The following patch series adds the ability to configure aliases for
user-defined formats. The first two patches add infrastructure for the
ease of adding additional formats, and infrastructure for defining
format aliases, respectively. The final patch adds support for defining
format aliases via a config option.

This is the fourth version of the patch series. The most notable change
from the last iteration is the removal of changes to other pretty-format
options, such as the "conditional colors" patch and "%H/%h --abbrev"
patches. I still think those were good patches, but the %H change in
particular was controversial enough that it would have needlessly stood
in the way of the rest of the series, and '%?C' looked out-of-place in
this series without any other format changes. The removed patches will
be worked on independently and will be submitted separately later.

Jeff King noted an artifact of format.pretty.<name> left in the
documentation, thanks.

Junio asked for a better error message when looped-aliases were found,
so that's been put in.

The copyright notice for the added test has been changed to the standard
boilerplate, because there's no reason to bring politics into a patch.

Most other changes since the last iteration were simple style fixes,
mostly a result of muscle memory from the coding conventions I use at
work, though a couple of things (like not initializing static variables)
were more towards the "you can do that? But my CS teacher in highschool
told me not to!" end of the spectrum. Hopefully I've caught all the
style problems this time.

Will Palmer (3):
  pretty: make it easier to add new formats
  pretty: add infrastructure to allow format aliases
  pretty: add aliases for pretty formats

 Documentation/config.txt         |    9 ++
 Documentation/pretty-formats.txt |    7 ++-
 pretty.c                         |  154 ++++++++++++++++++++++++++++++++------
 t/t4205-log-pretty-formats.sh    |   66 ++++++++++++++++
 4 files changed, 212 insertions(+), 24 deletions(-)
 create mode 100755 t/t4205-log-pretty-formats.sh