summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRussell Belfer <rb@github.com>2013-10-02 14:13:11 -0700
committerRussell Belfer <rb@github.com>2013-10-03 10:45:04 -0700
commitaf302acaee95ff2ca3379b93cd886ebe96adff10 (patch)
tree83be2a9f600c43a5b6c1aba17239bab56e83d934 /src
parent618b7689e1cdd4ebd956949a95038fd49592a187 (diff)
downloadlibgit2-af302acaee95ff2ca3379b93cd886ebe96adff10.tar.gz
Clean up annoying warnings
The indexer code was generating warnings on Windows 64-bit. I looked closely at the logic and was able to simplify it a bit. Also this fixes some other Windows and Linux warnings.
Diffstat (limited to 'src')
-rw-r--r--src/indexer.c10
-rw-r--r--src/repository.c6
2 files changed, 10 insertions, 6 deletions
diff --git a/src/indexer.c b/src/indexer.c
index 4ce69fc8d..1270488f0 100644
--- a/src/indexer.c
+++ b/src/indexer.c
@@ -50,7 +50,7 @@ struct git_indexer_stream {
/* Fields for calculating the packfile trailer (hash of everything before it) */
char inbuf[GIT_OID_RAWSZ];
- int inbuf_len;
+ size_t inbuf_len;
git_hash_ctx trailer;
};
@@ -378,13 +378,13 @@ static int do_progress_callback(git_indexer_stream *idx, git_transfer_progress *
/* Hash everything but the last 20B of input */
static void hash_partially(git_indexer_stream *idx, const uint8_t *data, size_t size)
{
- int to_expell, to_keep;
+ size_t to_expell, to_keep;
if (size == 0)
return;
/* Easy case, dump the buffer and the data minus the last 20 bytes */
- if (size >= 20) {
+ if (size >= GIT_OID_RAWSZ) {
git_hash_update(&idx->trailer, idx->inbuf, idx->inbuf_len);
git_hash_update(&idx->trailer, data, size - GIT_OID_RAWSZ);
@@ -402,8 +402,8 @@ static void hash_partially(git_indexer_stream *idx, const uint8_t *data, size_t
}
/* We need to partially drain the buffer and then append */
- to_expell = abs(size - (GIT_OID_RAWSZ - idx->inbuf_len));
- to_keep = abs(idx->inbuf_len - to_expell);
+ to_keep = GIT_OID_RAWSZ - size;
+ to_expell = idx->inbuf_len - to_keep;
git_hash_update(&idx->trailer, idx->inbuf, to_expell);
diff --git a/src/repository.c b/src/repository.c
index 64f13978d..51cc76db4 100644
--- a/src/repository.c
+++ b/src/repository.c
@@ -890,6 +890,8 @@ static bool are_symlinks_supported(const char *wd_path)
return symlinks_supported;
}
+#ifdef GIT_USE_ICONV
+
static const char *nfc_file = "\xC3\x85\x73\x74\x72\xC3\xB6\x6D.XXXXXX";
static const char *nfd_file = "\x41\xCC\x8A\x73\x74\x72\x6F\xCC\x88\x6D.XXXXXX";
@@ -938,6 +940,8 @@ fail:
return need_precompose;
}
+#endif
+
static int create_empty_file(const char *path, mode_t mode)
{
int fd;
@@ -994,7 +998,7 @@ static int repo_init_config(
SET_REPO_CONFIG(
bool, "core.filemode", is_chmod_supported(git_buf_cstr(&cfg_path)));
-#if __APPLE__
+#ifdef GIT_USE_ICONV
SET_REPO_CONFIG(
bool, "core.precomposeunicode",
should_precompose_unicode_paths(is_bare ? repo_dir : work_dir));