README.org: fzf example: preview window make bigger

Was 50%:50% and by UX we do not need the half of the screen for the dictionary name.

.github/workflows/main.yml

@@ -20,14 +20,12 @@ jobs:
       fail-fast: true
 
       matrix:
-        os: [ubuntu-latest]
+        os: [ubuntu-22.04, ubuntu-latest]
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
+      - uses: jwlawson/actions-setup-cmake@v2
         with:
-          submodules: 'recursive'
+          cmake-version: '3.10'
-      - uses: jwlawson/actions-setup-cmake@v1.4
-        with:
-          cmake-version: '3.5.1'
           github-api-token: ${{ secrets.GITHUB_TOKEN }}
       - name: Check versions
         run: |
@@ -36,6 +34,10 @@ jobs:
           gcc --version
           echo "end of versions checking"
         shell: bash
+      - uses: awalsh128/cache-apt-pkgs-action@v1
+        with:
+          packages: libglib2.0-dev
+          version: 1.0
       - name: Run tests
         run: |
           set -e

CMakeLists.txt

@@ -1,7 +1,7 @@
-project(sdcv)
+cmake_minimum_required(VERSION 3.10 FATAL_ERROR)
+cmake_policy(VERSION 3.10)
 
-cmake_minimum_required(VERSION 3.5 FATAL_ERROR)
+project(sdcv)
-cmake_policy(VERSION 3.5)
 
 set(CMAKE_CXX_STANDARD 11)
 set(CMAKE_CXX_STANDARD_REQUIRED True)
@@ -27,7 +27,8 @@ if (WITH_READLINE)
   find_path(READLINE_INCLUDE_DIR readline/readline.h)
   find_library(READLINE_LIBRARY NAMES readline)
   if (NOT (READLINE_INCLUDE_DIR AND READLINE_LIBRARY))
-    set(WITH_READLINE False CACHE FORCE)
+    message(STATUS "readline library not FOUND, disable it's usage")
+    set(WITH_READLINE False CACHE BOOL "Use readline library" FORCE)
   endif ()
 endif (WITH_READLINE)
 
@@ -37,16 +38,16 @@ set(sdcv_SRCS
   src/sdcv.cpp
   src/readline.cpp
   src/readline.hpp
-  src/libwrapper.cpp 
+  src/libwrapper.cpp
   src/libwrapper.hpp
-  src/utils.cpp 
+  src/utils.cpp
   src/utils.hpp
 
   src/stardict_lib.cpp
   src/stardict_lib.hpp
   src/dictziplib.cpp
-  src/dictziplib.hpp  
+  src/dictziplib.hpp
-  src/distance.cpp 
+  src/distance.cpp
   src/distance.hpp
   src/mapfile.hpp
 )
@@ -82,10 +83,13 @@ configure_file(${CMAKE_CURRENT_SOURCE_DIR}/config.h.cmake
 include_directories(
 	${ZLIB_INCLUDE_DIR}
 	${GLIB2_INCLUDE_DIRS}
-	${READLINE_INCLUDE_DIR}
 	${CMAKE_CURRENT_SOURCE_DIR}/src/lib
 	${CMAKE_CURRENT_BINARY_DIR}
-)
+      )
+
+if (WITH_READLINE)
+  include_directories(${READLINE_INCLUDE_DIR})
+endif()
 
 #
 # Packing stuff
@@ -95,7 +99,7 @@ set(CPACK_PACKAGE_VENDOR "Evgeniy Dushistov <dushistov@mail.ru>")
 set(CPACK_PACKAGE_DESCRIPTION_FILE "${CMAKE_CURRENT_SOURCE_DIR}/README.org")
 set(CPACK_PACKAGE_VERSION_MAJOR "0")
 set(CPACK_PACKAGE_VERSION_MINOR "5")
-set(CPACK_PACKAGE_VERSION_PATCH "4")
+set(CPACK_PACKAGE_VERSION_PATCH "5")
 
 set(sdcv_VERSION
 	"${CPACK_PACKAGE_VERSION_MAJOR}.${CPACK_PACKAGE_VERSION_MINOR}.${CPACK_PACKAGE_VERSION_PATCH}")
