[PATCH] sensors: print the degree sign in the terminal's current locale

Aurelien Jarno aurelien at aurel32.net
Sun Mar 21 14:38:22 CET 2004


On Sun, Mar 21, 2004 at 02:36:30AM +0100, Aurelien Jarno wrote:
> Hi all,
> 
> To print the degree sign, sensors lookups the current locale, outputs it
> in UTF-8 if the locale is in UTF-8, and in latin-1 otherwise. However
> some people are using other locales. Moreover, it is rather irritating
> for people using a Japanese locale, as the character code 176 is a start
> of sequence of characters for EUC terminals.

Please find attached a new patch, I hope less ugly than the previous
one. I made it using the advice Jean Delvare give me on IRC.

Aurelien

-- 
  .''`.  Aurelien Jarno	              GPG: 1024D/F1BCDB73
 : :' :  Debian GNU/Linux developer | Electrical Engineering Student 
 `. `'   aurel32 at debian.org         | aurelien at aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net
-------------- next part --------------
Index: prog/sensors/main.c
===================================================================
RCS file: /home/cvs/lm_sensors2/prog/sensors/main.c,v
retrieving revision 1.90
diff -u -1 -b -p -r1.90 main.c
--- lm_sensors2.orig/prog/sensors/main.c	29 Feb 2004 18:43:53 -0000	1.90
+++ lm_sensors2/prog/sensors/main.c	21 Mar 2004 12:15:33 -0000
@@ -26,2 +26,3 @@
 #include <langinfo.h>
+#include <iconv.h>
 
@@ -156,2 +157,25 @@ void close_config_file(void)
 
+static void set_degstr(void)
+{
+  /* Size hardcoded for better performance. Don't forget to count \0 !!! */
+  size_t deg_latin1_size = 3;
+  char *deg_latin1_text[2] = {"\260C", "\260F"};
+  const char *deg_default_text[2] = {" C", " F"};
+  size_t nconv;
+  size_t degstr_size = sizeof(degstr);
+  char *degstr_ptr = degstr;
+
+  iconv_t cd = iconv_open(nl_langinfo(CODESET), "ISO-8859-1");
+  if (cd != (iconv_t) -1) {
+    nconv = iconv(cd, &(deg_latin1_text[fahrenheit]), &deg_latin1_size, &degstr_ptr, &degstr_size);
+    iconv_close(cd);
+    
+    if (nconv > 0) 
+      return;	   
+  }
+
+  /* There was an error during the conversion, use the default text */
+  strcpy(degstr, deg_default_text[fahrenheit]);
+}
+
 int main (int argc, char *argv[])
@@ -255,11 +279,3 @@ int main (int argc, char *argv[])
   /* build the degrees string */
-  if (strcmp(nl_langinfo(CODESET), "UTF-8") == 0)
-    sprintf(degstr, "%c%c", 0xc2, 0xb0);
-  else
-    sprintf(degstr, "%c", 176);
-  if (fahrenheit) {
-    strcat(degstr, "F");
-  } else {
-    strcat(degstr, "C");
-  }
+  set_degstr();
 


More information about the lm-sensors mailing list