Merge commit '5943a62042432b86d757200ef595d7aebb5c9bac'
authorAndy Wingo <wingo@pobox.com>
Thu, 22 Jan 2015 13:38:00 +0000 (14:38 +0100)
committerAndy Wingo <wingo@pobox.com>
Thu, 22 Jan 2015 13:38:00 +0000 (14:38 +0100)
1  2 
libguile/print.c

diff --combined libguile/print.c
@@@ -30,6 -30,7 +30,6 @@@
  
  #include <uniconv.h>
  #include <unictype.h>
 -#include <c-strcase.h>
  
  #include "libguile/_scm.h"
  #include "libguile/chars.h"
@@@ -40,6 -41,7 +40,6 @@@
  #include "libguile/macros.h"
  #include "libguile/procprop.h"
  #include "libguile/read.h"
 -#include "libguile/weaks.h"
  #include "libguile/programs.h"
  #include "libguile/alist.h"
  #include "libguile/struct.h"
@@@ -67,6 -69,9 +67,9 @@@
  static size_t display_string (const void *, int, size_t, SCM,
                              scm_t_string_failed_conversion_handler);
  
+ static size_t write_string (const void *, int, size_t, SCM,
+                           scm_t_string_failed_conversion_handler);
  static int display_character (scm_t_wchar, SCM,
                              scm_t_string_failed_conversion_handler);
  
@@@ -163,7 -168,7 +166,7 @@@ do                                                         
      {                                                         \
        if (pstate->top - pstate->list_offset >= pstate->level) \
        {                                                       \
 -        scm_putc ('#', port);                                 \
 +        scm_putc_unlocked ('#', port);                                        \
          return;                                               \
        }                                                       \
      }                                                         \
@@@ -307,9 -312,9 +310,9 @@@ print_circref (SCM port, scm_print_stat
    for (i = pstate->top - 1; 1; --i)
      if (scm_is_eq (PSTATE_STACK_REF(pstate, i), ref))
        break;
 -  scm_putc ('#', port);
 +  scm_putc_unlocked ('#', port);
    scm_intprint (i - self, 10, port);
 -  scm_putc ('#', port);
 +  scm_putc_unlocked ('#', port);
  }
  
  /* Print the name of a symbol. */
