Changeset 12329


Ignore:
Timestamp:
01/02/10 23:24:49 (11 years ago)
Author:
ehuelsmann
Message:

More notes about potential codepoint conversion bugs.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/abcl/src/org/armedbear/lisp/Stream.java

    r12323 r12329  
    414414              if (n >= 0)
    415415                {
    416                   char c = (char) n;
     416                  char c = (char) n; // ### BUG: Codepoint conversion
    417417                  Readtable rt = (Readtable) Symbol.CURRENT_READTABLE.symbolValue(thread);
    418418                  if (!rt.isWhitespace(c))
     
    464464                  return eofValue;
    465465              }
    466             char c = (char) n;
     466            char c = (char) n; // ### BUG: Codepoint conversion
    467467            if (rt.isWhitespace(c))
    468468              continue;
     
    502502                if (n >= 0)
    503503                  {
    504                     char c = (char) n;
     504                    char c = (char) n; // ### BUG: Codepoint conversion
    505505                    Readtable rt = FaslReadtable.getInstance();
    506506                    if (!rt.isWhitespace(c))
     
    539539                    return eofValue;
    540540              }
    541             char c = (char) n;
     541            char c = (char) n; // ### BUG: Codepoint conversion
    542542            if (rt.isWhitespace(c))
    543543                continue;
     
    730730                if (n < 0)
    731731                    return error(new EndOfFile(this));
    732                 char nextChar = (char) n;
     732                char nextChar = (char) n; // ### BUG: Codepoint conversion
    733733                if (isTokenDelimiter(nextChar, rt))
    734734                  {
     
    814814            if (n < 0)
    815815                return error(new EndOfFile(this));
    816             c = (char) n;
     816            c = (char) n; // ### BUG: Codepoint conversion
    817817            if (c < '0' || c > '9')
    818818                break;
     
    860860        if (n < 0)
    861861            return error(new EndOfFile(this));
    862         char c = (char) n;
     862        char c = (char) n; // ### BUG: Codepoint conversion
    863863        FastStringBuffer sb = new FastStringBuffer(c);
    864864        while (true)
     
    867867            if (n < 0)
    868868                break;
    869             c = (char) n;
     869            c = (char) n; // ### BUG: Codepoint conversion
    870870            if (rt.isWhitespace(c))
    871871                break;
     
    884884        n = LispCharacter.nameToChar(token);
    885885        if (n >= 0)
    886             return LispCharacter.getInstance((char)n);
     886            return LispCharacter.getInstance((char)n); // ### BUG: Codepoint conversion
    887887        return error(new LispError("Unrecognized character name: \"" + token + '"'));
    888888      }
     
    10501050                return null;
    10511051              }
    1052             char c = (char) n;
     1052            char c = (char) n; // ### BUG: Codepoint conversion
    10531053            byte syntaxType = rt.getSyntaxType(c);
    10541054            if (syntaxType == Readtable.SYNTAX_TYPE_SINGLE_ESCAPE)
     
    10611061                    return null;
    10621062                  }
    1063                 sb.append((char)n);
     1063                sb.append((char)n); // ### BUG: Codepoint conversion
    10641064                continue;
    10651065              }
     
    12421242                return flags;
    12431243              }
    1244             sb.setCharAt(0, (char) n);
     1244            sb.setCharAt(0, (char) n); // ### BUG: Codepoint conversion
    12451245            flags = new BitSet(1);
    12461246            flags.set(0);
     
    12731273          if (n < 0)
    12741274              break;
    1275           char c = (char) n;
     1275          char c = (char) n; // ### BUG: Codepoint conversion
    12761276          if (rt.isWhitespace(c))
    12771277            {
     
    12911291              if (n < 0)
    12921292                  break;
    1293               sb.append((char)n);
     1293              sb.append((char)n); // ### BUG: Codepoint conversion
    12941294              if (flags == null)
    12951295                  flags = new BitSet(sb.length());
     
    16321632                return 0;
    16331633              }
    1634             char c = (char) n;
     1634            char c = (char) n; // ### BUG: Codepoint conversion
    16351635            if (!rt.isWhitespace(c))
    16361636                return c;
     
    16911691                return thread.setValues(new SimpleString(sb), NIL);
    16921692            else
    1693                 sb.append((char)n);
     1693                sb.append((char)n); // ### BUG: Codepoint conversion
    16941694          }
    16951695      }
     
    17091709        if (n < 0)
    17101710            return error(new EndOfFile(this));
    1711         return LispCharacter.getInstance((char)n);
     1711        return LispCharacter.getInstance((char)n); // ### BUG: Codepoint conversion
    17121712      }
    17131713    catch (IOException e)
     
    17311731                return eofValue;
    17321732          }
    1733         return LispCharacter.getInstance((char)n);
     1733        return LispCharacter.getInstance((char)n); // ### BUG: Codepoint conversion
    17341734      }
    17351735    catch (IOException e)
Note: See TracChangeset for help on using the changeset viewer.