diff options
| author | Junio C Hamano <gitster@pobox.com> | 2012-12-07 14:10:56 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2012-12-07 14:10:56 -0800 |
| commit | fff26a68053022a2cbc39142c7c6f6d016837058 (patch) | |
| tree | 1ad5a3cd1a135213b3694f11fdcb52bcddec51b9 /sequencer.c | |
| parent | 6a402843c2bc0a72fb70c8795aee523d7f1582d3 (diff) | |
| parent | 0e18bcd5e937b9e5857892e7f5c80c3c5b1da5d4 (diff) | |
| download | git-fff26a68053022a2cbc39142c7c6f6d016837058.tar.gz git-fff26a68053022a2cbc39142c7c6f6d016837058.tar.xz | |
Merge branch 'jc/same-encoding' into maint
Various codepaths checked if two encoding names are the same using
ad-hoc code and some of them ended up asking iconv() to convert
between "utf8" and "UTF-8". The former is not a valid way to spell
the encoding name, but often people use it by mistake, and we
equated them in some but not all codepaths. Introduce a new helper
function to make these codepaths consistent.
* jc/same-encoding:
reencode_string(): introduce and use same_encoding()
Diffstat (limited to 'sequencer.c')
| -rw-r--r-- | sequencer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sequencer.c b/sequencer.c index e3723d209..73c396bd8 100644 --- a/sequencer.c +++ b/sequencer.c @@ -60,7 +60,7 @@ static int get_message(struct commit *commit, struct commit_message *out) out->reencoded_message = NULL; out->message = commit->buffer; - if (strcmp(encoding, git_commit_encoding)) + if (same_encoding(encoding, git_commit_encoding)) out->reencoded_message = reencode_string(commit->buffer, git_commit_encoding, encoding); if (out->reencoded_message) |