| From 7982e55ce3a8b3c60a47258ff7d37d0dd78c303d Mon Sep 17 00:00:00 2001 |
| From: Khem Raj <raj.khem@gmail.com> |
| Date: Tue, 20 Feb 2024 18:44:23 -0800 |
| Subject: [PATCH] rpi: Use alloca instead of variable length arrays |
| |
| Clang-18+ diagnoses this as error |
| |
| | ../git/src/ipa/rpi/controller/rpi/alsc.cpp:499:10: error: variable length arrays in C++ are a Clang extension [-Werror,-Wvla-cxx-extension] | 499 | int xLo[X], xHi[X]; |
| | | ^ |
| |
| Upstream-Status: Submitted [https://lists.libcamera.org/pipermail/libcamera-devel/2024-February/040529.html] |
| Signed-off-by: Khem Raj <raj.khem@gmail.com> |
| --- |
| src/ipa/rpi/controller/rpi/alsc.cpp | 4 ++-- |
| 1 file changed, 2 insertions(+), 2 deletions(-) |
| |
| diff --git a/src/ipa/rpi/controller/rpi/alsc.cpp b/src/ipa/rpi/controller/rpi/alsc.cpp |
| index 8a205c60..8c0ae8eb 100644 |
| --- a/src/ipa/rpi/controller/rpi/alsc.cpp |
| +++ b/src/ipa/rpi/controller/rpi/alsc.cpp |
| @@ -496,8 +496,8 @@ void resampleCalTable(const Array2D<double> &calTableIn, |
| * Precalculate and cache the x sampling locations and phases to save |
| * recomputing them on every row. |
| */ |
| - int xLo[X], xHi[X]; |
| - double xf[X]; |
| + int *xLo = (int*)alloca(X), *xHi = (int*)alloca(X); |
| + double *xf = (double*)alloca(X); |
| double scaleX = cameraMode.sensorWidth / |
| (cameraMode.width * cameraMode.scaleX); |
| double xOff = cameraMode.cropX / (double)cameraMode.sensorWidth; |
| -- |
| 2.43.2 |
| |