@@@ -338,7 -343,6 +341,7 @@@ quote_keywordish_symbols (void
    (INITIAL_IDENTIFIER_MASK                                              \
     | UC_CATEGORY_MASK_Nd | UC_CATEGORY_MASK_Mc | UC_CATEGORY_MASK_Me)
  
 +/* FIXME: Cache this information on the symbol, somehow.  */
  static int
  symbol_has_extended_read_syntax (SCM sym)
  {
  
    c = scm_i_symbol_ref (sym, 0);
  
 -  /* Single dot; conflicts with dotted-pair notation.  */
 -  if (len == 1 && c == '.')
 -    return 1;
 -
 -  /* Other initial-character constraints.  */
 -  if (c == '\'' || c == '`' || c == ',' || c == '"' || c == ';' || c == '#')
 -    return 1;
 -
 -  /* R7RS allows neither '|' nor '\' in bare symbols.  */
 -  if ((c == '|' || c == '\\') && SCM_PRINT_R7RS_SYMBOLS_P)
 -    return 1;
 +  switch (c) 
 +    {
 +    case '\'':
 +    case '`':
 +    case ',':
 +    case '"':
 +    case ';':
 +    case '#':
 +      /* Some initial-character constraints.  */
 +      return 1;
 +
 +    case '|':
 +    case '\\':
 +      /* R7RS allows neither '|' nor '\' in bare symbols.  */
 +      if (SCM_PRINT_R7RS_SYMBOLS_P)
 +        return 1;
 +      break;
    
 -  /* Keywords can be identified by trailing colons too.  */
 -  if (c == ':' || scm_i_symbol_ref (sym, len - 1) == ':')
 -    return quote_keywordish_symbols ();
 +    case ':':
 +      /* Symbols that look like keywords.  */
 +      return quote_keywordish_symbols ();
    
 -  /* Number-ish symbols.  */
 -  if (scm_is_true (scm_i_string_to_number (scm_symbol_to_string (sym), 10)))
 -    return 1;
 +    case '.':
 +      /* Single dot conflicts with dotted-pair notation.  */
 +      if (len == 1)
 +        return 1;
 +      /* Fall through to check numbers.  */
 +    case '+':
 +    case '-':
 +    case '0':
 +    case '1':
 +    case '2':
 +    case '3':
 +    case '4':
 +    case '5':
 +    case '6': 
 +    case '7':
 +    case '8':
 +    case '9':
 +     /* Number-ish symbols.  Numbers with radixes already caught be #
 +        above.  */
 +      if (scm_is_true (scm_i_string_to_number (scm_symbol_to_string (sym), 10)))
 +        return 1;
 +      break;
 +
 +    default:
 +      break;
 +    }
    
    /* Other disallowed first characters.  */
    if (!uc_is_general_category_withtable (c, INITIAL_IDENTIFIER_MASK))
      return 1;
  
 +  /* Keywords can be identified by trailing colons too.  */
 +  if (scm_i_symbol_ref (sym, len - 1) == ':')
 +    return quote_keywordish_symbols ();
 +
    /* Otherwise, any character that's in the identifier category mask is
       fine to pass through as-is, provided it's not one of the ASCII
       delimiters like `;'.  */
  static void
  print_normal_symbol (SCM sym, SCM port)
  {
 -  scm_display (scm_symbol_to_string (sym), port);
 +  size_t len;
 +  scm_t_string_failed_conversion_handler strategy;
 +
 +  len = scm_i_symbol_length (sym);
 +  strategy = SCM_PTAB_ENTRY (port)->ilseq_handler;
 +
 +  if (scm_i_is_narrow_symbol (sym))
 +    display_string (scm_i_symbol_chars (sym), 1, len, port, strategy);
 +  else
 +    display_string (scm_i_symbol_wide_chars (sym), 0, len, port, strategy);
  }
  
  static void
@@@ -450,7 -412,7 +453,7 @@@ print_extended_symbol (SCM sym, SCM por
    len = scm_i_symbol_length (sym);
    strategy = PORT_CONVERSION_HANDLER (port);
  
 -  scm_lfwrite ("#{", 2, port);
 +  scm_lfwrite_unlocked ("#{", 2, port);
  
    for (pos = 0; pos < len; pos++)
      {
          }
        else
          {
 -          scm_lfwrite ("\\x", 2, port);
 +          scm_lfwrite_unlocked ("\\x", 2, port);
            scm_intprint (c, 16, port);
 -          scm_putc (';', port);
 +          scm_putc_unlocked (';', port);
          }
      }
  
 -  scm_lfwrite ("}#", 2, port);
 +  scm_lfwrite_unlocked ("}#", 2, port);
  }
  
  static void
@@@ -486,7 -448,7 +489,7 @@@ print_r7rs_extended_symbol (SCM sym, SC
    len = scm_i_symbol_length (sym);
    strategy = PORT_CONVERSION_HANDLER (port);
  
 -  scm_putc ('|', port);
 +  scm_putc_unlocked ('|', port);
  
    for (pos = 0; pos < len; pos++)
      {
  
        switch (c)
          {
 -        case '\a': scm_lfwrite ("\\a", 2, port); break;
 -        case '\b': scm_lfwrite ("\\b", 2, port); break;
 -        case '\t': scm_lfwrite ("\\t", 2, port); break;
 -        case '\n': scm_lfwrite ("\\n", 2, port); break;
 -        case '\r': scm_lfwrite ("\\r", 2, port); break;
 -        case '|':  scm_lfwrite ("\\|", 2, port); break;
 -        case '\\': scm_lfwrite ("\\x5c;", 5, port); break;
 +        case '\a': scm_lfwrite_unlocked ("\\a", 2, port); break;
 +        case '\b': scm_lfwrite_unlocked ("\\b", 2, port); break;
 +        case '\t': scm_lfwrite_unlocked ("\\t", 2, port); break;
 +        case '\n': scm_lfwrite_unlocked ("\\n", 2, port); break;
 +        case '\r': scm_lfwrite_unlocked ("\\r", 2, port); break;
 +        case '|':  scm_lfwrite_unlocked ("\\|", 2, port); break;
 +        case '\\': scm_lfwrite_unlocked ("\\x5c;", 5, port); break;
          default:
            if (uc_is_general_category_withtable (c,
                                                  UC_CATEGORY_MASK_L
              }
            else
              {
 -              scm_lfwrite ("\\x", 2, port);
 +              scm_lfwrite_unlocked ("\\x", 2, port);
                scm_intprint (c, 16, port);
 -              scm_putc (';', port);
 +              scm_putc_unlocked (';', port);
              }
            break;
          }
      }
  
 -  scm_putc ('|', port);
 +  scm_putc_unlocked ('|', port);
  }
  
  /* FIXME: allow R6RS hex escapes instead of #{...}# or |...|.  */
 -void
 -scm_i_print_symbol_name (SCM sym, SCM port)
 +static void
 +print_symbol (SCM sym, SCM port)
  {
    if (!symbol_has_extended_read_syntax (sym))
      print_normal_symbol (sym, port);
  void
  scm_print_symbol_name (const char *str, size_t len, SCM port)
  {
 -  SCM symbol = scm_from_locale_symboln (str, len);
 -  scm_i_print_symbol_name (symbol, port);
 +  SCM symbol = scm_from_utf8_symboln (str, len);
 +  print_symbol (symbol, port);
  }
  
  /* Print generally.  Handles both write and display according to PSTATE.
@@@ -563,7 -525,7 +566,7 @@@ static void iprin1 (SCM exp, SCM port, 
          scm_intprint (i, 8, port);              \
        else                                      \
          {                                       \
 -          scm_puts ("x", port);                 \
 +          scm_puts_unlocked ("x", port);                 \
            scm_intprint (i, 16, port);           \
          }                                       \
      }                                           \
@@@ -584,33 -546,6 +587,33 @@@ scm_iprin1 (SCM exp, SCM port, scm_prin
      iprin1 (exp, port, pstate);
  }
  
 +static void
 +print_vector_or_weak_vector (SCM v, size_t len, SCM (*ref) (SCM, size_t),
 +                             SCM port, scm_print_state *pstate)
 +{
 +  long i;
 +  long last = len - 1;
 +  int cutp = 0;
 +  if (pstate->fancyp && len > pstate->length)
 +    {
 +      last = pstate->length - 1;
 +      cutp = 1;
 +    }
 +  for (i = 0; i < last; ++i)
 +    {
 +      scm_iprin1 (ref (v, i), port, pstate);
 +      scm_putc_unlocked (' ', port);
 +    }
 +  if (i == last)
 +    {
 +      /* CHECK_INTS; */
 +      scm_iprin1 (ref (v, i), port, pstate);
 +    }
 +  if (cutp)
 +    scm_puts_unlocked (" ...", port);
 +  scm_putc_unlocked (')', port);
 +}
 +
  static void
  iprin1 (SCM exp, SCM port, scm_print_state *pstate)
  {
        else if (SCM_IFLAGP (exp)
               && ((size_t) SCM_IFLAGNUM (exp) < (sizeof iflagnames / sizeof (char *))))
          {
 -          scm_puts (iflagnames [SCM_IFLAGNUM (exp)], port);
 +          scm_puts_unlocked (iflagnames [SCM_IFLAGNUM (exp)], port);
          }
        else
        {
                  goto print_struct;
                pwps = scm_i_port_with_print_state (port, pstate->handle);
                pstate->revealed = 1;
 -              scm_call_generic_2 (print, exp, pwps);
 +              scm_call_2 (print, exp, pwps);
              }
            else
              {
              break;
            }
          break;
 +        case scm_tc7_stringbuf:
 +          scm_i_print_stringbuf (exp, port, pstate);
 +          break;
          case scm_tc7_string:
-           if (SCM_WRITINGP (pstate))
-             {
-               size_t len, i;
-               display_character ('"', port, iconveh_question_mark);
-               len = scm_i_string_length (exp);
-               for (i = 0; i < len; ++i)
-               write_character (scm_i_string_ref (exp, i), port, 1);
-               display_character ('"', port, iconveh_question_mark);
-               scm_remember_upto_here_1 (exp);
-             }
-           else
-           {
-             size_t len, printed;
+         {
+           size_t len, printed;
  
-             len = scm_i_string_length (exp);
+           len = scm_i_string_length (exp);
+           if (SCM_WRITINGP (pstate))
+             {
+               printed = write_string (scm_i_string_data (exp),
+                                       scm_i_is_narrow_string (exp),
+                                       len, port,
+                                       PORT_CONVERSION_HANDLER (port));
+               len += 2;                   /* account for the quotes */
+             }
+           else
              printed = display_string (scm_i_string_data (exp),
                                        scm_i_is_narrow_string (exp),
                                        len, port,
                                        PORT_CONVERSION_HANDLER (port));
-             if (SCM_UNLIKELY (printed < len))
-               scm_encoding_error (__func__, errno,
-                                   "cannot convert to output locale",
-                                   port, scm_c_string_ref (exp, printed));
-           }
+           if (SCM_UNLIKELY (printed < len))
+             scm_encoding_error (__func__, errno,
+                                 "cannot convert to output locale",
+                                 port, scm_c_string_ref (exp, printed));
+         }
  
            scm_remember_upto_here_1 (exp);
            break;
        case scm_tc7_symbol:
          if (scm_i_symbol_is_interned (exp))
            {
 -            scm_i_print_symbol_name (exp, port);
 +            print_symbol (exp, port);
              scm_remember_upto_here_1 (exp);
            }
          else
            {
 -            scm_puts ("#<uninterned-symbol ", port);
 -            scm_i_print_symbol_name (exp, port);
 -            scm_putc (' ', port);
 +            scm_puts_unlocked ("#<uninterned-symbol ", port);
 +            print_symbol (exp, port);
 +            scm_putc_unlocked (' ', port);
              scm_uintprint (SCM_UNPACK (exp), 16, port);
 -            scm_putc ('>', port);
 +            scm_putc_unlocked ('>', port);
            }
          break;
        case scm_tc7_variable:
        case scm_tc7_hashtable:
          scm_i_hashtable_print (exp, port, pstate);
          break;
 +      case scm_tc7_weak_set:
 +        scm_i_weak_set_print (exp, port, pstate);
 +        break;
 +      case scm_tc7_weak_table:
 +        scm_i_weak_table_print (exp, port, pstate);
 +        break;
        case scm_tc7_fluid:
          scm_i_fluid_print (exp, port, pstate);
          break;
        case scm_tc7_frame:
          scm_i_frame_print (exp, port, pstate);
          break;
 -      case scm_tc7_objcode:
 -        scm_i_objcode_print (exp, port, pstate);
 -        break;
 -      case scm_tc7_vm:
 -        scm_i_vm_print (exp, port, pstate);
 -        break;
 +        case scm_tc7_keyword:
 +          scm_puts_unlocked ("#:", port);
 +          scm_iprin1 (scm_keyword_to_symbol (exp), port, pstate);
 +          break;
        case scm_tc7_vm_cont:
          scm_i_vm_cont_print (exp, port, pstate);
          break;
 -      case scm_tc7_prompt:
 -        scm_i_prompt_print (exp, port, pstate);
 -        break;
 -      case scm_tc7_with_fluids:
 -        scm_i_with_fluids_print (exp, port, pstate);
 -        break;
        case scm_tc7_array:
          ENTER_NESTED_DATA (pstate, exp, circref);
            scm_i_print_array (exp, port, pstate);
          break;
        case scm_tc7_wvect:
          ENTER_NESTED_DATA (pstate, exp, circref);
 -        if (SCM_IS_WHVEC (exp))
 -          scm_puts ("#wh(", port);
 -        else
 -          scm_puts ("#w(", port);
 -        goto common_vector_printer;
 +          scm_puts_unlocked ("#w(", port);
 +          print_vector_or_weak_vector (exp, scm_c_weak_vector_length (exp),
 +                                       scm_c_weak_vector_ref, port, pstate);
 +        EXIT_NESTED_DATA (pstate);
 +        break;
        case scm_tc7_vector:
          ENTER_NESTED_DATA (pstate, exp, circref);
 -        scm_puts ("#(", port);
 -      common_vector_printer:
 -        {
 -          register long i;
 -          long last = SCM_SIMPLE_VECTOR_LENGTH (exp) - 1;
 -          int cutp = 0;
 -          if (pstate->fancyp
 -              && SCM_SIMPLE_VECTOR_LENGTH (exp) > pstate->length)
 -            {
 -              last = pstate->length - 1;
 -              cutp = 1;
 -            }
 -          if (SCM_I_WVECTP (exp))
 -            {
 -              /* Elements of weak vectors may not be accessed via the
 -                 `SIMPLE_VECTOR_REF ()' macro.  */
 -              for (i = 0; i < last; ++i)
 -                {
 -                  scm_iprin1 (scm_c_weak_vector_ref (exp, i),
 -                              port, pstate);
 -                  scm_putc (' ', port);
 -                }
 -            }
 -          else
 -            {
 -              for (i = 0; i < last; ++i)
 -                {
 -                  scm_iprin1 (SCM_SIMPLE_VECTOR_REF (exp, i), port, pstate);
 -                  scm_putc (' ', port);
 -                }
 -            }
 -
 -          if (i == last)
 -            {
 -              /* CHECK_INTS; */
 -                scm_iprin1 (SCM_I_WVECTP (exp)
 -                            ? scm_c_weak_vector_ref (exp, i)
 -                            : SCM_SIMPLE_VECTOR_REF (exp, i),
 -                            port, pstate);
 -            }
 -          if (cutp)
 -            scm_puts (" ...", port);
 -          scm_putc (')', port);
 -        }
 +        scm_puts_unlocked ("#(", port);
 +          print_vector_or_weak_vector (exp, SCM_SIMPLE_VECTOR_LENGTH (exp),
 +                                       scm_c_vector_ref, port, pstate);
          EXIT_NESTED_DATA (pstate);
          break;
        case scm_tc7_port:
          {
 -          register long i = SCM_PTOBNUM (exp);
 -          if (i < scm_numptob
 -              && scm_ptobs[i].print
 -              && (scm_ptobs[i].print) (exp, port, pstate))
 +          scm_t_ptob_descriptor *ptob = SCM_PORT_DESCRIPTOR (exp);
 +          if (ptob->print && ptob->print (exp, port, pstate))
              break;
            goto punk;
          }
@@@ -956,7 -930,7 +956,7 @@@ display_string_as_utf8 (const void *str
  
        /* INPUT was successfully converted, entirely; print the
         result.  */
 -      scm_lfwrite (utf8_buf, utf8_len, port);
 +      scm_lfwrite_unlocked (utf8_buf, utf8_len, port);
        printed += i - printed;
      }
  
    return len;
  }
  
 +/* Write STR to PORT as ISO-8859-1.  STR is a LEN-codepoint string; it
 +   is narrow if NARROW_P is true, wide otherwise.  Return LEN.  */
 +static size_t
 +display_string_as_latin1 (const void *str, int narrow_p, size_t len,
 +                          SCM port,
 +                          scm_t_string_failed_conversion_handler strategy)
 +{
 +  size_t printed = 0;
 +
 +  if (narrow_p)
 +    {
 +      scm_lfwrite_unlocked (str, len, port);
 +      return len;
 +    }
 +
 +  while (printed < len)
 +    {
 +      char buf[256];
 +      size_t i;
 +
 +      for (i = 0; i < sizeof(buf) && printed < len; i++, printed++)
 +        {
 +          scm_t_wchar c = STR_REF (str, printed);
 +
 +          if (c < 256)
 +            buf[i] = c;
 +          else
 +            break;
 +        }
 +
 +      scm_lfwrite_unlocked (buf, i, port);
 +
 +      if (i < sizeof(buf) && printed < len)
 +        {
 +          if (strategy == SCM_FAILED_CONVERSION_ERROR)
 +            break;
 +          else if (strategy == SCM_FAILED_CONVERSION_ESCAPE_SEQUENCE)
 +            write_character_escaped (STR_REF (str, printed), 1, port);
 +          else
 +            /* STRATEGY is `SCM_FAILED_CONVERSION_QUESTION_MARK'.  */
 +            display_string ("?", 1, 1, port, strategy);
 +          printed++;
 +        }
 +    }
 +
 +  return printed;
 +}
 +
  /* Convert STR through PORT's output conversion descriptor and write the
     output to PORT.  Return the number of codepoints written.  */
  static size_t
@@@ -1036,8 -962,8 +1036,8 @@@ display_string_using_iconv (const void 
          pti->at_stream_start_for_bom_read = 0;
  
        /* Write a BOM if appropriate.  */
 -      if (SCM_UNLIKELY (c_strcasecmp(pt->encoding, "UTF-16") == 0
 -                        || c_strcasecmp(pt->encoding, "UTF-32") == 0))
 +      if (SCM_UNLIKELY (strcmp(pt->encoding, "UTF-16") == 0
 +                        || strcmp(pt->encoding, "UTF-32") == 0))
          display_character (SCM_UNICODE_BOM, port, iconveh_error);
      }
  
          iconv (id->output_cd, NULL, NULL, NULL, NULL);
  
          /* Print the OUTPUT_LEN bytes successfully converted.  */
 -        scm_lfwrite (encoded_output, output_len, port);
 +        scm_lfwrite_unlocked (encoded_output, output_len, port);
  
          /* See how many input codepoints these OUTPUT_LEN bytes
             corresponds to.  */
        {
          /* INPUT was successfully converted, entirely; print the
             result.  */
 -        scm_lfwrite (encoded_output, output_len, port);
 +        scm_lfwrite_unlocked (encoded_output, output_len, port);
          codepoints_read = i - printed;
          printed += codepoints_read;
        }
    return printed;
  }
  
