po/make-template.sh

@@ -1,19 +1,47 @@
-#!/bin/sh -e
+#!/bin/sh
 # This shell script generates the translation template.
 #
-# The reason for this not being inside CMakeLists.txt is that the translator
+# The reason for this not being inside CMakeLists.txt
-# should not need to run the whole configuration process just to get this file.
+# is that the translator should not need to run the whole
-dir=$(dirname $0)
+# configuration process just to get this single stupid file.
 
-re='^[ \t]*project *( *\([^ \t)]\{1,\}\) \{1,\}VERSION \{1,\}\([^ \t)]\{1,\}\).*'
+# Get the directory this script resides in so that the user
-package=$(sed -n "s/$re/\\1/p" "$dir/../CMakeLists.txt")
+# doesn't have to run the script from there
-version=$(sed -n "s/$re/\\2/p" "$dir/../CMakeLists.txt")
+DIR=$(dirname $0)
-if [ -z "$package" -o -z "$version" ]; then
+
+# Collect source files
+SOURCES=$(echo $DIR/../src/*.c)
+
+# Get the package name from CMakeLists.txt
+PACKAGE=$(sed -n '/^[ \t]*project[ \t]*([ \t]*\([^ \t)]\{1,\}\).*).*/{s//\1/p;q}' \
+	$DIR/../CMakeLists.txt)
+
+# Get the package version from CMakeLists.txt
+EXP_BEG='/^[ \t]*set[ \t]*([ \t]*project_VERSION_'
+EXP_END='[ \t]\{1,\}"\{0,1\}\([^)"]\{1,\}\)"\{0,1\}).*/{s//\1/p;q}'
+
+MAJOR=$(sed -n "${EXP_BEG}MAJOR${EXP_END}" $DIR/../CMakeLists.txt)
+MINOR=$(sed -n "${EXP_BEG}MINOR${EXP_END}" $DIR/../CMakeLists.txt)
+PATCH=$(sed -n "${EXP_BEG}PATCH${EXP_END}" $DIR/../CMakeLists.txt)
+
+if [ "$MAJOR" != "" ]; then
+	VERSION=$MAJOR
+	if [ "$MINOR" != "" ]; then
+		VERSION=$VERSION.$MINOR
+		if [ "$PATCH" != "" ]; then
+			VERSION=$VERSION.$PATCH
+		fi
+	fi
+fi
+
+if [ -z "$PACKAGE" -o -z "$VERSION" ]; then
 	echo "Failed to get information from CMakeLists.txt"
 	exit 1
 fi
 
-xgettext -LC -k_ -kN_ $dir/../src/*.c -o "$dir/$package.pot" \
+# Finally make the template
-	--package-name="$package" --package-version="$version" \
+xgettext -LC -k_ -kN_ $SOURCES -o "$DIR/$PACKAGE.pot" \
+	--package-name="$PACKAGE" --package-version="$VERSION" \
 	--copyright-holder="Přemysl Eric Janouch" \
-	--msgid-bugs-address="https://git.janouch.name/p/$package/issues"
+	--msgid-bugs-address="https://github.com/pjanouch/$PACKAGE/issues"
+

po/sdtui.pot

@@ -7,8 +7,8 @@
 msgid ""
 msgstr ""
 "Project-Id-Version: sdtui 0.1.0\n"
-"Report-Msgid-Bugs-To: https://git.janouch.name/p/sdtui/issues\n"
+"Report-Msgid-Bugs-To: https://github.com/pjanouch/sdtui/issues\n"
-"POT-Creation-Date: 2021-10-11 21:10+0200\n"
+"POT-Creation-Date: 2016-09-28 16:12+0200\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
@@ -17,137 +17,115 @@ msgstr ""
 "Content-Type: text/plain; charset=UTF-8\n"
 "Content-Transfer-Encoding: 8bit\n"
 
-#: ../src/sdgtk.c:289 ../src/sdtui.c:2481
+#: ../src/sdtui.c:283
-msgid "Warning"
-msgstr ""
-
-#: ../src/sdgtk.c:289 ../src/sdtui.c:2481
-msgid "failed to set the locale"
-msgstr ""
-
-#: ../src/sdgtk.c:299
-msgid "FILE..."
-msgstr ""
-
-#: ../src/sdgtk.c:304
-msgid "- StarDict GTK+ UI"
-msgstr ""
-
-#: ../src/sdgtk.c:355
-msgid "Follow selection"
-msgstr ""
-
-#: ../src/sdtui.c:320
 msgid "error in entry"
 msgstr ""
 
-#: ../src/sdtui.c:415
+#: ../src/sdtui.c:302
 msgid "no usable field found"
 msgstr ""
 
-#: ../src/sdtui.c:700
+#: ../src/sdtui.c:548
 msgid "Search"
 msgstr ""
 
-#: ../src/sdtui.c:726
+#: ../src/sdtui.c:573
 msgid "Cannot load configuration"
 msgstr ""
 
-#: ../src/sdtui.c:745
+#: ../src/sdtui.c:606
 msgid "Error loading dictionary"
 msgstr ""
 
-#: ../src/sdtui.c:750
+#: ../src/sdtui.c:612
 msgid ""
 "No dictionaries found either in the configuration or on the command line"
 msgstr ""
 
-#: ../src/sdtui.c:1114
+#: ../src/sdtui.c:966
 msgid "Terminal UI for StarDict dictionaries"
 msgstr ""
 
-#: ../src/sdtui.c:1117
+#: ../src/sdtui.c:969
 msgid "Type to search"
 msgstr ""
 
-#: ../src/sdtui.c:2095
+#: ../src/sdtui.c:1976
-#, c-format
-msgid "X11 connection failed (error code %d)"
-msgstr ""
-
-#: ../src/sdtui.c:2241
-#, c-format
-msgid "X11 request error (%d, major %d, minor %d)"
-msgstr ""
-
-#: ../src/sdtui.c:2476
 msgid "Output version information and exit"
 msgstr ""
 
-#: ../src/sdtui.c:2489
+#: ../src/sdtui.c:1981
+msgid "Warning"
+msgstr ""
+
+#: ../src/sdtui.c:1981
+msgid "failed to set the locale"
+msgstr ""
+
+#: ../src/sdtui.c:1989
 msgid "[dictionary.ifo...] - StarDict terminal UI"
 msgstr ""
 
-#: ../src/sdtui.c:2497
+#: ../src/sdtui.c:1997
 msgid "Error"
 msgstr ""
 
-#: ../src/sdtui.c:2497
+#: ../src/sdtui.c:1997
 msgid "option parsing failed"
 msgstr ""
 
-#: ../src/stardict.c:89
+#: ../src/stardict.c:91
 msgid "invalid header format"
 msgstr ""
 
-#: ../src/stardict.c:249
+#: ../src/stardict.c:227
 msgid "version not specified"
 msgstr ""
 
-#: ../src/stardict.c:260
+#: ../src/stardict.c:238
 msgid "invalid version"
 msgstr ""
 
-#: ../src/stardict.c:274
+#: ../src/stardict.c:252
 msgid "unknown key, ignoring"
 msgstr ""
 
-#: ../src/stardict.c:281
+#: ../src/stardict.c:259
 msgid "invalid encoding, must be valid UTF-8"
 msgstr ""
 
-#: ../src/stardict.c:298
+#: ../src/stardict.c:276
 msgid "invalid integer"
 msgstr ""
 
-#: ../src/stardict.c:308
+#: ../src/stardict.c:286
 msgid "option format error"
 msgstr ""
 
-#: ../src/stardict.c:318
+#: ../src/stardict.c:296
 msgid "no book name specified"
 msgstr ""
 
-#: ../src/stardict.c:324
+#: ../src/stardict.c:302
 msgid "word count not specified"
 msgstr ""
 
-#: ../src/stardict.c:330
+#: ../src/stardict.c:308
 msgid "index file size not specified"
 msgstr ""
 
-#: ../src/stardict.c:339
+#: ../src/stardict.c:317
 msgid "invalid index offset bits"
 msgstr ""
 
-#: ../src/stardict.c:827
+#: ../src/stardict.c:812
 msgid "cannot find .idx file"
 msgstr ""
 
-#: ../src/stardict.c:850
+#: ../src/stardict.c:835
 msgid "cannot find .dict file"
 msgstr ""
 
-#: ../src/stardict.c:1153 ../src/stardict.c:1178
+#: ../src/stardict.c:1130 ../src/stardict.c:1155
 msgid "invalid data entry"
 msgstr ""

src/sdtui.c

@@ -242,9 +242,8 @@ struct application
 	struct attrs    attrs[ATTRIBUTE_COUNT];
 };
 
-/// Shortcuts to retrieve named terminal attributes
+/// Shortcut to retrieve named terminal attributes
 #define APP_ATTR(name) self->attrs[ATTRIBUTE_ ## name].attrs
-#define APP_ATTR_IF(b, name1, name2) ((b) ? APP_ATTR (name1) : APP_ATTR (name2))
 
 /// Returns if the Unicode character is representable in the current locale.
 static gboolean
@@ -847,15 +846,12 @@ struct row_buffer
 	int total_width;                    ///< Total width of all characters
 };
 
-static RowBuffer
+static void
-row_buffer_make (Application *app)
+row_buffer_init (RowBuffer *self, Application *app)
 {
-	return (RowBuffer)
+	self->app = app;
-	{
+	self->chars = g_array_new (FALSE, TRUE, sizeof (RowChar));
-		.app = app,
+	self->total_width = 0;
-		.chars = g_array_new (FALSE, TRUE, sizeof (RowChar)),
-		.total_width = 0,
-	};
 }
 
 #define row_buffer_free(self) g_array_unref ((self)->chars)
@@ -865,9 +861,6 @@ static void
 row_buffer_append_length (RowBuffer *self,
 	const gchar *text, glong length, chtype attrs)
 {
-	if (length == 0)
-		return;
-
 	glong ucs4_len;
 	gunichar *ucs4 = g_utf8_to_ucs4_fast (text, length, &ucs4_len);
 	for (glong i = 0; i < ucs4_len; i++)
@@ -1022,20 +1015,21 @@ row_buffer_finish (RowBuffer *self, int width, chtype attrs)
 static void
 app_redraw_top (Application *self)
 {
-	RowBuffer buf = row_buffer_make (self);
+	RowBuffer buf;
+	row_buffer_init (&buf, self);
 	row_buffer_append (&buf, APP_TITLE, APP_ATTR (HEADER) | A_BOLD);
 
 	for (guint i = 0; i < self->dictionaries->len; i++)
 	{
 		Dictionary *dict = &g_array_index (self->dictionaries, Dictionary, i);
 		row_buffer_append (&buf, dict->name,
-			APP_ATTR_IF (self->dictionaries->len > 1
+			(self->dictionaries->len > 1 && self->dict == dict->dict)
-				&& self->dict == dict->dict, ACTIVE, HEADER));
+				? APP_ATTR (ACTIVE) : APP_ATTR (HEADER));
 	}
 	move (0, 0);
 	row_buffer_finish (&buf, COLS, APP_ATTR (HEADER));
 
-	buf = row_buffer_make (self);
+	row_buffer_init (&buf, self);
 	row_buffer_append (&buf, self->search_label, APP_ATTR (SEARCH));
 	gsize indent = buf.total_width;
 
@@ -1087,13 +1081,17 @@ app_show_message (Application *self, const gchar *lines[], gsize len)
 
 	while (len-- && i < LINES - TOP_BAR_CUTOFF)
 	{
-		RowBuffer buf = row_buffer_make (self);
-		row_buffer_append (&buf, *lines, 0);
-		gint x = (COLS - buf.total_width) / 2;
-
 		move (TOP_BAR_CUTOFF + i, 0);
 		clrtoeol ();
-		move (TOP_BAR_CUTOFF + i, MAX (x, 0));
+
+		gint x = (COLS - g_utf8_strlen (*lines, -1)) / 2;
+		if (x < 0)
+			x = 0;
+
+		RowBuffer buf;
+		row_buffer_init (&buf, self);
+		row_buffer_append (&buf, *lines, 0);
+		move (TOP_BAR_CUTOFF + i, x);
 		row_buffer_finish (&buf, -1, 0);
 
 		lines++;
@@ -1120,28 +1118,6 @@ app_show_help (Application *self)
 	app_show_message (self, lines, G_N_ELEMENTS (lines));
 }
 
-static void
-app_draw_word (Application *self,
-	ViewEntry *ve, size_t common_prefix, int width, chtype attrs)
-{
-	RowBuffer buf = row_buffer_make (self);
-	row_buffer_append_length (&buf, ve->word, common_prefix, attrs | A_BOLD);
-	row_buffer_append (&buf, ve->word + common_prefix, attrs);
-	row_buffer_finish (&buf, width, attrs);
-}
-
-static void
-app_draw_definition (Application *self,
-	ViewEntry *ve, guint index, int width, chtype attrs)
-{
-	RowBuffer buf = row_buffer_make (self);
-	row_buffer_append (&buf, " ", attrs);
-	row_buffer_append_with_formatting (&buf,
-		g_ptr_array_index (ve->definitions, index),
-		g_ptr_array_index (ve->formatting, index), attrs);
-	row_buffer_finish (&buf, width, attrs);
-}
-
 /// Redraw the dictionary view.
 static void
 app_redraw_view (Application *self)
@@ -1155,37 +1131,56 @@ app_redraw_view (Application *self)
 	move (TOP_BAR_CUTOFF, 0);
 	clrtobot ();
 
+	// TODO: clean this stuff up a bit, it's all rather ugly
+	gchar *input_utf8 = g_ucs4_to_utf8
+		((gunichar *) self->input->data, -1, NULL, NULL, NULL);
 	gint left_width = app_get_left_column_width (self);
-	gchar *input_utf8 = g_ucs4_to_utf8 ((gunichar *) self->input->data, -1,
-		NULL, NULL, NULL);
 
-	guint i, offset = self->top_offset, shown = 0;
+	guint i, k = self->top_offset, shown = 0;
 	for (i = 0; i < self->entries->len; i++)
 	{
 		ViewEntry *ve = g_ptr_array_index (self->entries, i);
-		size_t common_prefix = 0;
+		for (; k < ve->definitions->len; k++)
+		{
+			chtype attrs = ((self->top_position + i) & 1)
+				? APP_ATTR (ODD) : APP_ATTR (EVEN);
+
+			if (shown == self->selected)
+				row_buffer_merge_attributes (&attrs, self->focused
+					? APP_ATTR (SELECTION) : APP_ATTR (DEFOCUSED));
+
+			gboolean last = k + 1 == ve->definitions->len;
+			if (last && self->underline_last)
+				attrs |= A_UNDERLINE;
+
+			RowBuffer buf;
+			row_buffer_init (&buf, self);
+
+			size_t common = 0;
 			if (self->hl_prefix)
 			{
-			common_prefix = stardict_longest_common_collation_prefix
+				common = stardict_longest_common_collation_prefix
 					(self->dict, ve->word, input_utf8);
-		}
-		chtype ve_attrs = APP_ATTR_IF ((self->top_position + i) & 1, ODD, EVEN);
-		for (; offset < ve->definitions->len; offset++)
-		{
-			chtype attrs = ve_attrs;
-			if ((offset + 1 == ve->definitions->len) && self->underline_last)
-				attrs |= A_UNDERLINE;
-			if (shown == self->selected)
-				row_buffer_merge_attributes (&attrs,
-					APP_ATTR_IF (self->focused, SELECTION, DEFOCUSED));
 
-			app_draw_word (self, ve, common_prefix, left_width, attrs);
+				gchar *prefix = g_strndup (ve->word, common);
-			app_draw_definition (self, ve, offset, COLS - left_width, attrs);
+				row_buffer_append (&buf, prefix, attrs | A_BOLD);
+				g_free (prefix);
+			}
+			row_buffer_append (&buf, ve->word + common, attrs);
+			row_buffer_finish (&buf, left_width, attrs);
+
+			row_buffer_init (&buf, self);
+			row_buffer_append (&buf, " ", attrs);
+			row_buffer_append_with_formatting (&buf,
+				g_ptr_array_index (ve->definitions, k),
+				g_ptr_array_index (ve->formatting, k), attrs);
+			row_buffer_finish (&buf, COLS - left_width, attrs);
+
 			if ((gint) ++shown == LINES - TOP_BAR_CUTOFF)
 				goto done;
 		}
 
-		offset = 0;
+		k = 0;
 	}
 
 done:
@@ -2415,7 +2410,8 @@ log_handler_curses (Application *self, const gchar *message)
 	in_processing = TRUE;
 	SAVE_CURSOR
 
-	RowBuffer buf = row_buffer_make (self);
+	RowBuffer buf;
+	row_buffer_init (&buf, self);
 	row_buffer_append (&buf, message, A_REVERSE);
 	move (0, 0);
 	row_buffer_finish (&buf, COLS, A_REVERSE);