On Fri, Oct 23, 1998 at 11:27:36AM +0400, Leonid Pauzner wrote:
 >  Recently was discussed a problem compiling def7_uni.tbl by few people,
 >  I look into makeuctbl.c and found a code in question (see below):
 >  
 >  def7_uni use blank "id_append", but we have
 >      id_append[i+1] = '\0';
 >  outside "else" when "i" really uninitialized for default table.
 >  So this may (probably) give an unpredictable side effect.
 >  
Congratulations !

Well you really can call this *unpredictable*. E.g., in my case
        printf("%3d", unicount[i]);
was not executed. But duplicate it to stderr for debugging :
        printf("%3d", unicount[i]);
        fprintf(stderr, "%3d", unicount[i]);
along with the other printf statements, and you will end up with a broken
*file* def7_uni.h, but correct output on your *terminal*.

Anyway Leonid is right, and since
        *id_append = '\0';
for the default map, it should be save to move
        id_append[i+1] = '\0';
up into the else branch. This fixes the problem in my case.

Thanks a lot,

 - Serge

-- 
 Serge Munhoven                              Internet: address@hidden