[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