[vte/wip/egmont/bidi: 68/79] fine grained vis2log and log2vis



commit 85ec6ade4fba76b9b2134582b186f3c211cf7e08
Author: Egmont Koblinger <egmont gmail com>
Date:   Fri Aug 31 18:28:00 2018 +0200

    fine grained vis2log and log2vis

 src/bidi.cc | 69 ++++++++++++++++++++++++++++++++++++++++++-------------------
 src/bidi.hh |  8 +++----
 2 files changed, 52 insertions(+), 25 deletions(-)
---
diff --git a/src/bidi.cc b/src/bidi.cc
index e5ecbec6..70f7abf2 100644
--- a/src/bidi.cc
+++ b/src/bidi.cc
@@ -65,45 +65,72 @@ void BidiRow::set_width(vte::grid::column_t width)
         m_width = width;
 }
 
-/* Converts from logical to visual column. Offscreen columns are mirrored
- * for RTL lines, e.g. (assuming 80 columns) -1 <=> 80, -2 <=> 81 etc. */
-vte::grid::column_t BidiRow::log2vis(vte::grid::column_t col) const
+/* Converts from logical to visual column.
+ *
+ * The input column is assumed to be scaled up by the factor, and so will be the return value.
+ * (They can contain e.g. half cell coordinates or pixel coordinates.)
+ * The remainder is preserved or mirrored depending on the character's resolved directionality.
+ *
+ * Offscreen values are mirrored for RTL lines.
+ *
+ * Example: 80 columns, logical column 36 has RTL char mapped to visual column 42. Factor is 10.
+ * Input values 360..369 are converted to return values 429..420 respectively.
+ * If the line's base dir is RTL, offscreen values are mapped like -1 <=> 800, -2 <=> 801 etc. */
+vte::grid::column_t BidiRow::log2vis(vte::grid::column_t col, int factor) const
 {
-        if (col >= 0 && col < m_width) {
-                return m_log2vis[col];
+        if (col >= 0 && col < m_width * factor) {
+                int rem = col % factor;
+                col /= factor;
+                col = m_log2vis[col];
+                rem = m_vis_rtl[col] ? factor - 1 - rem : rem;
+                return col * factor + rem;
         } else {
-                return m_base_rtl ? m_width - 1 - col : col;
+                return m_base_rtl ? m_width * factor - 1 - col : col;
         }
 }
 
-/* Converts from visual to logical column. Offscreen columns are mirrored
- * for RTL lines, e.g. (assuming 80 columns) -1 <=> 80, -2 <=> 81 etc. */
-vte::grid::column_t BidiRow::vis2log(vte::grid::column_t col) const
+/* Converts from visual to logical column.
+ *
+ * See log2vis() for details. */
+vte::grid::column_t BidiRow::vis2log(vte::grid::column_t col, int factor) const
 {
-        if (col >= 0 && col < m_width) {
-                return m_vis2log[col];
+        if (col >= 0 && col < m_width * factor) {
+                int rem = col % factor;
+                col /= factor;
+                rem = m_vis_rtl[col] ? factor - 1 - rem : rem;
+                col = m_vis2log[col];
+                return col * factor + rem;
         } else {
-                return m_base_rtl ? m_width - 1 - col : col;
+                return m_base_rtl ? m_width * factor - 1 - col : col;
         }
 }
 
-/* Whether the cell at the given visual position has RTL directionality.
- * For offscreen columns the line's base direction is returned. */
-bool BidiRow::vis_is_rtl(vte::grid::column_t col) const
+/* Whether the cell at the given logical position has RTL resolved directionality.
+ *
+ * The input column is assumed to be scaled up by the factor. (It can contain e.g.
+ * half cell coordinate or pixel coordinate.)
+ *
+ * For offscreen columns the line's base direction is returned.
+ *
+ * (Note: the caller doing the division would be error prone at negative columns.) */
+bool BidiRow::log_is_rtl(vte::grid::column_t col, int factor) const
 {
-        if (col >= 0 && col < m_width) {
+        if (col >= 0 && col < m_width * factor) {
+                col /= factor;
+                col = m_log2vis[col];
                 return m_vis_rtl[col];
         } else {
                 return m_base_rtl;
         }
 }
 
-/* Whether the cell at the given logical position has RTL directionality.
- * For offscreen columns the line's base direction is returned. */
-bool BidiRow::log_is_rtl(vte::grid::column_t col) const
+/* Whether the cell at the given visual position has RTL resolved directionality.
+ *
+ * See log_is_rtl() for details. */
+bool BidiRow::vis_is_rtl(vte::grid::column_t col, int factor) const
 {
-        if (col >= 0 && col < m_width) {
-                col = m_log2vis[col];
+        if (col >= 0 && col < m_width * factor) {
+                col /= factor;
                 return m_vis_rtl[col];
         } else {
                 return m_base_rtl;
diff --git a/src/bidi.hh b/src/bidi.hh
index 7044af74..093c35f6 100644
--- a/src/bidi.hh
+++ b/src/bidi.hh
@@ -44,10 +44,10 @@ public:
         BidiRow& operator= (BidiRow const& o) = delete;
         BidiRow& operator= (BidiRow&& o) = delete;
 
-        vte::grid::column_t log2vis(vte::grid::column_t col) const;
-        vte::grid::column_t vis2log(vte::grid::column_t col) const;
-        bool log_is_rtl(vte::grid::column_t col) const;
-        bool vis_is_rtl(vte::grid::column_t col) const;
+        vte::grid::column_t log2vis(vte::grid::column_t col, int factor = 1) const;
+        vte::grid::column_t vis2log(vte::grid::column_t col, int factor = 1) const;
+        bool log_is_rtl(vte::grid::column_t col, int factor = 1) const;
+        bool vis_is_rtl(vte::grid::column_t col, int factor = 1) const;
         bool base_is_rtl() const;
         bool has_foreign() const;
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]