- #undef STR_REF
  /* Display the LEN codepoints in STR to PORT according to STRATEGY;
     return the number of codepoints successfully displayed.  If NARROW_P,
     then STR is interpreted as a sequence of `char', denoting a Latin-1
@@@ -1136,6 -1060,7 +1134,6 @@@ static size_
  display_string (const void *str, int narrow_p,
                size_t len, SCM port,
                scm_t_string_failed_conversion_handler strategy)
 -
  {
    scm_t_port_internal *pti;
  
  
    if (pti->encoding_mode == SCM_PORT_ENCODING_MODE_UTF8)
      return display_string_as_utf8 (str, narrow_p, len, port);
 +  else if (pti->encoding_mode == SCM_PORT_ENCODING_MODE_LATIN1)
 +    return display_string_as_latin1 (str, narrow_p, len, port, strategy);
    else
 -    return display_string_using_iconv (str, narrow_p, len,
 -                                     port, strategy);
 +    return display_string_using_iconv (str, narrow_p, len, port, strategy);
  }
  
- /* Attempt to display CH to PORT according to STRATEGY.  Return non-zero
-    if CH was successfully displayed, zero otherwise (e.g., if it was not
+ /* Attempt to display CH to PORT according to STRATEGY.  Return one if
+    CH was successfully displayed, zero otherwise (e.g., if it was not
     representable in PORT's encoding.)  */
  static int
  display_character (scm_t_wchar ch, SCM port,
    return display_string (&ch, 0, 1, port, strategy) == 1;
  }
  
