From 031a748b2a9bf6dcfd3983d8f0f616718fcddd88 Mon Sep 17 00:00:00 2001 From: Mark Salyzyn Date: Thu, 27 Feb 2014 16:56:15 -0800 Subject: [PATCH] libsparse: deal with build warning messages - mostly unused parameters, added defs.h to hold __unused macro - read_all prototype not included, added output_file.h to acquire Change-Id: Ib7ee00614acab53a6bd6a64f7e26a8c13dbe6297 --- libsparse/defs.h | 23 +++++++++++++++++++++++ libsparse/output_file.c | 9 +++++---- libsparse/sparse.c | 3 ++- libsparse/sparse_read.c | 5 ++++- 4 files changed, 34 insertions(+), 6 deletions(-) create mode 100644 libsparse/defs.h diff --git a/libsparse/defs.h b/libsparse/defs.h new file mode 100644 index 00000000..34e63c57 --- /dev/null +++ b/libsparse/defs.h @@ -0,0 +1,23 @@ +/* + * Copyright (C) 2014 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#ifndef _LIBSPARSE_DEFS_H_ + +#ifndef __unused +#define __unused __attribute__((__unused__)) +#endif + +#endif /* _LIBSPARSE_DEFS_H_ */ diff --git a/libsparse/output_file.c b/libsparse/output_file.c index e63c4a92..1cf8d8dc 100644 --- a/libsparse/output_file.c +++ b/libsparse/output_file.c @@ -29,6 +29,7 @@ #include #include +#include "defs.h" #include "output_file.h" #include "sparse_format.h" #include "sparse_crc32.h" @@ -264,7 +265,7 @@ static struct output_file_ops gz_file_ops = { .close = gz_file_close, }; -static int callback_file_open(struct output_file *out, int fd) +static int callback_file_open(struct output_file *out __unused, int fd __unused) { return 0; } @@ -287,7 +288,7 @@ static int callback_file_skip(struct output_file *out, int64_t off) return 0; } -static int callback_file_pad(struct output_file *out, int64_t len) +static int callback_file_pad(struct output_file *out __unused, int64_t len __unused) { return -1; } @@ -631,8 +632,8 @@ static struct output_file *output_file_new_normal(void) } struct output_file *output_file_open_callback(int (*write)(void *, const void *, int), - void *priv, unsigned int block_size, int64_t len, int gz, int sparse, - int chunks, int crc) + void *priv, unsigned int block_size, int64_t len, + int gz __unused, int sparse, int chunks, int crc) { int ret; struct output_file_callback *outc; diff --git a/libsparse/sparse.c b/libsparse/sparse.c index 741e8c6f..0f107b00 100644 --- a/libsparse/sparse.c +++ b/libsparse/sparse.c @@ -19,6 +19,7 @@ #include +#include "defs.h" #include "sparse_file.h" #include "output_file.h" @@ -189,7 +190,7 @@ int sparse_file_callback(struct sparse_file *s, bool sparse, bool crc, return ret; } -static int out_counter_write(void *priv, const void *data, int len) +static int out_counter_write(void *priv, const void *data __unused, int len) { int64_t *count = priv; *count += len; diff --git a/libsparse/sparse_read.c b/libsparse/sparse_read.c index 704bcfad..873c87c7 100644 --- a/libsparse/sparse_read.c +++ b/libsparse/sparse_read.c @@ -29,6 +29,8 @@ #include +#include "defs.h" +#include "output_file.h" #include "sparse_crc32.h" #include "sparse_file.h" #include "sparse_format.h" @@ -175,7 +177,8 @@ static int process_fill_chunk(struct sparse_file *s, unsigned int chunk_size, } static int process_skip_chunk(struct sparse_file *s, unsigned int chunk_size, - int fd, unsigned int blocks, unsigned int block, uint32_t *crc32) + int fd __unused, unsigned int blocks, + unsigned int block __unused, uint32_t *crc32) { int ret; int chunk;