diff options
| author |  Jakub Narebski <jnareb@gmail.com> | 2008-12-09 23:48:51 +0100 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2008-12-10 00:21:05 -0800 |
| commit | d2ce10d7b7d67ff8b50ae749ce4c5b1a2b8d133c (patch) | |
| tree | 4d49149e53dfcee38172618a894f5af1f0ec4ae1 /builtin-grep.c | |
| parent | gitweb: Move 'lineno' id from link to row element in git_blame (diff) | |
| download | tgif-d2ce10d7b7d67ff8b50ae749ce4c5b1a2b8d133c.tar.xz | |
gitweb: A bit of code cleanup in git_blame()
Among others, here are the highlights:
* move variable declaration closer to the place it is set and used,
if possible,
* uniquify and simplify coding style a bit, which includes removing
unnecessary '()'.
* check type only if $hash was defined, as otherwise from the way
git_get_hash_by_path() is called (and works), we know that it is
a blob,
* use modern calling convention for git-blame,
* remove unused variable,
* don't use implicit variables ($_),
* add some comments
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Acked-by: Luben Tuikov <ltuikov@yahoo.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-grep.c')
0 files changed, 0 insertions, 0 deletions