bloom: fix make sparse
warning
* We need a `final_new_line` to make our source code as text file, per POSIX and C specification. * `bloom_filters` should be limited to interal linkage only Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com> Signed-off-by: Ramsay Jones <ramsay@ramsayjones.plus.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
07d8ea56f2
commit
066b70ae97
4
bloom.c
4
bloom.c
@ -9,7 +9,7 @@
|
|||||||
|
|
||||||
define_commit_slab(bloom_filter_slab, struct bloom_filter);
|
define_commit_slab(bloom_filter_slab, struct bloom_filter);
|
||||||
|
|
||||||
struct bloom_filter_slab bloom_filters;
|
static struct bloom_filter_slab bloom_filters;
|
||||||
|
|
||||||
struct pathmap_hash_entry {
|
struct pathmap_hash_entry {
|
||||||
struct hashmap_entry entry;
|
struct hashmap_entry entry;
|
||||||
@ -273,4 +273,4 @@ int bloom_filter_contains(const struct bloom_filter *filter,
|
|||||||
}
|
}
|
||||||
|
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
2
bloom.h
2
bloom.h
@ -87,4 +87,4 @@ int bloom_filter_contains(const struct bloom_filter *filter,
|
|||||||
const struct bloom_key *key,
|
const struct bloom_key *key,
|
||||||
const struct bloom_filter_settings *settings);
|
const struct bloom_filter_settings *settings);
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
@ -3,7 +3,7 @@
|
|||||||
#include "test-tool.h"
|
#include "test-tool.h"
|
||||||
#include "commit.h"
|
#include "commit.h"
|
||||||
|
|
||||||
struct bloom_filter_settings settings = DEFAULT_BLOOM_FILTER_SETTINGS;
|
static struct bloom_filter_settings settings = DEFAULT_BLOOM_FILTER_SETTINGS;
|
||||||
|
|
||||||
static void add_string_to_filter(const char *data, struct bloom_filter *filter) {
|
static void add_string_to_filter(const char *data, struct bloom_filter *filter) {
|
||||||
struct bloom_key key;
|
struct bloom_key key;
|
||||||
|
@ -114,4 +114,4 @@ test_expect_success EXPENSIVE 'get bloom filter for commit with 513 changes' '
|
|||||||
test_cmp expect actual
|
test_cmp expect actual
|
||||||
'
|
'
|
||||||
|
|
||||||
test_done
|
test_done
|
||||||
|
@ -152,4 +152,4 @@ test_expect_success 'Use Bloom filters if they exist in the latest but not all c
|
|||||||
test_bloom_filters_used_when_some_filters_are_missing "-- A/B"
|
test_bloom_filters_used_when_some_filters_are_missing "-- A/B"
|
||||||
'
|
'
|
||||||
|
|
||||||
test_done
|
test_done
|
||||||
|
Reference in New Issue
Block a user