+ /* Same as 'display_string', but using the 'write' syntax.  */
+ static size_t
+ write_string (const void *str, int narrow_p,
+             size_t len, SCM port,
+             scm_t_string_failed_conversion_handler strategy)
+ {
+   size_t printed;
+   printed = display_character ('"', port, strategy);
+   if (printed > 0)
+     {
+       size_t i;
+       for (i = 0; i < len; ++i)
+       {
+         write_character (STR_REF (str, i), port, 1);
+         printed++;
+       }
+       printed += display_character ('"', port, strategy);
+     }
+   return printed;
+ }
+ #undef STR_REF
  /* Attempt to pretty-print CH, a combining character, to PORT.  Return
     zero upon failure, non-zero otherwise.  The idea is to print CH above
     a dotted circle to make it more visible.  */
@@@ -1191,7 -1143,7 +1217,7 @@@ write_character_escaped (scm_t_wchar ch
          /* Use special escapes for some C0 controls.  */
          buf[0] = '\\';
          buf[1] = escapes[ch - 0x07];
 -        scm_lfwrite (buf, 2, port);
 +        scm_lfwrite_unlocked (buf, 2, port);
        }
        else if (!SCM_R6RS_ESCAPES_P)
        {
              buf[1] = 'x';
              buf[2] = hex[ch / 16];
              buf[3] = hex[ch % 16];
 -            scm_lfwrite (buf, 4, port);
 +            scm_lfwrite_unlocked (buf, 4, port);
            }
          else if (ch <= 0xFFFF)
            {
              buf[3] = hex[(ch & 0xF00) >> 8];
              buf[4] = hex[(ch & 0xF0) >> 4];
              buf[5] = hex[(ch & 0xF)];
 -            scm_lfwrite (buf, 6, port);
 +            scm_lfwrite_unlocked (buf, 6, port);
            }
          else if (ch > 0xFFFF)
            {
              buf[5] = hex[(ch & 0xF00) >> 8];
              buf[6] = hex[(ch & 0xF0) >> 4];
              buf[7] = hex[(ch & 0xF)];
 -            scm_lfwrite (buf, 8, port);
 +            scm_lfwrite_unlocked (buf, 8, port);
            }
        }
        else
          buf[i] = 'x';
          i --;
          buf[i] = '\\';
 -        scm_lfwrite (buf + i, 9 - i, port);
 +        scm_lfwrite_unlocked (buf + i, 9 - i, port);
        }
      }
    else
  
        name = scm_i_charname (SCM_MAKE_CHAR (ch));
        if (name != NULL)
 -      scm_puts (name, port);
 +      scm_puts_unlocked (name, port);
        else
        PRINT_CHAR_ESCAPE (ch, port);
      }
  scm_intprint (scm_t_intmax n, int radix, SCM port)
  {
    char num_buf[SCM_INTBUFLEN];
 -  scm_lfwrite (num_buf, scm_iint2str (n, radix, num_buf), port);
 +  scm_lfwrite_unlocked (num_buf, scm_iint2str (n, radix, num_buf), port);
  }
  
  void 
  scm_uintprint (scm_t_uintmax n, int radix, SCM port)
  {
    char num_buf[SCM_INTBUFLEN];
 -  scm_lfwrite (num_buf, scm_iuint2str (n, radix, num_buf), port);
 +  scm_lfwrite_unlocked (num_buf, scm_iuint2str (n, radix, num_buf), port);
  }
  
  /* Print an object of unrecognized type.
  void 
  scm_ipruk (char *hdr, SCM ptr, SCM port)
  {
 -  scm_puts ("#<unknown-", port);
 -  scm_puts (hdr, port);
 +  scm_puts_unlocked ("#<unknown-", port);
 +  scm_puts_unlocked (hdr, port);
    if (1) /* (scm_in_heap_p (ptr)) */ /* FIXME */
      {
 -      scm_puts (" (0x", port);
 +      scm_puts_unlocked (" (0x", port);
        scm_uintprint (SCM_CELL_WORD_0 (ptr), 16, port);
 -      scm_puts (" . 0x", port);
 +      scm_puts_unlocked (" . 0x", port);
        scm_uintprint (SCM_CELL_WORD_1 (ptr), 16, port);
 -      scm_puts (") @", port);
 +      scm_puts_unlocked (") @", port);
      }
 -  scm_puts (" 0x", port);
 +  scm_puts_unlocked (" 0x", port);
    scm_uintprint (SCM_UNPACK (ptr), 16, port);
 -  scm_putc ('>', port);
 +  scm_putc_unlocked ('>', port);
  }
  
  