@@ -107,8 +111,10 @@ add_executable(sdcv ${sdcv_SRCS})
 target_link_libraries(sdcv
   ${GLIB2_LIBRARIES}
   ${ZLIB_LIBRARIES}
-  ${READLINE_LIBRARY}
 )
+if (WITH_READLINE)
+  target_link_libraries(sdcv ${READLINE_LIBRARY})
+endif()
 if (ENABLE_NLS)
   set_directory_properties(PROPERTIES ADDITIONAL_MAKE_CLEAN_FILES "locale")
 endif ()

NEWS

@@ -1,3 +1,13 @@
+Version 0.5.5
+    - Avoid crashes when passing unknown dicts to the -u flag (by NiLuJe)
+    - Use off_t for stuff mainly assigned to a stat.st_size value
+Version 0.5.4
+    - Use binary search for synonyms
+    - Various improvments in work with synonyms
+    - Added --json (same as --json-output) to match man
+    - Show all matched result
+    - More robust parsing of ifo file
+    - Prevent crash if file size of files not matched expecting one for .oft files
 Version 0.5.3
   - Use single quotes around JSON data to reduce need for escaping
   - Store integer magic in cache file
@@ -36,7 +46,7 @@ Version 0.4.2
 * Russian translation update
 
 Version 0.4.1
-* Recreate cache if idx file was modified 
+* Recreate cache if idx file was modified
 * Abbility to use pager(SDCV_PAGER)
 * Add Chinese (traditional)  translation
 * Add  Ukrainian translation

README.org

