[libcamera-devel] [PATCH 4/5] libcamera: log: Fix coding style warnings
Niklas Söderlund
niklas.soderlund at ragnatech.se
Thu Jan 17 23:02:01 CET 2019
Hi Kieran,
Thanks for your work.
On 2019-01-17 20:20:42 +0000, Kieran Bingham wrote:
> The checkstyle utility highlights some minor warnings against the logger
> implementation.
>
> Fix these up.
>
> Fixes: edbd2059d8a4 ("libcamera: Add initial logger")
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> ---
> src/libcamera/include/log.h | 6 +++---
> src/libcamera/log.cpp | 2 +-
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/libcamera/include/log.h b/src/libcamera/include/log.h
> index c1af3741ffee..cc3f9404a3d4 100644
> --- a/src/libcamera/include/log.h
> +++ b/src/libcamera/include/log.h
> @@ -23,11 +23,11 @@ class LogMessage
> {
> public:
> LogMessage(const char *fileName, unsigned int line,
> - LogSeverity severity);
> - LogMessage(const LogMessage&) = delete;
> + LogSeverity severity);
> + LogMessage(const LogMessage &) = delete;
> ~LogMessage();
>
> - std::ostream& stream() { return msgStream; }
> + std::ostream &stream() { return msgStream; }
>
> private:
> std::ostringstream msgStream;
> diff --git a/src/libcamera/log.cpp b/src/libcamera/log.cpp
> index 9b2ae1bac8c8..74cba383363d 100644
> --- a/src/libcamera/log.cpp
> +++ b/src/libcamera/log.cpp
> @@ -66,7 +66,7 @@ namespace libcamera {
>
> static const char *log_severity_name(LogSeverity severity)
> {
> - static const char * const names[] = {
> + static const char *const names[] = {
> " DBG",
> " INFO",
> " WARN",
> --
> 2.17.1
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
--
Regards,
Niklas Söderlund
More information about the libcamera-devel
mailing list