@@@ -1389,7 -1341,7 +1415,7 @@@ scm_iprlist (char *hdr, SCM exp, int tl
  {
    register SCM hare, tortoise;
    long floor = pstate->top - 2;
 -  scm_puts (hdr, port);
 +  scm_puts_unlocked (hdr, port);
    /* CHECK_INTS; */
    if (pstate->fancyp)
      goto fancy_printing;
        if (scm_is_eq (PSTATE_STACK_REF(pstate, i), exp))
          goto circref;
        PUSH_REF (pstate, exp);
 -      scm_putc (' ', port);
 +      scm_putc_unlocked (' ', port);
        /* CHECK_INTS; */
        scm_iprin1 (SCM_CAR (exp), port, pstate);
      }
    if (!SCM_NULL_OR_NIL_P (exp))
      {
 -      scm_puts (" . ", port);
 +      scm_puts_unlocked (" . ", port);
        scm_iprin1 (exp, port, pstate);
      }
  
  end:
 -  scm_putc (tlr, port);
 +  scm_putc_unlocked (tlr, port);
    pstate->top = floor + 2;
    return;
    
@@@ -1451,7 -1403,7 +1477,7 @@@ fancy_printing
          {
            if (n == 0)
              {
 -              scm_puts (" ...", port);
 +              scm_puts_unlocked (" ...", port);
                goto skip_tail;
              }
            else
          }
        PUSH_REF(pstate, exp);
        ++pstate->list_offset;
 -      scm_putc (' ', port);
 +      scm_putc_unlocked (' ', port);
        /* CHECK_INTS; */
        scm_iprin1 (SCM_CAR (exp), port, pstate);
        }
    }
    if (!SCM_NULL_OR_NIL_P (exp))
      {
 -      scm_puts (" . ", port);
 +      scm_puts_unlocked (" . ", port);
        scm_iprin1 (exp, port, pstate);
      }
  skip_tail:
@@@ -1477,7 -1429,7 +1503,7 @@@ fancy_circref
    pstate->list_offset -= pstate->top - floor - 2;
    
  circref:
 -  scm_puts (" . ", port);
 +  scm_puts_unlocked (" . ", port);
    print_circref (port, pstate, exp);
    goto end;
  }
@@@ -1502,11 -1454,7 +1528,11 @@@ scm_write (SCM obj, SCM port
  
    SCM_ASSERT (scm_valid_oport_value_p (port), port, SCM_ARG2, s_write);
  
 +  scm_dynwind_begin (0);
 +  scm_dynwind_lock_port (SCM_COERCE_OUTPORT (port));
    scm_prin1 (obj, port, 1);
 +  scm_dynwind_end ();
 +
    return SCM_UNSPECIFIED;
  }
  
@@@ -1521,11 -1469,7 +1547,11 @@@ scm_display (SCM obj, SCM port
  
    SCM_ASSERT (scm_valid_oport_value_p (port), port, SCM_ARG2, s_display);
  
 +  scm_dynwind_begin (0);
 +  scm_dynwind_lock_port (SCM_COERCE_OUTPORT (port));
    scm_prin1 (obj, port, 0);
 +  scm_dynwind_end ();
 +
    return SCM_UNSPECIFIED;
  }
  