@@ -1,6 +1,9 @@
 #+OPTIONS: ^:nil
 [[https://github.com/Dushistov/sdcv/actions?query=workflow%3ACI+branch%3Amaster][https://github.com/Dushistov/sdcv/workflows/CI/badge.svg]]
 [[https://github.com/Dushistov/sdcv/blob/master/LICENSE][https://img.shields.io/badge/license-GPL%202-brightgreen.svg]]
+
+* sdcv
+*sdcv* is a simple, cross-platform, text-based utility for working with dictionaries in [[http://stardict-4.sourceforge.net/][StarDict]] format.
 * How to compile and install
 #+BEGIN_SRC sh
 mkdir /tmp/build-sdcv
@@ -26,6 +29,26 @@ To report bugs use https://github.com/Dushistov/sdcv/issues ,
 if it is not possible you can report it via email to dushistov at mail dot ru.
 Be sure to include the word "sdcv" somewhere in the "Subject:" field.
 
+* Integration with [[https://github.com/junegunn/fzf][fzf]]
+Useful when you have multiple dictionaries
+#+BEGIN_SRC sh
+fzf --prompt="Dict: " \
+    --phony \
+    --bind "enter:reload(sdcv {q} -n --json | jq '.[].dict' -r)" \
+    --preview "sdcv {q} -en --use-dict={}" \
+    --preview-window=right:70%:wrap \
+   < <(echo)
+#+END_SRC
+
+* Integration with readline
+This lines can be added to inputrc file (~/.inputrc, /etc/inputrc),
+to abort multiply usage with ESC:
+#+begin_src
+$if sdcv
+    "\e\e": "-1\n"
+$endif
+#+end_src
+
 * Notes to developer
 ** make source code release
 #+BEGIN_SRC sh

po/ka.po

@@ -0,0 +1,147 @@
+# Georgian translation for sdcv.
+# Copyright (C) 2025 sdcv authors
+# This file is distributed under the same license as the sdcv project.
+# Temuri Doghonadze <temuri.doghonadze@gmail.com>, 2025.
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: sdcv 0.5\n"
+"Report-Msgid-Bugs-To: dushistov@mail.ru\n"
+"POT-Creation-Date: 2020-08-14 12:58+0300\n"
+"PO-Revision-Date: 2025-01-26 03:17+0100\n"
+"Last-Translator: Temuri Doghonadze <temuri.doghonadze@gmail.com>\n"
+"Language-Team: Georgian <(nothing)>\n"
+"Language: ka\n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: 8bit\n"
+"X-Generator: Poedit 3.5\n"
+
+#: ../src/libwrapper.cpp:300
+msgid "popen failed"
+msgstr "popen ჩავარდა"
+
+#: ../src/libwrapper.cpp:341
+#, c-format
+msgid "Can not convert %s to utf8.\n"
+msgstr "%s-ის utf8-ში გადაყვანა შეუძლებელია.\n"
+
+#: ../src/libwrapper.cpp:399 ../src/libwrapper.cpp:433
+#, c-format
+msgid "Found %zu items, similar to %s.\n"
+msgstr "აღმოჩენილია %zu ელემენტი, რომელიც %s-ს ჰგავს.\n"
+
+#: ../src/libwrapper.cpp:417
+msgid "Your choice[-1 to abort]: "
+msgstr "თქვენი არჩევანი[-1 გასაუქმებლად]: "
+
+#: ../src/libwrapper.cpp:427
+#, c-format
+msgid ""
+"Invalid choice.\n"
+"It must be from 0 to %zu or -1.\n"
+msgstr ""
+"არასწორი არჩევანი.\n"
+"უნდა იყოს 0-დან %zu-მდე, ან -1.\n"
+
+#: ../src/libwrapper.cpp:446
+#, c-format
+msgid "Nothing similar to %s, sorry :(\n"
+msgstr "%s-ს არაფერი ჰგავს :(\n"
+
+#: ../src/sdcv.cpp:89
+msgid "display version information and exit"
+msgstr "ვერსიის ჩვენება და გასვლა"
+
+#: ../src/sdcv.cpp:91
+msgid "display list of available dictionaries and exit"
+msgstr "ხელმისაწვდომი ლექსიკონების ჩვენება და გასვლა"
+
+#: ../src/sdcv.cpp:93
+msgid "for search use only dictionary with this bookname"
+msgstr "ძებნისთვის, მხოლოდ, ამ სახელის მქონე ლექსიკონის გამოყენება"
+
+#: ../src/sdcv.cpp:94
+msgid "bookname"
+msgstr "ლექსიკონის_სახელი"
+
+#: ../src/sdcv.cpp:96
+msgid "for use in scripts"
+msgstr "სკრიპტებში გამოსაყენებლად"
+
+#: ../src/sdcv.cpp:98
+msgid "print the result formatted as JSON"
+msgstr "შედეგების JSON ფორმატში გამოტანა"
+
+#: ../src/sdcv.cpp:100
+msgid "do not fuzzy-search for similar words, only return exact matches"
+msgstr ""
+"მსგავსი სიტყვებისთვის არაზუსტი ძებნა გამოყენებული არ იქნება. "
+"დაბრუნდება, მხოლოდ, ზუსტი დამთხვევები"
+
+#: ../src/sdcv.cpp:102
+msgid "output must be in utf8"
+msgstr "გამოტანა utf8-ში უნდა იყოს"
+
+#: ../src/sdcv.cpp:104
+msgid "input of sdcv in utf8"
+msgstr "პროგრამაში შეყვანა utf8 -ში"
+
+#: ../src/sdcv.cpp:106
+msgid "use this directory as path to stardict data directory"
+msgstr ""
+"ამ საქაღალდის გამოყენება stardict-ის მონაცემების საქაღალდის ბილიკად"
+
+#: ../src/sdcv.cpp:107
+msgid "path/to/dir"
+msgstr "ბილიკი/სასურველ/საქაღალდემდე"
+
+#: ../src/sdcv.cpp:109
+msgid ""
+"only use the dictionaries in data-dir, do not search in user and system "
+"directories"
+msgstr ""
+"ლექსიკონების, მხოლოდ, data-dir-დან გამოყენება. მომხმარებლის და სისტემურ "
+"საქაღალდეებში ძებნა არ მოხდება"
+
+#: ../src/sdcv.cpp:111
+msgid "colorize the output"
+msgstr "ფერებში გამოტანა"
+
+#: ../src/sdcv.cpp:116
+msgid " words"
+msgstr " სიტყვა"
+
+#: ../src/sdcv.cpp:122
+#, c-format
+msgid "Invalid command line arguments: %s\n"
+msgstr "არასწორი ბრძანების სტრიქონის არგუმენტები: %s\n"
+
+#: ../src/sdcv.cpp:128
+#, c-format
+msgid "Console version of Stardict, version %s\n"
+msgstr "Startdict-ის კონსოლის ვერსია. ვერსია %s\n"
+
+#: ../src/sdcv.cpp:206
+#, c-format
+msgid "g_mkdir failed: %s\n"
+msgstr "g_mkdir ჩავარდა: %s\n"
+
+#: ../src/sdcv.cpp:222
+msgid "Enter word or phrase: "
+msgstr "შეიყვანეთ სიტყვა ან ფრაზა: "
+
+#: ../src/sdcv.cpp:230
+#, c-format
+msgid "There are no words/phrases to translate.\n"
+msgstr "სათარგმნი სიტყვების/ფრაზების გარეშე.\n"
+
+#: ../src/sdcv.cpp:242
+#, c-format
+msgid "Dictionary's name   Word count\n"
+msgstr "ლექსიკონის სახელი     სიტყვების რაოდენობა\n"
+
+#: ../src/utils.cpp:48
+#, c-format
+msgid "Can not convert %s to current locale.\n"
+msgstr "ვერ გადავიყვანე %s მიმდინარე ლოკალში.\n"

src/dictziplib.hpp

@@ -27,7 +27,7 @@ public:
 private:
     const char *start; /* start of mmap'd area */
     const char *end; /* end of mmap'd area */
-    unsigned long size; /* size of mmap */
+    off_t size; /* size of mmap */
 
     int type;
     z_stream zStream;
@@ -47,7 +47,7 @@ private:
     std::string origFilename;
     std::string comment;
     unsigned long crc;
-    unsigned long length;
+    off_t length;
     unsigned long compressedLength;
     DictCache cache[DICT_CACHE_SIZE];
     MapFile mapfile;

src/libwrapper.cpp

@@ -23,6 +23,7 @@
 #endif
 
 #include <cstring>
+#include <cstdio> //for popen
 #include <map>
 #include <memory>
 
@@ -415,10 +416,9 @@ search_result Library::process_phrase(const char *loc_str, IReadLine &io, bool f
                        colorize_output_ ? ESC_END : "");
             }
             int choise;
-            std::unique_ptr<IReadLine> choice_readline(create_readline_object());
             for (;;) {
                 std::string str_choise;
-                choice_readline->read(_("Your choice[-1 to abort]: "), str_choise);
+                io.read(_("Your choice[-1 to abort]: "), str_choise);
                 sscanf(str_choise.c_str(), "%d", &choise);
                 if (choise >= 0 && choise < int(res_list.size())) {
                     sdcv_pager pager;

src/mapfile.hpp

@@ -22,13 +22,13 @@ public:
     ~MapFile();
     MapFile(const MapFile &) = delete;
     MapFile &operator=(const MapFile &) = delete;
-    bool open(const char *file_name, unsigned long file_size);
+    bool open(const char *file_name, off_t file_size);
     gchar *begin() { return data; }
 
 private:
     char *data = nullptr;
-    unsigned long size = 0ul;
 #ifdef HAVE_MMAP
+    size_t size = 0u;
     int mmap_fd = -1;
 #elif defined(_WIN32)
     HANDLE hFile = 0;
@@ -36,9 +36,8 @@ private:
 #endif
 };
 
-inline bool MapFile::open(const char *file_name, unsigned long file_size)
+inline bool MapFile::open(const char *file_name, off_t file_size)
 {
-    size = file_size;
 #ifdef HAVE_MMAP
     if ((mmap_fd = ::open(file_name, O_RDONLY)) < 0) {
         // g_print("Open file %s failed!\n",fullfilename);
@@ -46,14 +45,16 @@ inline bool MapFile::open(const char *file_name, unsigned long file_size)
     }
     struct stat st;
     if (fstat(mmap_fd, &st) == -1 || st.st_size < 0 || (st.st_size == 0 && S_ISREG(st.st_mode))
-        || sizeof(st.st_size) > sizeof(file_size) || static_cast<unsigned long>(st.st_size) != file_size) {
+        || st.st_size != file_size) {
         close(mmap_fd);
         return false;
     }
 
-    data = (gchar *)mmap(nullptr, file_size, PROT_READ, MAP_SHARED, mmap_fd, 0);
+    size = static_cast<size_t>(st.st_size);
+    data = (gchar *)mmap(nullptr, size, PROT_READ, MAP_SHARED, mmap_fd, 0);
     if ((void *)data == (void *)(-1)) {
         // g_print("mmap file %s failed!\n",idxfilename);
+        size = 0u;
         data = nullptr;
         return false;
     }

src/sdcv.cpp

@@ -186,10 +186,13 @@ try {
         }
 
         // add bookname to list
-        gchar **p = get_impl(use_dict_list);
+        for (gchar **p = get_impl(use_dict_list); *p != nullptr; ++p) {
-        while (*p) {
+            auto it = bookname_to_ifo.find(*p);
-            order_list.push_back(bookname_to_ifo.at(*p));
+            if (it != bookname_to_ifo.end()) {
-            ++p;
+                order_list.push_back(it->second);
+            } else {
+                fprintf(stderr, _("Unknown dictionary: %s\n"), *p);
+            }
         }
     } else {
         std::string ordering_cfg_file = std::string(g_get_user_config_dir()) + G_DIR_SEPARATOR_S "sdcv_ordering";
@@ -201,7 +204,12 @@ try {
         if (ordering_file != nullptr) {
             std::string line;
             while (stdio_getline(ordering_file, line)) {
-                order_list.push_back(bookname_to_ifo.at(line));
+                auto it = bookname_to_ifo.find(line);
+                if (it != bookname_to_ifo.end()) {
+                    order_list.push_back(it->second);
+                } else {
+                    fprintf(stderr, _("Unknown dictionary: %s\n"), line.c_str());
+                }
             }
             fclose(ordering_file);
         }

src/stardict_lib.cpp

@@ -429,7 +429,7 @@ public:
         if (idxfile)
             fclose(idxfile);
     }
-    bool load(const std::string &url, gulong wc, gulong fsize, bool verbose) override;
+    bool load(const std::string &url, gulong wc, off_t fsize, bool verbose) override;
     const gchar *get_key(glong idx) override;
     void get_data(glong idx) override { get_key(idx); }
     const gchar *get_key_and_data(glong idx) override
@@ -489,7 +489,7 @@ public:
     {
     }
     ~WordListIndex() { g_free(idxdatabuf); }
-    bool load(const std::string &url, gulong wc, gulong fsize, bool verbose) override;
+    bool load(const std::string &url, gulong wc, off_t fsize, bool verbose) override;
     const gchar *get_key(glong idx) override { return wordlist[idx]; }
     void get_data(glong idx) override;
     const gchar *get_key_and_data(glong idx) override
@@ -615,7 +615,7 @@ bool OffsetIndex::save_cache(const std::string &url, bool verbose)
     return false;
 }
 
-bool OffsetIndex::load(const std::string &url, gulong wc, gulong fsize, bool verbose)
+bool OffsetIndex::load(const std::string &url, gulong wc, off_t fsize, bool verbose)
 {
     wordcount = wc;
     gulong npages = (wc - 1) / ENTR_PER_PAGE + 2;
@@ -758,7 +758,7 @@ bool OffsetIndex::lookup(const char *str, std::set<glong> &idxs, glong &next_idx
     return bFound;
 }
 
-bool WordListIndex::load(const std::string &url, gulong wc, gulong fsize, bool)
+bool WordListIndex::load(const std::string &url, gulong wc, off_t fsize, bool)
 {
     gzFile in = gzopen(url.c_str(), "rb");
     if (in == nullptr)
@@ -771,7 +771,7 @@ bool WordListIndex::load(const std::string &url, gulong wc, gulong fsize, bool)
     if (len < 0)
         return false;
 
-    if (gulong(len) != fsize)
+    if (static_cast<off_t>(len) != fsize)
         return false;
 
     wordlist.resize(wc + 1);
@@ -920,7 +920,7 @@ bool Dict::Lookup(const char *str, std::set<glong> &idxs, glong &next_idx)
 
 bool Dict::load(const std::string &ifofilename, bool verbose)
 {
-    gulong idxfilesize;
+    off_t idxfilesize;
     if (!load_ifofile(ifofilename, idxfilesize))
         return false;
 
@@ -964,7 +964,7 @@ bool Dict::load(const std::string &ifofilename, bool verbose)
     return true;
 }
 
-bool Dict::load_ifofile(const std::string &ifofilename, gulong &idxfilesize)
+bool Dict::load_ifofile(const std::string &ifofilename, off_t &idxfilesize)
 {
     DictInfo dict_info;
     if (!dict_info.load_from_ifo_file(ifofilename, false))
@@ -989,7 +989,7 @@ bool Dict::LookupWithRule(GPatternSpec *pspec, glong *aIndex, int iBuffLen)
     int iIndexCount = 0;
 
     for (guint32 i = 0; i < narticles() && iIndexCount < (iBuffLen - 1); i++)
-        if (g_pattern_match_string(pspec, get_key(i)))
+        if (g_pattern_spec_match_string(pspec, get_key(i)))
             aIndex[iIndexCount++] = i;
 
     aIndex[iIndexCount] = -1; // -1 is the end.
@@ -1047,12 +1047,12 @@ bool Libs::LookupSimilarWord(const gchar *sWord, std::set<glong> &iWordIndices,
         }
         // Upper the first character and lower others.
         if (!bFound) {
-            gchar *nextchar = g_utf8_next_char(sWord);
+            const gchar *rest = g_utf8_next_char(sWord);
-            gchar *firstchar = g_utf8_strup(sWord, nextchar - sWord);
+            gchar *firstchar = g_utf8_strup(sWord, rest - sWord);
-            nextchar = g_utf8_strdown(nextchar, -1);
+            gchar *rest_lowercase = g_utf8_strdown(rest, -1);
-            casestr = g_strdup_printf("%s%s", firstchar, nextchar);
+            casestr =  g_strconcat(firstchar, rest_lowercase, nullptr);
+            g_free(rest_lowercase);
             g_free(firstchar);
-            g_free(nextchar);
             if (strcmp(casestr, sWord)) {
                 if (oLib[iLib]->Lookup(casestr, iWordIndices))
                     bFound = true;

src/stardict_lib.hpp

@@ -77,8 +77,8 @@ struct DictInfo {
     std::string website;
     std::string date;
     std::string description;
-    guint32 index_file_size;
+    off_t index_file_size;
-    guint32 syn_file_size;
+    off_t syn_file_size;
     std::string sametypesequence;
 
     bool load_from_ifo_file(const std::string &ifofilename, bool istreedict);
@@ -91,7 +91,7 @@ public:
     guint32 wordentry_size;
 
     virtual ~IIndexFile() {}
-    virtual bool load(const std::string &url, gulong wc, gulong fsize, bool verbose) = 0;
+    virtual bool load(const std::string &url, gulong wc, off_t fsize, bool verbose) = 0;
     virtual const gchar *get_key(glong idx) = 0;
     virtual void get_data(glong idx) = 0;
     virtual const gchar *get_key_and_data(glong idx) = 0;
@@ -160,7 +160,7 @@ private:
     std::unique_ptr<IIndexFile> idx_file;
     std::unique_ptr<SynFile> syn_file;
 
-    bool load_ifofile(const std::string &ifofilename, gulong &idxfilesize);
+    bool load_ifofile(const std::string &ifofilename, off_t &idxfilesize);
 };
 
 class Libs