{"id":1432,"url":"https://patchwork.libcamera.org/api/patches/1432/?format=json","web_url":"https://patchwork.libcamera.org/patch/1432/","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":"<20190616133402.21934-3-niklas.soderlund@ragnatech.se>","date":"2019-06-16T13:33:48","name":"[libcamera-devel,v3,02/16] libcamera: geometry: SizeRange: Extend with step information","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"e4a1a4be587f2748202ae707e6515bec143cc059","submitter":{"id":5,"url":"https://patchwork.libcamera.org/api/people/5/?format=json","name":"Niklas Söderlund","email":"niklas.soderlund@ragnatech.se"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/1432/mbox/","series":[{"id":358,"url":"https://patchwork.libcamera.org/api/series/358/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=358","date":"2019-06-16T13:33:46","name":"libcamera: Add support for format information and validation","version":3,"mbox":"https://patchwork.libcamera.org/series/358/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/1432/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/1432/checks/","tags":{},"headers":{"Return-Path":"","Received":["from bin-mail-out-05.binero.net (bin-mail-out-05.binero.net\n\t[195.74.38.228])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 5D5CC6475D\n\tfor ;\n\tSun, 16 Jun 2019 15:35:32 +0200 (CEST)","from bismarck.berto.se (unknown [89.233.230.99])\n\tby bin-vsp-out-01.atm.binero.net (Halon) with ESMTPA\n\tid 959b668a-903b-11e9-8ab4-005056917a89;\n\tSun, 16 Jun 2019 15:35:20 +0200 (CEST)"],"X-Halon-ID":"959b668a-903b-11e9-8ab4-005056917a89","Authorized-sender":"niklas@soderlund.pp.se","From":"=?utf-8?q?Niklas_S=C3=B6derlund?= ","To":"libcamera-devel@lists.libcamera.org","Date":"Sun, 16 Jun 2019 15:33:48 +0200","Message-Id":"<20190616133402.21934-3-niklas.soderlund@ragnatech.se>","X-Mailer":"git-send-email 2.21.0","In-Reply-To":"<20190616133402.21934-1-niklas.soderlund@ragnatech.se>","References":"<20190616133402.21934-1-niklas.soderlund@ragnatech.se>","MIME-Version":"1.0","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v3 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":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Sun, 16 Jun 2019 13:35:32 -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 \nReviewed-by: Jacopo Mondi \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..dfc264b2700f7f61 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 gets incremented by,\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 SizeRange instances describing a single size the incremental step values\n+ * are fixed 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","v3","02/16"]}