All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Henrik Grubbström (Grubba)" <grubba@grubba.org>
To: git@vger.kernel.org
Cc: "Henrik Grubbström  " <grubba@grubba.org>
Subject: [PATCH RFC 0/5] Patches to avoid reporting conversion changes.
Date: Fri, 16 Apr 2010 18:09:57 +0200	[thread overview]
Message-ID: <cover.1271432034.git.grubba@grubba.org> (raw)

This is the first go at having the git index keep track of the
conversion mode and corresponding normalized blob for files.

The approach is relatively straight-forward:
  * struct cache_entry has been extended with two fields,
    norm_flags and norm_sha1, to keep track of the data.
  * An accessor function ce_norm_sha1() has been added,
    which checks that the fields seem valid, and otherwise
    recalculates them.
  * run_diff_files() and ce_compare_data() compare against
    the normalized blob.

Still missing is a testsuite, and I may have missed some place
that ought to use ce_norm_sha1().

Thanks to Junio C Hamano for the suggestion.

Henrik Grubbström (Grubba) (5):
  sha1_file: Added index_blob().
  cache: Added ce_norm_sha1() and related cache_entry fields.
  cache: Added index extension "NORM".
  reachable: Made the gc aware of the ce_norm_sha1.
  cache: Use ce_norm_sha1().

 cache.h      |   29 +++++++++++++++++++++++++++++
 convert.c    |   35 +++++++++++++++++++++++++++++++++++
 diff-lib.c   |    9 ++++++---
 reachable.c  |    2 ++
 read-cache.c |   51 ++++++++++++++++++++++++++++++++++++++++++++-------
 sha1_file.c  |   19 +++++++++++++++++++
 6 files changed, 135 insertions(+), 10 deletions(-)

             reply	other threads:[~2010-04-16 16:43 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-16 16:09 Henrik Grubbström (Grubba) [this message]
2010-04-16 16:09 ` [PATCH RFC 1/5] sha1_file: Added index_blob() Henrik Grubbström (Grubba)
2010-04-16 16:09 ` [PATCH RFC 2/5] cache: Added ce_norm_sha1() and related cache_entry fields Henrik Grubbström (Grubba)
2010-04-16 16:10 ` [PATCH RFC 3/5] cache: Added index extension "NORM" Henrik Grubbström (Grubba)
2010-04-16 16:10 ` [PATCH RFC 4/5] reachable: Made the gc aware of the ce_norm_sha1 Henrik Grubbström (Grubba)
2010-04-16 16:10 ` [PATCH RFC 5/5] cache: Use ce_norm_sha1() Henrik Grubbström (Grubba)
2010-04-20  7:25   ` Junio C Hamano
2010-04-20 15:39     ` Henrik Grubbström
2010-04-20 19:12       ` Junio C Hamano
2010-04-25 11:25         ` Henrik Grubbström
2010-04-16 18:02 ` [PATCH RFC 0/5] Patches to avoid reporting conversion changes Jari Aalto
2010-04-16 18:06   ` Randal L. Schwartz
2010-04-17 19:32     ` Jari Aalto
2010-04-17 19:34       ` Randal L. Schwartz
2010-04-17 22:07         ` Sverre Rabbelier
2010-04-17 22:32           ` Jakub Narebski
2010-04-17 22:47             ` Sverre Rabbelier
2010-04-17 22:58               ` Randal L. Schwartz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=cover.1271432034.git.grubba@grubba.org \
    --to=grubba@grubba.org \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.