Cover Letter Detail
Show a cover letter.
GET /api/covers/16142/?format=api
{ "id": 16142, "url": "https://patchwork.libcamera.org/api/covers/16142/?format=api", "web_url": "https://patchwork.libcamera.org/cover/16142/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20220603220712.73673-1-Rauch.Christian@gmx.de>", "date": "2022-06-03T22:07:07", "name": "[libcamera-devel,v6,0/5] generate and use fixed-sized Span Control types", "submitter": { "id": 111, "url": "https://patchwork.libcamera.org/api/people/111/?format=api", "name": "Christian Rauch", "email": "Rauch.Christian@gmx.de" }, "mbox": "https://patchwork.libcamera.org/cover/16142/mbox/", "series": [ { "id": 3158, "url": "https://patchwork.libcamera.org/api/series/3158/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=3158", "date": "2022-06-03T22:07:07", "name": "generate and use fixed-sized Span Control types", "version": 6, "mbox": "https://patchwork.libcamera.org/series/3158/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/covers/16142/comments/", "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "X-Original-To": "parsemail@patchwork.libcamera.org", "Delivered-To": "parsemail@patchwork.libcamera.org", "Received": [ "from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 65C32BD161\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 3 Jun 2022 22:07:19 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 8D41365640;\n\tSat, 4 Jun 2022 00:07:18 +0200 (CEST)", "from mout.gmx.net (mout.gmx.net [212.227.15.15])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 1EA316040E\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tSat, 4 Jun 2022 00:07:16 +0200 (CEST)", "from localhost.localdomain ([92.18.80.244]) by mail.gmx.net\n\t(mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id\n\t1M3DO3-1nwQda2Gmz-003fXN; Sat, 04 Jun 2022 00:07:15 +0200" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1654294038;\n\tbh=ZNrab0D8EUOz7DZQiRZuLY2X8IHsPU6uC9+seTUIiUw=;\n\th=To:Date:Subject:List-Id:List-Unsubscribe:List-Archive:List-Post:\n\tList-Help:List-Subscribe:From:Reply-To:From;\n\tb=M5wgq1UivkIz1djCplpwfuPRhLWPL78BD+Fnas2mlOhYmE1m9iTaLniSJCEJz7X2N\n\t+CcD/ETQX/1vJkoI+U5x94LqDCB/zRWD+JI4U1OUmI2vsnmQbq3oyM1BPi1R5EyRwS\n\tZ/dedZk1cagPqwafW0VivB9gf9ThlVIiwuVFqZ1leO6CJLPwHXgFpkECQ5hcr2P19r\n\tRpqQ5mGsNqEA9whf9z6ttHoLisswimiZ33z5/GfW5a44pDo2bySuOGrw/LJ9pLrftZ\n\tStyg/DAkE3ugYnsv5ebCsraRvUJ7XvfPyFCGT4mKSSThy5U55S9Nky4KLLoujwYREI\n\tzNL9KXEn4etpA==", "v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net;\n\ts=badeba3b8450; t=1654294035;\n\tbh=ZNrab0D8EUOz7DZQiRZuLY2X8IHsPU6uC9+seTUIiUw=;\n\th=X-UI-Sender-Class:From:To:Cc:Subject:Date;\n\tb=lX6YN5UjM3aAOYVN3dm1LTAbxsqj/9oqRUl3fyUbr9MYs5/gcthlD4tE1tuaaRHCc\n\t+qTo8p/1T9lYNN+rjwASCMi48tcmU09IjJINCID/beardn6gnQZ96Teo3b1VyUlbkQ\n\t9nKNGNzEVzW3K/X0X4rAQXL7mJznAvdY6/FUz7Ro=" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=gmx.net header.i=@gmx.net\n\theader.b=\"lX6YN5Uj\"; dkim-atps=neutral", "X-UI-Sender-Class": "01bb95c1-4bf8-414a-932a-4f6e2808ef9c", "To": "libcamera-devel@lists.libcamera.org", "Date": "Fri, 3 Jun 2022 23:07:07 +0100", "Message-Id": "<20220603220712.73673-1-Rauch.Christian@gmx.de>", "X-Mailer": "git-send-email 2.34.1", "MIME-Version": "1.0", "X-Provags-ID": "V03:K1:h5+/dGP23ckpuFYnjproleyr8QEnnB1Wz3n8gaO8AwyO91gA3Mx\n\tVsoKeH7l5LO5Ix1mX7IGbMHYkRQSQ6BIJcW02siqnIyxmwrfcjZzuss13BBCiN5AiRxvm6P\n\tOokH64Wym7MeS3TqSlUefwlEwktVxokW76X+QJA37Y0PWi6d6SK02dujEj0ek02Rql2uLAX\n\tYTYR6NFIG9jLd+n+LejQQ==", "X-Spam-Flag": "NO", "X-UI-Out-Filterresults": "notjunk:1; V03:K0:alih0APUkes=:n8mjQ42q74Fij7kk7AvXou\n\tk9AyfIbVIoTRm20OpJnLBQJRvSO/lYqGzu06KCqfmYqXAhhVECKUqT1KgBw/HSEZ7vxmdLlan\n\tcShyx5Gf/+GaDJ8GMavCXW6DX+k6YKt/2v3B/LfLkskym+UKeLWrc6h3JryJWDPJU0hfPOoPP\n\tJG3T6ttN44M57Dird3V8gF/qTsIwFT033Dj5GAG1tZd1/ICLf09mtftxYfdWUrwQ3gNm2uV2k\n\tHH1Sx2hMLGWAaP8R84cNtLwbIX8kDcor5pRTB+itFVXIlGXqZKge5JXBEbCtEH0LEYYWpPJRy\n\tL7avAEUeOclGJAkutOe+tQEQSsnwng1unEAoV/0NwdNa9cNDP9wxlWK7A9jveIZkL+rkvD1CB\n\tD2I3ESNmLi4hdUs4QBxKWhOpZEK3FTgS5oglpy8z9Vz4LQzhm60ZgQcWzG9Y5XMBs+4kvZ8K8\n\ty9tv1x+cxicnk5rdgn2GPT2kC/tFaM4SHAbfP2CF4R22jIgluiXyJqsjWtT9JwbNIDyN8ZgDS\n\tm2g0wPSOcWRwsyTjI/eoDCVqJgofMyBJrx6xd7y1Vfv9pXHdaqigG+nvdi0Y0eK1xwiPfjQeL\n\thPtAcA6IheBr+7RqP34JqPvVQHD9omv1YwNFhqoIDBg2QHDwRGaIpXqy2E1WQhzs3vkiahG7j\n\tnbylhucWLLVkJvgrKCQ0o7lKA8rXTcC6MnfWSGOnOx/ZY3GcM5qIaWEa4DZu5V8HAQbstmC0H\n\tSfHKCUgozuSfRYGuPxonAmGt0l7cdeRT5v2O0cFmk+FVauyY5XwKVioEkSFw7Ai2P+fm7btlk\n\tETGS7nNBxiLL0OTZO+2Yjoyr6P6+kpRIv296PWEPp3Pr/esc9rW4VEv58qt6PghQKcqlhws+4\n\tr7ugi0wUuy4p1dkZXMQDvrIpwvZ7fLgQo++dEIRMjS9fwIdGCy2l/hQ9tAj/Y9g7k2AFWqaUx\n\tfrRLe2ZvAnxHnqgpyoBmIdQTtXNrkakeXEZCn0hYluh65YFjQO1CmoL+OH2LwR4G874BKLWBo\n\ta9Cabxua6KTDha2+LmYbbZV6ty8ebOhEcqjJFJsvVC11GX/SrwPUCvXRBgDyvdmMvEYKJWoL8\n\tsvR2zWl2U7eCk8h/9hSfIr/ucvyrhwjBBX58QOICCkBajoMAB7s0r1mvw==", "Subject": "[libcamera-devel] [PATCH v6 0/5] generate and use fixed-sized Span\n\tControl types", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "From": "Christian Rauch via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>", "Reply-To": "Christian Rauch <Rauch.Christian@gmx.de>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Hello,\n\nThanks for all the feedback. This revision 1) fixes the \"gen-controls.py\" script for Python versions below 3.8 (e.g. for the Chrome OS SDK) and it 2) simplifies the optional value handling. Specifically, wherever a value was explicitly checked with \"contains\", I am now dereferencing the optional value directly (*prop.get()) without using \"value()\", which might throw an exception, and without the fallback \"value_or(<default>)\".\n\nI am also attaching a minor patch to fix a whitespace error that was not discovered by the checker.\n\nBest,\n\nChristian Rauch (5):\n libcamera: controls: Use std::optional to handle invalid control\n values\n fix whitespace for indentation (replace spaces with tab)\n libcamera: controls: Define size of array controls as a shape vector\n libcamera: controls: Generate fixed- and variable-sized Span Controls\n libcamera: controls: Apply explicit fixed-sized Span type casts\n\n include/libcamera/controls.h | 7 ++--\n src/android/camera_capabilities.cpp | 8 ++---\n src/android/camera_device.cpp | 21 ++++++------\n src/android/camera_hal_manager.cpp | 2 +-\n src/cam/main.cpp | 4 +--\n src/ipa/raspberrypi/raspberrypi.cpp | 21 ++++++------\n src/libcamera/control_ids.yaml | 4 +--\n src/libcamera/pipeline/ipu3/ipu3.cpp | 11 +++----\n .../pipeline/raspberrypi/raspberrypi.cpp | 9 +++---\n src/libcamera/property_ids.yaml | 4 +--\n src/qcam/dng_writer.cpp | 15 +++++----\n utils/gen-controls.py | 32 +++++++++++++------\n 12 files changed, 77 insertions(+), 61 deletions(-)\n\n--\n2.34.1" }