[libcamera-devel] [PATCH] qcam: main_window: Fix include ordering

Utkarsh Tiwari utkarsh02t at gmail.com
Tue Jul 5 18:46:40 CEST 2022


On Tue, Jul 05, 2022 at 04:32:48PM +0100, Kieran Bingham via libcamera-devel wrote:
> Fix the sort order of the QT headers to match the expected
> sort from clang-format.
> 
> Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> ---
>  src/qcam/main_window.h | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/src/qcam/main_window.h b/src/qcam/main_window.h
> index 3fbe872c0b5b..fc70920f990c 100644
> --- a/src/qcam/main_window.h
> +++ b/src/qcam/main_window.h
> @@ -10,14 +10,6 @@
>  #include <memory>
>  #include <vector>
>  
> -#include <QElapsedTimer>
> -#include <QIcon>
> -#include <QMainWindow>
> -#include <QMutex>
> -#include <QObject>
> -#include <QQueue>
> -#include <QTimer>
> -
>  #include <libcamera/camera.h>
>  #include <libcamera/camera_manager.h>
>  #include <libcamera/controls.h>
> @@ -26,7 +18,16 @@
>  #include <libcamera/request.h>
>  #include <libcamera/stream.h>
>  
> +#include <QElapsedTimer>
> +#include <QIcon>
> +#include <QMainWindow>
> +#include <QMutex>
> +#include <QObject>
> +#include <QQueue>
> +#include <QTimer>
> +
>  #include "../cam/stream_options.h"
> +
>  #include "viewfinder.h"
>  
>  class QAction;
> -- 
> 2.34.1
> 

This can also include the fix to ordering in main_window.cpp

--- a/src/qcam/main_window.cpp
+++ b/src/qcam/main_window.cpp
@@ -30,6 +30,7 @@
 #include <QtDebug>
 
 #include "../cam/image.h"
+
 #include "dng_writer.h"
 #ifndef QT_NO_OPENGL
 #include "viewfinder_gl.h"



More information about the libcamera-devel mailing list