fgetc() returns an int which should never ever be put in a char.
authorAlfons Hoogervorst <alfons@proteus.demon.nl>
Sun, 30 Dec 2001 23:14:31 +0000 (23:14 +0000)
committerAlfons Hoogervorst <alfons@proteus.demon.nl>
Sun, 30 Dec 2001 23:14:31 +0000 (23:14 +0000)
ChangeLog.claws
configure.in
src/ldif.c
src/mutt.c

index 58a93e0..2ac03b5 100644 (file)
@@ -1,3 +1,19 @@
+2001-12-31 [alfons]    0.6.6claws34
+       
+       * src/ldif.c
+       * src/mutt.c
+               fgetc() returns a signed integer which doesn't
+               fit in a char. so c truncates this to a char
+               but we may lose significant bits. i.e. we may
+               comparing 0xFF which may not be equal to -1
+               unless properly casted to signed char which
+               we won't do because fgetc() returns a signed
+               integer which doesn't fit in a char. so c 
+               truncates this to a char but we may lose 
+               significat bits. (you got the idea, thanks
+               to Gustavo Noronha Silva <kov@debian.org>
+               and the Debian folks!)
+
 2001-12-30 [alfons]    0.6.6claws33
 
        * src/prefs_common.c
index 1710550..5f20766 100644 (file)
@@ -8,7 +8,7 @@ MINOR_VERSION=6
 MICRO_VERSION=6
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=claws33
+EXTRA_VERSION=claws34
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$MICRO_VERSION$EXTRA_VERSION
 
 dnl
index e2fb8fc..9818503 100644 (file)
@@ -226,7 +226,7 @@ static void ldif_close_file( LdifFile *ldifFile ) {
 */
 static gchar *ldif_get_line( LdifFile *ldifFile ) {
        gchar buf[ LDIFBUFSIZE ];
-       gchar ch;
+       int ch;
        gchar *ptr;
 
        if( feof( ldifFile->file ) ) return NULL;
index c8f443b..d547c07 100644 (file)
@@ -142,7 +142,7 @@ static void mutt_close_file( MuttFile *muttFile ) {
 */
 static gchar *mutt_get_line( MuttFile *muttFile, gboolean *flagCont ) {
        gchar buf[ MUTTBUFSIZE ];
-       gchar ch, lch;
+       int ch, lch;
        gchar *ptr, *lptr;
 
        *flagCont = FALSE;