[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#127005: package is broken on arm, powerpc and s390 due to wrong assumption on char signedness



On Tue, 1 Jan 2002, Gerhard Tonn wrote:

> > These three are not necessarily bugs, though the code should be looked
> > over to be certain.
> >
>
> I think these are bugs. See code fragments below.
>
>
>  if ((succ_index=T1_GetEncodingIndex( FontID, (char*) ligs->succ))==-1) {
>       /* successor is not current encoding */
>       ligs=ligs->next;
>       continue;
>     }
>
> if ((lig_index=T1_GetEncodingIndex( FontID, (char*) ligs->lig))==-1) {
>       /* Specified ligature is not in current encoding */
>       ligs=ligs->next;
>       continue;
>     }
>...

Some comments on this:

- These fragments are in libs/t1lib/t1finfo.c which is a local (upstream)
  copy of the library in the t1lib package.

- succ_index and lig_index are int instead of char in t1lib-1.3.1 (not yet
  packaged for Debian):

--- /home/bunk/Debian/tetex/tetex-bin-1.0.7+20011202/libs/t1lib/t1finfo.c	Sun Oct 14 20:14:23 2001
+++ lib/t1lib/t1finfo.c	Sun May 27 05:40:11 2001
@@ -510,9 +510,9 @@
   char *c_name;
   char cc_name[128];
   static char succ[MAX_LIGS];
-  char succ_index;
+  int succ_index;
   static char lig[MAX_LIGS];
-  char lig_index;
+  int lig_index;

   Ligature *ligs;
   int i,j;

- The only reason why we use this local copy is that building teTeX with
  "--with-system-t1lib" which would use the shared library from the t1lib1
  package seems to be currently broken (I've already reported this to the
  tetex-pretest list).


I've CC'ed David Huggins-Daines who maintains the Debian package of t1lib
and I'll forward it to the tetex-pretest list to get it fixed in the next
upstream teTeX package.


> Regards,
> Gerhard


cu
Adrian








Reply to: