Patch Detail
Show a patch.
GET /api/1.1/patches/12978/?format=api
{ "id": 12978, "url": "https://patchwork.libcamera.org/api/1.1/patches/12978/?format=api", "web_url": "https://patchwork.libcamera.org/patch/12978/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/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": "<20210715211459.19373-8-laurent.pinchart@ideasonboard.com>", "date": "2021-07-15T21:14:33", "name": "[libcamera-devel,v3,07/33] cam: options: Disable copy for parsers", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "27868f3ea513986a64922b17306c7151c67de312", "submitter": { "id": 2, "url": "https://patchwork.libcamera.org/api/1.1/people/2/?format=api", "name": "Laurent Pinchart", "email": "laurent.pinchart@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/12978/mbox/", "series": [ { "id": 2242, "url": "https://patchwork.libcamera.org/api/1.1/series/2242/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=2242", "date": "2021-07-15T21:14:26", "name": "Multi-camera support in the cam application", "version": 3, "mbox": "https://patchwork.libcamera.org/series/2242/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/12978/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/12978/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 924E0C3226\n\tfor <parsemail@patchwork.libcamera.org>;\n\tThu, 15 Jul 2021 21:15:18 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 86F1768551;\n\tThu, 15 Jul 2021 23:15:13 +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 ED7686853A\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 15 Jul 2021 23:15:06 +0200 (CEST)", "from pendragon.lan (62-78-145-57.bb.dnainternet.fi [62.78.145.57])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 76BF756B\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tThu, 15 Jul 2021 23:15:06 +0200 (CEST)" ], "Authentication-Results": "lancelot.ideasonboard.com;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"H2TtDy8K\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1626383706;\n\tbh=oYV6JVyDxp7EFxRmXWOnlY6026MGPvJj1KvTaYunqVM=;\n\th=From:To:Subject:Date:In-Reply-To:References:From;\n\tb=H2TtDy8K/fE4vtLjDETH4IaoNnHAA8Ha308yEKwYoYTNPA8UvjXpsbxH9SmF4e/N3\n\temIeUAckVa1LzM0ZRUMcX4D65FlW39/DJajaKgXcwMpDu4m3W0t463drOEujHeqt8A\n\t4+f+it6guO7lPQK2bcabzr8oViHuOg4yMfHmpaYQ=", "From": "Laurent Pinchart <laurent.pinchart@ideasonboard.com>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Fri, 16 Jul 2021 00:14:33 +0300", "Message-Id": "<20210715211459.19373-8-laurent.pinchart@ideasonboard.com>", "X-Mailer": "git-send-email 2.31.1", "In-Reply-To": "<20210715211459.19373-1-laurent.pinchart@ideasonboard.com>", "References": "<20210715211459.19373-1-laurent.pinchart@ideasonboard.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v3 07/33] cam: options: Disable copy for\n\tparsers", "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": "Copying the OptionsParser class would result in the optionsMap_ entries\npointing to Option entries of the original instance. As there's no use\ncase for copying the class, disable copying.\n\nDisable copying of KeyValueParser as well for consistency as there's no\nuse case either.\n\nSigned-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\nReviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>\n---\n src/cam/options.h | 6 ++++++\n 1 file changed, 6 insertions(+)", "diff": "diff --git a/src/cam/options.h b/src/cam/options.h\nindex 01a5d36a63fb..4418e201bf1f 100644\n--- a/src/cam/options.h\n+++ b/src/cam/options.h\n@@ -68,6 +68,9 @@ public:\n \tvirtual Options parse(const char *arguments);\n \n private:\n+\tKeyValueParser(const KeyValueParser &) = delete;\n+\tKeyValueParser &operator=(const KeyValueParser &) = delete;\n+\n \tfriend class OptionsParser;\n \tvoid usage(int indent);\n \n@@ -95,6 +98,9 @@ public:\n \tvoid usage();\n \n private:\n+\tOptionsParser(const OptionsParser &) = delete;\n+\tOptionsParser &operator=(const OptionsParser &) = delete;\n+\n \tvoid usageOptions(const std::list<Option> &options, unsigned int indent);\n \n \tstd::list<Option> options_;\n", "prefixes": [ "libcamera-devel", "v3", "07/33" ] }