[gspell/wip/icu: 5/5] icu: write implementation notes
- From: Sébastien Wilmet <swilmet src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gspell/wip/icu: 5/5] icu: write implementation notes
- Date: Tue, 6 Oct 2020 00:23:29 +0000 (UTC)
commit a4ae7d150a10d40653ba15717283c3b47a22e2c0
Author: Sébastien Wilmet <swilmet gnome org>
Date: Tue Oct 6 02:20:02 2020 +0200
icu: write implementation notes
https://gitlab.gnome.org/GNOME/gspell/-/issues/14
gspell/gspell-icu.c | 15 +++++++++++++++
1 file changed, 15 insertions(+)
---
diff --git a/gspell/gspell-icu.c b/gspell/gspell-icu.c
index e915c5f..418e5a2 100644
--- a/gspell/gspell-icu.c
+++ b/gspell/gspell-icu.c
@@ -20,6 +20,21 @@
#include "gspell-icu.h"
#include <unicode/uloc.h>
+/* Implementation notes:
+ *
+ * Before using the ICU library, the iso-codes package was used instead. It was
+ * fine on Linux since the iso-codes package is usually already installed. On
+ * MS Windows, to package an application the iso-codes package needed to be
+ * bundled with the app, which is not great because it's a quite sizeable dep.
+ *
+ * The ICU library is usually already installed too on Linux, and for Windows
+ * it's already an (indirect) dep of gedit (the Tepl library uses the ICU).
+ *
+ * MS Windows also provides a native API for something similar to
+ * _gspell_icu_get_language_name_from_code(). But it's easier to manage only one
+ * implementation (namely, based on the ICU library).
+ */
+
/* Some of these functions are copied from the Tepl library. */
/* Wrapper around u_strToUTF8() that handles the pre-flighting.
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]