~exec64/imv-devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
1

[PATCH v2] Add support for libgrapheme as an icu replacement

Details
Message ID
<20211207134057.13935-1-cem@ckyln.com>
DKIM signature
missing
Download raw message
Patch: +39 -1
Hello,

You were right, I didn't realise how it could be much more simpler,
thank you. Here is the second version of the patch for libgrapheme.
---
 meson.build       | 11 ++++++++++-
 meson_options.txt |  8 ++++++++
 src/console.c     | 21 +++++++++++++++++++++
 3 files changed, 39 insertions(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index 7cf64b5..b2849aa 100644
--- a/meson.build
+++ b/meson.build
@@ -38,6 +38,15 @@ else
  target_single_ws = false
endif

_unicode = get_option('unicode')
if _unicode == 'icu'
  unicode_lib = dependency('icu-io')
  add_project_arguments('-DIMV_USE_ICU', language: 'c')
else
  unicode_lib = cc.find_library('grapheme')
  add_project_arguments('-DIMV_USE_GRAPHEME', language: 'c')
endif

gl_dep = dependency('gl', required: false)
if not gl_dep.found()
  # libglvnd fallback for pure-wayland systems
@@ -49,7 +58,7 @@ deps_for_imv = [
  gl_dep,
  dependency('threads'),
  dependency('xkbcommon'),
  dependency('icu-io'),
  unicode_lib,
  dependency('inih', fallback : ['inih', 'inih_dep']),
  m_dep,
]
diff --git a/meson_options.txt b/meson_options.txt
index 389b7fd..c13ef7a 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -8,6 +8,14 @@ option('windows',
  description : 'window system to use'
)

# Unicode backend - default is ICU
option('unicode',
  type: 'combo',
  value: 'icu',
  choices : ['icu', 'grapheme'],
  description : 'unicode library to use'
)

option('test',
  type : 'feature',
  description : 'enable tests'
diff --git a/src/console.c b/src/console.c
index 073274f..12bb4c3 100644
--- a/src/console.c
+++ b/src/console.c
@@ -6,8 +6,15 @@
#include <ctype.h>
#include <stdlib.h>
#include <string.h>

#ifdef IMV_USE_ICU
#include <unicode/utext.h>
#include <unicode/ubrk.h>
#endif

#ifdef IMV_USE_GRAPHEME
#include <grapheme.h>
#endif

struct imv_console {
  char *buffer;
@@ -25,6 +32,7 @@ struct imv_console {
/* Iterates forwards over characters in a UTF-8 string */
static size_t next_char(char *buffer, size_t position)
{
  #ifdef IMV_USE_ICU
  size_t result = position;
  UErrorCode status = U_ZERO_ERROR;
  UText *ut = utext_openUTF8(NULL, buffer, -1, &status);
@@ -42,11 +50,15 @@ static size_t next_char(char *buffer, size_t position)
  utext_close(ut);
  assert(U_SUCCESS(status));
  return result;
  #elif defined (IMV_USE_GRAPHEME)
  return position + grapheme_bytelen(buffer + position);
  #endif
}

/* Iterates backwards over characters in a UTF-8 string */
static size_t prev_char(char *buffer, size_t position)
{
  #ifdef IMV_USE_ICU
  size_t result = position;
  UErrorCode status = U_ZERO_ERROR;
  UText *ut = utext_openUTF8(NULL, buffer, -1, &status);
@@ -63,6 +75,15 @@ static size_t prev_char(char *buffer, size_t position)

  utext_close(ut);
  assert(U_SUCCESS(status));
  #elif defined (IMV_USE_GRAPHEME)
  size_t result = 0;
  do {
    const size_t step = grapheme_bytelen(buffer + result)
      if (result + step >= position)
        break;
    result += step
  } while (step > 0)
  #endif
  return result;
}

-- 
2.34.1
Details
Message ID
<CGA893XOOXWF.24TQNIOILHZMT@tpad>
In-Reply-To
<20211207134057.13935-1-cem@ckyln.com> (view parent)
DKIM signature
missing
Download raw message
> Here is the second version of the patch for libgrapheme.

Thanks! I gave it a slight tweak and polish, since I realised the logic I
suggested had a bug in next_char. Changes pushed.
Reply to thread Export thread (mbox)