From d1934a7cfd99abbf9502aef6e6ecd897f78aa026 Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Fri, 15 Jun 2007 13:25:35 +0200 Subject: [PATCH] Using nl_langinfo(CODESET) to determine the local codeset to use; libsi uses canonical codeset names --- CONTRIBUTORS | 2 ++ HISTORY | 3 ++ libsi/si.c | 94 ++++++++++++++++++++++++++-------------------------- tools.c | 4 +-- vdr.c | 27 +++++++++------ 5 files changed, 70 insertions(+), 60 deletions(-) diff --git a/CONTRIBUTORS b/CONTRIBUTORS index 985055ba..396c8775 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -886,6 +886,7 @@ Ludwig Nussel for removing the LOCK_THREAD from the LIRC thread for making the Makefile patch friendlier for a patch that was used for implementing setting the user id + for pointing out that the canonical spelling of codesets is with '-' Thomas Koch for his support in keeping the Premiere World channels up to date in 'channels.conf' @@ -1739,6 +1740,7 @@ Thomas G for suggesting to extend the version number reported with the '-V' option to also show the current APIVERSION for fixing i18n characters for the Hungarian texts + for implementing using nl_langinfo(CODESET) to determine the local codeset to use David Woodhouse for his help in replacing the get/put_unaligned() macros from asm/unaligned.h with diff --git a/HISTORY b/HISTORY index 3bbec1b1..f64279f9 100644 --- a/HISTORY +++ b/HISTORY @@ -5235,3 +5235,6 @@ Video Disk Recorder Revision History - Increased APIVERSION (forgot to do that in 1.5.2 and 1.5.3). - Fixed a crash in i18n character set conversion (thanks to Alexander Riedel and Christian Wieninger for patches that cured part of the problem). +- Using nl_langinfo(CODESET) to determine the local codeset to use (thanks to + Thomas Günther). The codeset names in 'libsi/si.c' have been changed to the + canonical spelling with '-' (thanks to Ludwig Nussel for pointing this out). diff --git a/libsi/si.c b/libsi/si.c index 2eb3dfd6..88950908 100644 --- a/libsi/si.c +++ b/libsi/si.c @@ -6,7 +6,7 @@ * the Free Software Foundation; either version 2 of the License, or * * (at your option) any later version. * * * - * $Id: si.c 1.20 2007/06/10 09:31:34 kls Exp $ + * $Id: si.c 1.21 2007/06/15 13:08:31 kls Exp $ * * ***************************************************************************/ @@ -250,59 +250,59 @@ char *String::getText(char *buffer, char *shortVersion, int sizeBuffer, int size } static const char *CharacterTables1[] = { - NULL, // 0x00 - "ISO8859-5", // 0x01 - "ISO8859-6", // 0x02 - "ISO8859-7", // 0x03 - "ISO8859-8", // 0x04 - "ISO8859-9", // 0x05 - "ISO8859-10", // 0x06 - "ISO8859-11", // 0x07 - "ISO8859-12", // 0x08 - "ISO8859-13", // 0x09 - "ISO8859-14", // 0x0A - "ISO8859-15", // 0x0B - NULL, // 0x0C - NULL, // 0x0D - NULL, // 0x0E - NULL, // 0x0F - NULL, // 0x10 - "UTF16", // 0x11 - "EUC-KR", // 0x12 - "GB2312", // 0x13 - "GBK", // 0x14 - "UTF8", // 0x15 - NULL, // 0x16 - NULL, // 0x17 - NULL, // 0x18 - NULL, // 0x19 - NULL, // 0x1A - NULL, // 0x1B - NULL, // 0x1C - NULL, // 0x1D - NULL, // 0x1E - NULL, // 0x1F + NULL, // 0x00 + "ISO-8859-5", // 0x01 + "ISO-8859-6", // 0x02 + "ISO-8859-7", // 0x03 + "ISO-8859-8", // 0x04 + "ISO-8859-9", // 0x05 + "ISO-8859-10", // 0x06 + "ISO-8859-11", // 0x07 + "ISO-8859-12", // 0x08 + "ISO-8859-13", // 0x09 + "ISO-8859-14", // 0x0A + "ISO-8859-15", // 0x0B + NULL, // 0x0C + NULL, // 0x0D + NULL, // 0x0E + NULL, // 0x0F + NULL, // 0x10 + "UTF-16", // 0x11 + "EUC-KR", // 0x12 + "GB2312", // 0x13 + "GBK", // 0x14 + "UTF-8", // 0x15 + NULL, // 0x16 + NULL, // 0x17 + NULL, // 0x18 + NULL, // 0x19 + NULL, // 0x1A + NULL, // 0x1B + NULL, // 0x1C + NULL, // 0x1D + NULL, // 0x1E + NULL, // 0x1F }; #define SingleByteLimit 0x0B static const char *CharacterTables2[] = { NULL, // 0x00 - "ISO8859-1", // 0x01 - "ISO8859-2", // 0x02 - "ISO8859-3", // 0x03 - "ISO8859-4", // 0x04 - "ISO8859-5", // 0x05 - "ISO8859-6", // 0x06 - "ISO8859-7", // 0x07 - "ISO8859-8", // 0x08 - "ISO8859-9", // 0x09 - "ISO8859-10", // 0x0A - "ISO8859-11", // 0x0B + "ISO-8859-1", // 0x01 + "ISO-8859-2", // 0x02 + "ISO-8859-3", // 0x03 + "ISO-8859-4", // 0x04 + "ISO-8859-5", // 0x05 + "ISO-8859-6", // 0x06 + "ISO-8859-7", // 0x07 + "ISO-8859-8", // 0x08 + "ISO-8859-9", // 0x09 + "ISO-8859-10", // 0x0A + "ISO-8859-11", // 0x0B NULL, // 0x0C - "ISO8859-13", // 0x0D - "ISO8859-14", // 0x0E - "ISO8859-15", // 0x0F + "ISO-8859-13", // 0x0D + "ISO-8859-14", // 0x0E + "ISO-8859-15", // 0x0F }; #define NumEntries(Table) (sizeof(Table) / sizeof(char *)) diff --git a/tools.c b/tools.c index 465335ae..cec2555a 100644 --- a/tools.c +++ b/tools.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: tools.c 1.125 2007/06/15 12:46:38 kls Exp $ + * $Id: tools.c 1.126 2007/06/15 13:11:12 kls Exp $ */ #include "tools.h" @@ -732,7 +732,7 @@ void cCharSetConv::SetSystemCharacterTable(const char *CharacterTable) { free(systemCharacterTable); systemCharacterTable = NULL; - if (!strcasestr(CharacterTable, "UTF")) { + if (!strcasestr(CharacterTable, "UTF-8")) { // Set up a map for the character values 128...255: char buf[129]; for (int i = 0; i < 128; i++) diff --git a/vdr.c b/vdr.c index f7ff1b58..f3218eb2 100644 --- a/vdr.c +++ b/vdr.c @@ -22,11 +22,12 @@ * * The project's page is at http://www.cadsoft.de/vdr * - * $Id: vdr.c 1.291 2007/06/09 12:33:53 kls Exp $ + * $Id: vdr.c 1.292 2007/06/15 13:20:34 kls Exp $ */ #include #include +#include #include #include #include @@ -493,18 +494,22 @@ int main(int argc, char *argv[]) // Set the system character table: - char *LangEnv = setlocale(LC_CTYPE, ""); - if (!LangEnv) - LangEnv = getenv("LANG"); // last resort in case locale stuff isn't installed - if (LangEnv) { - char *CodeSet = strchr(LangEnv, '.'); - if (CodeSet) { - CodeSet++; // skip the dot - bool known = SI::SetSystemCharacterTable(CodeSet); - isyslog("codeset is '%s' - %s", CodeSet, known ? "known" : "unknown"); - cCharSetConv::SetSystemCharacterTable(CodeSet); + char *CodeSet = NULL; + if (setlocale(LC_CTYPE, "")) + CodeSet = nl_langinfo(CODESET); + else { + char *LangEnv = getenv("LANG"); // last resort in case locale stuff isn't installed + if (LangEnv) { + CodeSet = strchr(LangEnv, '.'); + if (CodeSet) + CodeSet++; // skip the dot } } + if (CodeSet) { + bool known = SI::SetSystemCharacterTable(CodeSet); + isyslog("codeset is '%s' - %s", CodeSet, known ? "known" : "unknown"); + cCharSetConv::SetSystemCharacterTable(CodeSet); + } // Main program loop variables - need to be here to have them initialized before any EXIT():