@@@ -1639,7 -1583,7 +1665,7 @@@ SCM_DEFINE (scm_newline, "newline", 0, 
  
    SCM_VALIDATE_OPORT_VALUE (1, port);
  
 -  scm_putc ('\n', SCM_COERCE_OUTPORT (port));
 +  scm_putc_unlocked ('\n', SCM_COERCE_OUTPORT (port));
    return SCM_UNSPECIFIED;
  }
  #undef FUNC_NAME
@@@ -1684,7 -1628,7 +1710,7 @@@ static in
  port_with_ps_print (SCM obj, SCM port, scm_print_state *pstate)
  {
    obj = SCM_PORT_WITH_PS_PORT (obj);
 -  return scm_ptobs[SCM_PTOBNUM (obj)].print (obj, port, pstate);
 +  return SCM_PORT_DESCRIPTOR (obj)->print (obj, port, pstate);
  }
  
  SCM
@@@ -1731,6 -1675,8 +1757,6 @@@ scm_init_print (
  {
    SCM type;
  
 -  scm_gc_register_root (&print_state_pool);
 -  scm_gc_register_root (&scm_print_state_vtable);
    type = scm_make_vtable (scm_from_locale_string (SCM_PRINT_STATE_LAYOUT),
                            SCM_BOOL_F);
    scm_set_struct_vtable_name_x (type, scm_from_latin1_symbol ("print-state"));