[libcamera-devel,1/5] meson: Switch to C++17

Message ID 20200822200037.20892-2-laurent.pinchart@ideasonboard.com
State Accepted
Headers show
Series
  • libcamera: Switch to C++17
Related show

Commit Message

Laurent Pinchart Aug. 22, 2020, 8 p.m. UTC
Due to popular request, move from C++14 to C++17. This will allow
dropping some custom constructs (such as a custom utils::clamp),
benefiting from new language features, and dropping gcc 5 and 6 from the
compilation tests.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kieran Bingham Aug. 24, 2020, 9:43 a.m. UTC | #1
Hi Laurent,

On 22/08/2020 21:00, Laurent Pinchart wrote:
> Due to popular request, move from C++14 to C++17. This will allow
> dropping some custom constructs (such as a custom utils::clamp),
> benefiting from new language features, and dropping gcc 5 and 6 from the
> compilation tests.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Yay!

Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

> ---
>  meson.build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meson.build b/meson.build
> index ec54e68f3635..ce1d1c63ddc6 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -6,7 +6,7 @@ project('libcamera', 'c', 'cpp',
>      default_options : [
>          'werror=true',
>          'warning_level=2',
> -        'cpp_std=c++14',
> +        'cpp_std=c++17',
>      ],
>      license : 'LGPL 2.1+')
>

Patch

diff --git a/meson.build b/meson.build
index ec54e68f3635..ce1d1c63ddc6 100644
--- a/meson.build
+++ b/meson.build
@@ -6,7 +6,7 @@  project('libcamera', 'c', 'cpp',
     default_options : [
         'werror=true',
         'warning_level=2',
-        'cpp_std=c++14',
+        'cpp_std=c++17',
     ],
     license : 'LGPL 2.1+')