Hi Chan-yeol,
> ---
> src/log.c | 12 ++++++++++++
> src/log.h | 1 +
> 2 files changed, 13 insertions(+), 0 deletions(-)
>
> diff --git a/src/log.c b/src/log.c
> index 4a50117..adbfba9 100644
> --- a/src/log.c
> +++ b/src/log.c
> @@ -55,6 +55,18 @@ void error(const char *format, ...)
> va_end(ap);
> }
>
> +void warn(const char *format, ...)
> +{
> + va_list ap;
> +
> + va_start(ap, format);
> +
> + vsyslog(LOG_WARNING, format, ap);
> +
> + va_end(ap);
> +}
> +
> +
the patch is fine, but just for consistence sake, you should put warn()
after info() function.
And no double empty lines between functions please. That is not part of
our coding style. And if you saw it somewhere else in the code, then
that is an oversight.
> void btd_debug(const char *format, ...)
> {
> va_list ap;
> diff --git a/src/log.h b/src/log.h
> index 51c52e6..cdcb146 100644
> --- a/src/log.h
> +++ b/src/log.h
> @@ -23,6 +23,7 @@
>
> void info(const char *format, ...) __attribute__((format(printf, 1, 2)));
> void error(const char *format, ...) __attribute__((format(printf, 1, 2)));
> +void warn(const char *format, ...) __attribute__((format(printf, 1, 2)));
Same here. warn() comes after info().
>
> void btd_debug(const char *format, ...) __attribute__((format(printf, 1, 2)));
>
Regards
Marcel
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
