[libcamera-devel] [PATCH 4/5] libcamera: Drop semicolon after LOG_{DECLARE, DEFINE}_CATEGORY

[Umang Jain]

Hi Laurent,

On 10/20/20 5:33 AM, Laurent Pinchart wrote:
> No semicolon is needed after the macro. Drop it in the few locations
> that have one.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
LGTM.
Reviewed-by: Umang Jain <email at uajain.com>
> ---
>   src/libcamera/byte_stream_buffer.cpp                  | 2 +-
>   src/libcamera/camera_sensor.cpp                       | 2 +-
>   src/libcamera/file.cpp                                | 2 +-
>   src/libcamera/pipeline/raspberrypi/staggered_ctrl.cpp | 2 +-
>   src/libcamera/pipeline/simple/converter.cpp           | 2 +-
>   src/libcamera/sysfs.cpp                               | 2 +-
>   6 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/libcamera/byte_stream_buffer.cpp b/src/libcamera/byte_stream_buffer.cpp
> index eb9003927266..7fc6f6944471 100644
> --- a/src/libcamera/byte_stream_buffer.cpp
> +++ b/src/libcamera/byte_stream_buffer.cpp
> @@ -19,7 +19,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DEFINE_CATEGORY(Serialization);
> +LOG_DEFINE_CATEGORY(Serialization)
>   
>   /**
>    * \class ByteStreamBuffer
> diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp
> index 78c7ceec7c44..935de528c496 100644
> --- a/src/libcamera/camera_sensor.cpp
> +++ b/src/libcamera/camera_sensor.cpp
> @@ -28,7 +28,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DEFINE_CATEGORY(CameraSensor);
> +LOG_DEFINE_CATEGORY(CameraSensor)
>   
>   /**
>    * \struct CameraSensorInfo
> diff --git a/src/libcamera/file.cpp b/src/libcamera/file.cpp
> index 04b0cb61a1bf..3a3f5bb63ffc 100644
> --- a/src/libcamera/file.cpp
> +++ b/src/libcamera/file.cpp
> @@ -23,7 +23,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DEFINE_CATEGORY(File);
> +LOG_DEFINE_CATEGORY(File)
>   
>   /**
>    * \class File
> diff --git a/src/libcamera/pipeline/raspberrypi/staggered_ctrl.cpp b/src/libcamera/pipeline/raspberrypi/staggered_ctrl.cpp
> index 0572acc93d63..62605c0fceee 100644
> --- a/src/libcamera/pipeline/raspberrypi/staggered_ctrl.cpp
> +++ b/src/libcamera/pipeline/raspberrypi/staggered_ctrl.cpp
> @@ -17,7 +17,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DEFINE_CATEGORY(RPI_S_W);
> +LOG_DEFINE_CATEGORY(RPI_S_W)
>   
>   namespace RPi {
>   
> diff --git a/src/libcamera/pipeline/simple/converter.cpp b/src/libcamera/pipeline/simple/converter.cpp
> index 75fb297ebd58..b4ee021f6210 100644
> --- a/src/libcamera/pipeline/simple/converter.cpp
> +++ b/src/libcamera/pipeline/simple/converter.cpp
> @@ -21,7 +21,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DECLARE_CATEGORY(SimplePipeline);
> +LOG_DECLARE_CATEGORY(SimplePipeline)
>   
>   SimpleConverter::SimpleConverter(MediaDevice *media)
>   	: m2m_(nullptr)
> diff --git a/src/libcamera/sysfs.cpp b/src/libcamera/sysfs.cpp
> index 6c8e9554188d..3ebe66f8d69b 100644
> --- a/src/libcamera/sysfs.cpp
> +++ b/src/libcamera/sysfs.cpp
> @@ -22,7 +22,7 @@
>   
>   namespace libcamera {
>   
> -LOG_DEFINE_CATEGORY(SysFs);
> +LOG_DEFINE_CATEGORY(SysFs)
>   
>   namespace sysfs {
>