[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: lynx-dev was: def7_uni problem
|
From: |
Leonid Pauzner |
|
Subject: |
Re: lynx-dev was: def7_uni problem |
|
Date: |
Fri, 23 Oct 1998 15:26:54 +0400 (MSD) |
>> 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.
> you're right - it makes 'i' equal to 256, which is not what was intended.
> (setting it to zero looks right).
the better way to move id_append[i+1] = '\0';
one line upper, before "}"...
>> if (this_isDefaultMap == -1) {
>> this_isDefaultMap = !strncmp(this_MIMEcharset,"iso-8859-1", 10);
>> }
>> fprintf(stderr,
>> "makeuctb: %s: %stranslation map",
>> this_MIMEcharset, (this_isDefaultMap ? "default " : ""));
>> if (this_isDefaultMap == 1) {
>> *id_append = '\0';
> i = 0;
>> } else {
>> for (i = 0, p = this_MIMEcharset;
>> *p && (i < UC_MAXLEN_ID_APPEND-1);
>> p++, i++) {
>> id_append[i+1] = isalnum(*p) ? *p : '_';
>> }
>> }
>> id_append[i+1] = '\0';
>> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> --
> Thomas E. Dickey
> address@hidden
> http://www.clark.net/pub/dickey
> _________________________________________________________________
> References:
> * lynx-dev was: def7_uni problem
> * From: "Leonid Pauzner" <address@hidden>
> _________________________________________________________________
> * Prev: Re: lynx-dev status
> * Next: Re: lynx-dev status
> * Index(es):
> + Main
> + Thread
> _________________________________________________________________
> Lynx mailing list archives
> [FLORA HOME] [LYNX Home]