diff options
| author | Eric Lambert <eric_lambert@xyratex.com> | 2014-09-15 08:27:25 -0700 |
|---|---|---|
| committer | Eric Lambert <eric_lambert@xyratex.com> | 2014-09-15 08:27:25 -0700 |
| commit | 295bbbd03e64a41123f1d611b15bf670a48d0bbb (patch) | |
| tree | 57cda9fc789817c1cf8ddcf2a6b9bf7fb6583ffe | |
| parent | ca6e99d7751da91eea50bbf6584cff64fbf80b63 (diff) | |
| download | liberasurecode-295bbbd03e64a41123f1d611b15bf670a48d0bbb.tar.gz | |
A little bit of code clean up based on feedback/reviews
| -rw-r--r-- | src/backends/jerasure/jerasure_rs_cauchy.c | 18 | ||||
| -rw-r--r-- | test/libec_slap.c | 2 |
2 files changed, 11 insertions, 9 deletions
diff --git a/src/backends/jerasure/jerasure_rs_cauchy.c b/src/backends/jerasure/jerasure_rs_cauchy.c index 042166a..1c8b528 100644 --- a/src/backends/jerasure/jerasure_rs_cauchy.c +++ b/src/backends/jerasure/jerasure_rs_cauchy.c @@ -39,14 +39,17 @@ struct ec_backend jerasure_rs_cauchy; typedef int* (*cauchy_original_coding_matrix_func)(int, int, int); typedef int* (*jerasure_matrix_to_bitmatrix_func)(int, int, int, int *); -typedef int** (*jerasure_smart_bitmatrix_to_schedule_func)(int, int, int, int *); -typedef void (*jerasure_bitmatrix_encode_func)(int, int, int, int *, char **, char **, int, int); -typedef int (*jerasure_bitmatrix_decode_func)(int, int, int, int *, int, int *,char **, char **, int, int); +typedef int** (*jerasure_smart_bitmatrix_to_schedule_func) + (int, int, int, int *); +typedef void (*jerasure_bitmatrix_encode_func) + (int, int, int, int *, char **, char **, int, int); +typedef int (*jerasure_bitmatrix_decode_func) + (int, int, int, int *, int, int *,char **, char **, int, int); typedef int * (*jerasure_erasures_to_erased_func)(int, int, int *); -typedef int (*jerasure_make_decoding_bitmatrix_func)(int, int, int, int *, int *, int *, int *); -typedef void (*jerasure_bitmatrix_dotprod_func)(int, int, int *, int *, int,char **, char **, int, int); - - +typedef int (*jerasure_make_decoding_bitmatrix_func) + (int, int, int, int *, int *, int *, int *); +typedef void (*jerasure_bitmatrix_dotprod_func) + (int, int, int *, int *, int,char **, char **, int, int); /* * ToDo (KMG): Should we make this a parameter, or is that @@ -103,7 +106,6 @@ static int jerasure_rs_cauchy_decode(void *desc, char **data, char **parity, struct jerasure_rs_cauchy_descriptor *jerasure_desc = (struct jerasure_rs_cauchy_descriptor*)desc; - /* FIXME - make jerasure_matrix_decode return a value */ return jerasure_desc->jerasure_bitmatrix_decode(jerasure_desc->k, jerasure_desc->m, jerasure_desc->w, diff --git a/test/libec_slap.c b/test/libec_slap.c index a0b5f2d..7054e4b 100644 --- a/test/libec_slap.c +++ b/test/libec_slap.c @@ -143,7 +143,7 @@ out: static void fill_buffer(char *buf, size_t size, int seed) { - int i; + size_t i; buf[0] = seed; for (i=1; i < size; i++) { |