Patch Detail
Show a patch.
GET /api/patches/1398/?format=api
{ "id": 1398, "url": "https://patchwork.libcamera.org/api/patches/1398/?format=api", "web_url": "https://patchwork.libcamera.org/patch/1398/", "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": "<20190612004359.15772-3-niklas.soderlund@ragnatech.se>", "date": "2019-06-12T00:43:45", "name": "[libcamera-devel,v2,02/16] libcamera: geometry: SizeRange: Extend with step information", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "091b1c0c9b839747d033d02e5fa00de43c974ef9", "submitter": { "id": 5, "url": "https://patchwork.libcamera.org/api/people/5/?format=api", "name": "Niklas Söderlund", "email": "niklas.soderlund@ragnatech.se" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/1398/mbox/", "series": [ { "id": 351, "url": "https://patchwork.libcamera.org/api/series/351/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=351", "date": "2019-06-12T00:43:43", "name": "libcamera: Add support for format information and validation", "version": 2, "mbox": "https://patchwork.libcamera.org/series/351/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/1398/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/1398/checks/", "tags": {}, "headers": { "Return-Path": "<niklas.soderlund@ragnatech.se>", "Received": [ "from bin-mail-out-06.binero.net (bin-mail-out-06.binero.net\n\t[195.74.38.229])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 8C3D161F83\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tWed, 12 Jun 2019 02:44:54 +0200 (CEST)", "from bismarck.berto.se (unknown [89.233.230.99])\n\tby bin-vsp-out-03.atm.binero.net (Halon) with ESMTPA\n\tid 34d571ee-8cab-11e9-8601-0050569116f7;\n\tWed, 12 Jun 2019 02:44:16 +0200 (CEST)" ], "X-Halon-ID": "34d571ee-8cab-11e9-8601-0050569116f7", "Authorized-sender": "niklas@soderlund.pp.se", "From": "=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Wed, 12 Jun 2019 02:43:45 +0200", "Message-Id": "<20190612004359.15772-3-niklas.soderlund@ragnatech.se>", "X-Mailer": "git-send-email 2.21.0", "In-Reply-To": "<20190612004359.15772-1-niklas.soderlund@ragnatech.se>", "References": "<20190612004359.15772-1-niklas.soderlund@ragnatech.se>", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=UTF-8", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v2 02/16] libcamera: geometry: SizeRange:\n\tExtend with step information", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.23", "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>", "X-List-Received-Date": "Wed, 12 Jun 2019 00:44:54 -0000" }, "content": "The size range described might be subject to certain step\nlimitations. Make it possible to record this information.\n\nSigned-off-by: Niklas Söderlund <niklas.soderlund@ragnatech.se>\n---\n include/libcamera/geometry.h | 13 +++++++++--\n src/libcamera/geometry.cpp | 44 +++++++++++++++++++++++++++++++++---\n 2 files changed, 52 insertions(+), 5 deletions(-)", "diff": "diff --git a/include/libcamera/geometry.h b/include/libcamera/geometry.h\nindex ec5ed2bee196c82d..b2a8e5b0e005e4db 100644\n--- a/include/libcamera/geometry.h\n+++ b/include/libcamera/geometry.h\n@@ -73,18 +73,27 @@ struct SizeRange {\n \t}\n \n \tSizeRange(unsigned int width, unsigned int height)\n-\t\t: min(width, height), max(width, height)\n+\t\t: min(width, height), max(width, height), hStep(1), vStep(1)\n \t{\n \t}\n \n \tSizeRange(unsigned int minW, unsigned int minH,\n \t\t unsigned int maxW, unsigned int maxH)\n-\t\t: min(minW, minH), max(maxW, maxH)\n+\t\t: min(minW, minH), max(maxW, maxH), hStep(1), vStep(1)\n+\t{\n+\t}\n+\n+\tSizeRange(unsigned int minW, unsigned int minH,\n+\t\t unsigned int maxW, unsigned int maxH,\n+\t\t unsigned int hstep, unsigned int vstep)\n+\t\t: min(minW, minH), max(maxW, maxH), hStep(hstep), vStep(vstep)\n \t{\n \t}\n \n \tSize min;\n \tSize max;\n+\tunsigned int hStep;\n+\tunsigned int vStep;\n };\n \n bool operator==(const SizeRange &lhs, const SizeRange &rhs);\ndiff --git a/src/libcamera/geometry.cpp b/src/libcamera/geometry.cpp\nindex 26a05b9e7d800077..c56c4e399ef307f5 100644\n--- a/src/libcamera/geometry.cpp\n+++ b/src/libcamera/geometry.cpp\n@@ -186,9 +186,24 @@ bool operator<(const Size &lhs, const Size &rhs)\n * \\struct SizeRange\n * \\brief Describe a range of sizes\n *\n- * SizeRange describes a range of sizes included in the [min, max]\n- * interval for both the width and the height. If the minimum and\n- * maximum sizes are identical it represents a single size.\n+ * SizeRange describes a range of sizes included in the [min, max] interval\n+ * for both the width and the height. If the minimum and maximum sizes are\n+ * identical it represents a single size.\n+ *\n+ * The SizeRange may also describe a vertical and horizontal step. The step\n+ * describes the value in pixels the width/height size can be incremented with,\n+ * starting from the minimum size.\n+ *\n+ *\twidth = min.width + hStep * x\n+ *\theight = min.height + vStep * y\n+ *\n+ *\tWhere:\n+ *\n+ *\twidth <= max.width\n+ *\theight < max.height\n+ *\n+ * For SizeRanges describing a single size the incremental step values are fixed\n+ * to 1.\n */\n \n /**\n@@ -213,6 +228,19 @@ bool operator<(const Size &lhs, const Size &rhs)\n * \\param[in] maxH The maximum height\n */\n \n+/**\n+ * \\fn SizeRange::SizeRange(unsigned int minW, unsigned int minH,\n+ *\t\t\t unsigned int maxW, unsigned int maxH,\n+ *\t\t\t unsigned int hstep, unsigned int vstep)\n+ * \\brief Construct an initialized size range\n+ * \\param[in] minW The minimum width\n+ * \\param[in] minH The minimum height\n+ * \\param[in] maxW The maximum width\n+ * \\param[in] maxH The maximum height\n+ * \\param[in] hstep The horizontal step\n+ * \\param[in] vstep The vertical step\n+ */\n+\n /**\n * \\var SizeRange::min\n * \\brief The minimum size\n@@ -223,6 +251,16 @@ bool operator<(const Size &lhs, const Size &rhs)\n * \\brief The maximum size\n */\n \n+/**\n+ * \\var SizeRange::hStep\n+ * \\brief The horizontal step\n+ */\n+\n+/**\n+ * \\var SizeRange::vStep\n+ * \\brief The vertical step\n+ */\n+\n /**\n * \\brief Compare size ranges for equality\n * \\return True if the two size ranges are equal, false otherwise\n", "prefixes": [ "libcamera-devel", "v2", "02/16" ] }