{"id":20026,"url":"https://patchwork.libcamera.org/api/patches/20026/?format=json","web_url":"https://patchwork.libcamera.org/patch/20026/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20240508205914.15611-2-laurent.pinchart@ideasonboard.com>","date":"2024-05-08T20:59:13","name":"[v1,1/2] libcamera: Drop file name from header comment blocks in templates","commit_ref":"829acb3ab0b178110f449bed238ef117fa96408d","pull_url":null,"state":"accepted","archived":false,"hash":"4c3a1663acbbfda57d289af6d65673a1ae448da3","submitter":{"id":2,"url":"https://patchwork.libcamera.org/api/people/2/?format=json","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/20026/mbox/","series":[{"id":4300,"url":"https://patchwork.libcamera.org/api/series/4300/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=4300","date":"2024-05-08T20:59:12","name":"libcamera: Drop remaining file name from header comment blocks","version":1,"mbox":"https://patchwork.libcamera.org/series/4300/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/20026/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/20026/checks/","tags":{},"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 6DDBFC32B5\n\tfor <parsemail@patchwork.libcamera.org>;\n\tWed,  8 May 2024 20:59:27 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id D6D6B6343D;\n\tWed,  8 May 2024 22:59:26 +0200 (CEST)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 77C366343D\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed,  8 May 2024 22:59:24 +0200 (CEST)","from pendragon.ideasonboard.com (81-175-209-231.bb.dnainternet.fi\n\t[81.175.209.231])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 17086FD6\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed,  8 May 2024 22:59:21 +0200 (CEST)"],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"wO2RenlI\"; dkim-atps=neutral","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1715201961;\n\tbh=p10l/nL79Zm6TfbiJOHNOZYqeCyaLfc9SIl5XIcqCgs=;\n\th=From:To:Subject:Date:In-Reply-To:References:From;\n\tb=wO2RenlI/RDFzkhY2BfzvK5z9nsurDHh9HiCVttZ/W0oIMSGN4sLs4e7ZBU+QC50d\n\tNVYVfac9yz1+c2YFk2dqlepioqrbdgP3XsS/itxca5XtoBhdxBt6PrAou8FiWGD1dI\n\tHJea2InNknsg5iF5NkU2BfF0FS0vPYSKiF1LCc1I=","From":"Laurent Pinchart <laurent.pinchart@ideasonboard.com>","To":"libcamera-devel@lists.libcamera.org","Subject":"[PATCH v1 1/2] libcamera: Drop file name from header comment blocks\n\tin templates","Date":"Wed,  8 May 2024 23:59:13 +0300","Message-ID":"<20240508205914.15611-2-laurent.pinchart@ideasonboard.com>","X-Mailer":"git-send-email 2.43.2","In-Reply-To":"<20240508205914.15611-1-laurent.pinchart@ideasonboard.com>","References":"<20240508205914.15611-1-laurent.pinchart@ideasonboard.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","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>","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>"},"content":"Source files in libcamera start by a comment block header, which\nincludes the file name and a one-line description of the file contents.\nWhile the latter is useful to get a quick overview of the file contents\nat a glance, the former is mostly a source of inconvenience. The name in\nthe comments can easily get out of sync with the file name when files\nare renamed, and copy & paste during development have often lead to\nincorrect names being used to start with.\n\nReaders of the source code are expected to know which file they're\nlooking it. Drop the file name from the header comment blocks in\ntemplate files and templates embedded in generator scripts.\n\nSigned-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n---\n include/libcamera/control_ids.h.in                              | 2 +-\n include/libcamera/formats.h.in                                  | 2 +-\n include/libcamera/internal/tracepoints.h.in                     | 2 +-\n include/libcamera/property_ids.h.in                             | 2 +-\n include/libcamera/version.h.in                                  | 2 +-\n src/libcamera/ipa_pub_key.cpp.in                                | 2 +-\n src/libcamera/version.cpp.in                                    | 2 +-\n utils/gen-header.sh                                             | 2 +-\n utils/ipc/extract-docs.py                                       | 2 +-\n .../generators/libcamera_templates/core_ipa_interface.h.tmpl    | 2 +-\n .../generators/libcamera_templates/core_ipa_serializer.h.tmpl   | 2 +-\n .../generators/libcamera_templates/module_ipa_interface.h.tmpl  | 2 +-\n .../generators/libcamera_templates/module_ipa_proxy.cpp.tmpl    | 2 +-\n .../ipc/generators/libcamera_templates/module_ipa_proxy.h.tmpl  | 2 +-\n .../libcamera_templates/module_ipa_proxy_worker.cpp.tmpl        | 2 +-\n .../generators/libcamera_templates/module_ipa_serializer.h.tmpl | 2 +-\n 16 files changed, 16 insertions(+), 16 deletions(-)","diff":"diff --git a/include/libcamera/control_ids.h.in b/include/libcamera/control_ids.h.in\nindex 58dd48e169c7..293ba966fbc4 100644\n--- a/include/libcamera/control_ids.h.in\n+++ b/include/libcamera/control_ids.h.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2019, Google Inc.\n  *\n- * control_ids.h - Control ID list\n+ * Control ID list\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/include/libcamera/formats.h.in b/include/libcamera/formats.h.in\nindex ead5287d8299..6ae7634fed34 100644\n--- a/include/libcamera/formats.h.in\n+++ b/include/libcamera/formats.h.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * formats.h - Formats\n+ * Formats\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/include/libcamera/internal/tracepoints.h.in b/include/libcamera/internal/tracepoints.h.in\nindex d0fc136598b6..f0962091da8f 100644\n--- a/include/libcamera/internal/tracepoints.h.in\n+++ b/include/libcamera/internal/tracepoints.h.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) {{year}}, Google Inc.\n  *\n- * tracepoints.h - Tracepoints with lttng\n+ * Tracepoints with lttng\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/include/libcamera/property_ids.h.in b/include/libcamera/property_ids.h.in\nindex f51ba028bb2e..e6edabca771f 100644\n--- a/include/libcamera/property_ids.h.in\n+++ b/include/libcamera/property_ids.h.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2019, Google Inc.\n  *\n- * property_ids.h - Property ID list\n+ * Property ID list\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/include/libcamera/version.h.in b/include/libcamera/version.h.in\nindex 6e24d0a89fcb..50bf1001ec0f 100644\n--- a/include/libcamera/version.h.in\n+++ b/include/libcamera/version.h.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2019, Google Inc.\n  *\n- * version.h - Library version information\n+ * Library version information\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/src/libcamera/ipa_pub_key.cpp.in b/src/libcamera/ipa_pub_key.cpp.in\nindex 01e5333b4d30..5d8c92c2c39d 100644\n--- a/src/libcamera/ipa_pub_key.cpp.in\n+++ b/src/libcamera/ipa_pub_key.cpp.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2020, Laurent Pinchart <laurent.pinchart@ideasonboard.com>\n  *\n- * ipa_pub_key.cpp - IPA module signing public key\n+ * IPA module signing public key\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/src/libcamera/version.cpp.in b/src/libcamera/version.cpp.in\nindex 5aec08a189f3..bf5a2c309676 100644\n--- a/src/libcamera/version.cpp.in\n+++ b/src/libcamera/version.cpp.in\n@@ -2,7 +2,7 @@\n /*\n  * Copyright (C) 2019, Google Inc.\n  *\n- * version.cpp - libcamera version\n+ * libcamera version\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/gen-header.sh b/utils/gen-header.sh\nindex 8b66c5dd0f47..d4692758eeb4 100755\n--- a/utils/gen-header.sh\n+++ b/utils/gen-header.sh\n@@ -9,7 +9,7 @@ cat <<EOF > \"$dst_file\"\n /*\n  * Copyright (C) 2018-2019, Google Inc.\n  *\n- * libcamera.h - libcamera public API\n+ * libcamera public API\n  */\n \n #pragma once\ndiff --git a/utils/ipc/extract-docs.py b/utils/ipc/extract-docs.py\nindex e7302b9a3798..61f44caef52a 100755\n--- a/utils/ipc/extract-docs.py\n+++ b/utils/ipc/extract-docs.py\n@@ -38,7 +38,7 @@ def main(argv):\n /*\n  * Copyright (C) 2021, Google Inc.\n  *\n- * {pipeline}_ipa_interface.cpp - Docs file for generated {pipeline}.mojom\n+ * Docs file for generated {pipeline}.mojom\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/core_ipa_interface.h.tmpl b/utils/ipc/generators/libcamera_templates/core_ipa_interface.h.tmpl\nindex c60b99b8e049..7f2d0810823b 100644\n--- a/utils/ipc/generators/libcamera_templates/core_ipa_interface.h.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/core_ipa_interface.h.tmpl\n@@ -7,7 +7,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * core_ipa_interface.h - libcamera core definitions for Image Processing Algorithms\n+ * libcamera core definitions for Image Processing Algorithms\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/core_ipa_serializer.h.tmpl b/utils/ipc/generators/libcamera_templates/core_ipa_serializer.h.tmpl\nindex 5738a1aa3405..036518f6cfbd 100644\n--- a/utils/ipc/generators/libcamera_templates/core_ipa_serializer.h.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/core_ipa_serializer.h.tmpl\n@@ -8,7 +8,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * core_ipa_serializer.h - Data serializer for core libcamera definitions for IPA\n+ * Data serializer for core libcamera definitions for IPA\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/module_ipa_interface.h.tmpl b/utils/ipc/generators/libcamera_templates/module_ipa_interface.h.tmpl\nindex 160601f72348..4d88a3d77d0f 100644\n--- a/utils/ipc/generators/libcamera_templates/module_ipa_interface.h.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/module_ipa_interface.h.tmpl\n@@ -7,7 +7,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * {{module_name}}_ipa_interface.h - Image Processing Algorithm interface for {{module_name}}\n+ * Image Processing Algorithm interface for {{module_name}}\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/module_ipa_proxy.cpp.tmpl b/utils/ipc/generators/libcamera_templates/module_ipa_proxy.cpp.tmpl\nindex f64c3c93bfcb..ce3cc5ab62a3 100644\n--- a/utils/ipc/generators/libcamera_templates/module_ipa_proxy.cpp.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/module_ipa_proxy.cpp.tmpl\n@@ -8,7 +8,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * {{module_name}}_ipa_proxy.cpp - Image Processing Algorithm proxy for {{module_name}}\n+ * Image Processing Algorithm proxy for {{module_name}}\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/module_ipa_proxy.h.tmpl b/utils/ipc/generators/libcamera_templates/module_ipa_proxy.h.tmpl\nindex 6e823598930e..e213b18a051f 100644\n--- a/utils/ipc/generators/libcamera_templates/module_ipa_proxy.h.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/module_ipa_proxy.h.tmpl\n@@ -8,7 +8,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * {{module_name}}_ipa_proxy.h - Image Processing Algorithm proxy for {{module_name}}\n+ * Image Processing Algorithm proxy for {{module_name}}\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/module_ipa_proxy_worker.cpp.tmpl b/utils/ipc/generators/libcamera_templates/module_ipa_proxy_worker.cpp.tmpl\nindex b65dc4cf31c5..1f990d3f9d1c 100644\n--- a/utils/ipc/generators/libcamera_templates/module_ipa_proxy_worker.cpp.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/module_ipa_proxy_worker.cpp.tmpl\n@@ -8,7 +8,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * {{module_name}}_ipa_proxy_worker.cpp - Image Processing Algorithm proxy worker for {{module_name}}\n+ * Image Processing Algorithm proxy worker for {{module_name}}\n  *\n  * This file is auto-generated. Do not edit.\n  */\ndiff --git a/utils/ipc/generators/libcamera_templates/module_ipa_serializer.h.tmpl b/utils/ipc/generators/libcamera_templates/module_ipa_serializer.h.tmpl\nindex 8b709705aa32..cd5a65a9200e 100644\n--- a/utils/ipc/generators/libcamera_templates/module_ipa_serializer.h.tmpl\n+++ b/utils/ipc/generators/libcamera_templates/module_ipa_serializer.h.tmpl\n@@ -8,7 +8,7 @@\n /*\n  * Copyright (C) 2020, Google Inc.\n  *\n- * {{module_name}}_ipa_serializer.h - Image Processing Algorithm data serializer for {{module_name}}\n+ * Image Processing Algorithm data serializer for {{module_name}}\n  *\n  * This file is auto-generated. Do not edit.\n  */\n","prefixes":["v1","1/2"]}