Comment on comment.
[p5sagit/p5-mst-13.2.git] / perlio.c
index 925e3fb..65b2358 100644 (file)
--- a/perlio.c
+++ b/perlio.c
@@ -40,6 +40,59 @@ PerlIO_apply_layers(pTHX_ PerlIO *f, const char *mode, const char *names)
  /* NOTREACHED */
  return -1;
 }
+
+int
+perlsio_binmode(FILE *fp, int iotype, int mode)
+{
+/* This used to be contents of do_binmode in doio.c */
+#ifdef DOSISH
+#  if defined(atarist) || defined(__MINT__)
+    if (!fflush(fp)) {
+       if (mode & O_BINARY)
+           ((FILE*)fp)->_flag |= _IOBIN;
+       else
+           ((FILE*)fp)->_flag &= ~ _IOBIN;
+       return 1;
+    }
+    return 0;
+#  else
+    if (PerlLIO_setmode(fileno(fp), mode) != -1) {
+#    if defined(WIN32) && defined(__BORLANDC__)
+       /* The translation mode of the stream is maintained independent
+        * of the translation mode of the fd in the Borland RTL (heavy
+        * digging through their runtime sources reveal).  User has to
+        * set the mode explicitly for the stream (though they don't
+        * document this anywhere). GSAR 97-5-24
+        */
+       fseek(fp,0L,0);
+       if (mode & O_BINARY)
+           fp->flags |= _F_BIN;
+       else
+           fp->flags &= ~ _F_BIN;
+#    endif
+       return 1;
+    }
+    else
+       return 0;
+#  endif
+#else
+#  if defined(USEMYBINMODE)
+    if (my_binmode(fp, iotype, mode) != FALSE)
+       return 1;
+    else
+       return 0;
+#  else
+    return 1;
+#  endif
+#endif
+}
+
+int
+PerlIO_binmode(pTHX_ PerlIO *fp, int iotype, int mode, const char *names)
+{
+ return perlsio_binmode(fp,iotype,mode);
+}
+
 #endif
 
 #if !defined(PERL_IMPLICIT_SYS)
@@ -193,7 +246,7 @@ PerlIO_cleantable(PerlIO **tablep)
    for (i=PERLIO_TABLE_SIZE-1; i > 0; i--)
     {
      PerlIO *f = table+i;
-     if (*f) 
+     if (*f)
       {
        PerlIO_close(f);
       }
@@ -218,6 +271,7 @@ PerlIO_pop(PerlIO *f)
  PerlIOl *l = *f;
  if (l)
   {
+   PerlIO_debug(__FUNCTION__ " f=%p %s\n",f,l->tab->name);
    (*l->tab->Popped)(f);
    *f = l->next;
    Safefree(l);
@@ -410,14 +464,21 @@ PerlIO_default_layer(I32 n)
  len  = av_len(PerlIO_layer_av);
  if (len < 1)
   {
-   if (PerlIO_stdio.Set_ptrcnt)
+   if (O_BINARY != O_TEXT)
     {
-     av_push(PerlIO_layer_av,SvREFCNT_inc(PerlIO_find_layer(PerlIO_stdio.name,0)));
+     av_push(PerlIO_layer_av,SvREFCNT_inc(PerlIO_find_layer(PerlIO_crlf.name,0)));
     }
    else
-    {
-     av_push(PerlIO_layer_av,SvREFCNT_inc(PerlIO_find_layer(PerlIO_perlio.name,0)));
-    }
+    { 
+     if (PerlIO_stdio.Set_ptrcnt)
+      {
+       av_push(PerlIO_layer_av,SvREFCNT_inc(PerlIO_find_layer(PerlIO_stdio.name,0)));
+      }
+     else 
+      {
+       av_push(PerlIO_layer_av,SvREFCNT_inc(PerlIO_find_layer(PerlIO_perlio.name,0)));
+      }
+    } 
    len  = av_len(PerlIO_layer_av);
   }
  if (n < 0)
@@ -431,6 +492,42 @@ PerlIO_default_layer(I32 n)
  return tab;
 }
 
+#define PerlIO_default_top() PerlIO_default_layer(-1)
+#define PerlIO_default_btm() PerlIO_default_layer(0)
+
+void
+PerlIO_stdstreams()
+{
+ if (!_perlio)
+  {
+   PerlIO_allocate();
+   PerlIO_fdopen(0,"Ir" PERLIO_STDTEXT);
+   PerlIO_fdopen(1,"Iw" PERLIO_STDTEXT);
+   PerlIO_fdopen(2,"Iw" PERLIO_STDTEXT);
+  }
+}
+
+PerlIO *
+PerlIO_push(PerlIO *f,PerlIO_funcs *tab,const char *mode)
+{
+ PerlIOl *l = NULL;
+ Newc('L',l,tab->size,char,PerlIOl);
+ if (l)
+  {
+   Zero(l,tab->size,char);
+   l->next = *f;
+   l->tab  = tab;
+   *f      = l;
+   PerlIO_debug(__FUNCTION__ " f=%p %s %s\n",f,tab->name,(mode) ? mode : "(Null)"); 
+   if ((*l->tab->Pushed)(f,mode) != 0)
+    {
+     PerlIO_pop(f);
+     return NULL;
+    }
+  }
+ return f;
+}
+
 int
 PerlIO_apply_layers(pTHX_ PerlIO *f, const char *mode, const char *names)
 {
@@ -450,19 +547,34 @@ PerlIO_apply_layers(pTHX_ PerlIO *f, const char *mode, const char *names)
         e++;
        if (e > s)
         {
-         SV *layer = PerlIO_find_layer(s,e-s);
-         if (layer)
+         if ((e - s) == 3 && strncmp(s,"raw",3) == 0)
           {
-           PerlIO_funcs *tab = INT2PTR(PerlIO_funcs *, SvIV(SvRV(layer)));
-           if (tab)
+           /* Pop back to bottom layer */
+           if (PerlIONext(f))
             {
-             PerlIO *new = PerlIO_push(f,tab,mode);
-             if (!new)
-              return -1;
+             PerlIO_flush(f);
+             while (PerlIONext(f))
+              {
+               PerlIO_pop(f);
+              }
             }
           }
          else
-          Perl_warn(aTHX_ "perlio: unknown layer \"%.*s\"",(e-s),s);
+          {
+           SV *layer = PerlIO_find_layer(s,e-s);
+           if (layer)
+            {
+             PerlIO_funcs *tab = INT2PTR(PerlIO_funcs *, SvIV(SvRV(layer)));
+             if (tab)
+              {
+               PerlIO *new = PerlIO_push(f,tab,mode);
+               if (!new)
+                return -1;
+              }
+            }
+           else
+            Perl_warn(aTHX_ "perlio: unknown layer \"%.*s\"",(int)(e-s),s);
+          }
         }
        s = e;
       }
@@ -471,43 +583,41 @@ PerlIO_apply_layers(pTHX_ PerlIO *f, const char *mode, const char *names)
  return 0;
 }
 
-#define PerlIO_default_top() PerlIO_default_layer(-1)
-#define PerlIO_default_btm() PerlIO_default_layer(0)
 
-void
-PerlIO_stdstreams()
-{
- if (!_perlio)
-  {
-   PerlIO_allocate();
-   PerlIO_fdopen(0,"Ir");
-   PerlIO_fdopen(1,"Iw");
-   PerlIO_fdopen(2,"Iw");
-  }
-}
 
-PerlIO *
-PerlIO_push(PerlIO *f,PerlIO_funcs *tab,const char *mode)
+/*--------------------------------------------------------------------------------------*/
+/* Given the abstraction above the public API functions */
+
+int
+PerlIO_binmode(pTHX_ PerlIO *f, int iotype, int mode, const char *names)
 {
- PerlIOl *l = NULL;
- Newc('L',l,tab->size,char,PerlIOl);
- if (l)
+ PerlIO_debug(__FUNCTION__ " f=%p %s %c %x %s\n",
+              f,PerlIOBase(f)->tab->name,iotype,mode, (names) ? names : "(Null)"); 
+ if (!names || (O_TEXT != O_BINARY && mode & O_BINARY))
   {
-   Zero(l,tab->size,char);
-   l->next = *f;
-   l->tab  = tab;
-   *f      = l;
-   if ((*l->tab->Pushed)(f,mode) != 0)
+   PerlIO *top = f;
+   PerlIOl *l;
+   while (l = *top)
     {
-     PerlIO_pop(f);
-     return NULL;
+     if (PerlIOBase(top)->tab == &PerlIO_crlf)
+      {
+       PerlIO_flush(top);
+       PerlIOBase(top)->flags &= ~PERLIO_F_CRLF;       
+       break;
+      }
+     top = PerlIONext(top);
     }
   }
- return f;
+ return PerlIO_apply_layers(aTHX_ f, NULL, names) == 0 ? TRUE : FALSE;
+}
+
+#undef PerlIO__close
+int
+PerlIO__close(PerlIO *f)
+{
+ return (*PerlIOBase(f)->tab->Close)(f);
 }
 
-/*--------------------------------------------------------------------------------------*/
-/* Given the abstraction above the public API functions */
 
 #undef PerlIO_close
 int
@@ -662,7 +772,8 @@ PerlIO_error(PerlIO *f)
 void
 PerlIO_clearerr(PerlIO *f)
 {
- (*PerlIOBase(f)->tab->Clearerr)(f);
+ if (f && *f)
+  (*PerlIOBase(f)->tab->Clearerr)(f);
 }
 
 #undef PerlIO_setlinebuf
@@ -770,24 +881,43 @@ PerlIOBase_fileno(PerlIO *f)
  return PerlIO_fileno(PerlIONext(f));
 }
 
+char *
+PerlIO_modestr(PerlIO *f,char *buf)
+{
+ char *s = buf;
+ IV flags = PerlIOBase(f)->flags;
+ if (flags & PERLIO_F_CANREAD)
+  *s++ = 'r'; 
+ if (flags & PERLIO_F_CANWRITE)
+  *s++ = 'w'; 
+ if (flags & PERLIO_F_CRLF)
+  *s++ = 't'; 
+ else
+  *s++ = 'b'; 
+ *s = '\0';
+ return buf;
+}
+
 IV
 PerlIOBase_pushed(PerlIO *f, const char *mode)
 {
  PerlIOl *l = PerlIOBase(f);
+ const char *omode = mode;
+ char temp[8];
  l->flags  &= ~(PERLIO_F_CANREAD|PERLIO_F_CANWRITE|
-                PERLIO_F_TRUNCATE|PERLIO_F_APPEND|PERLIO_F_BINARY);
+                PERLIO_F_TRUNCATE|PERLIO_F_APPEND);
  if (mode)
   {
    switch (*mode++)
     {
      case 'r':
-      l->flags = PERLIO_F_CANREAD;
+      l->flags |= PERLIO_F_CANREAD;
       break;
      case 'a':
-      l->flags = PERLIO_F_APPEND|PERLIO_F_CANWRITE;
+      l->flags |= PERLIO_F_APPEND|PERLIO_F_CANWRITE;
       break;
      case 'w':
-      l->flags = PERLIO_F_TRUNCATE|PERLIO_F_CANWRITE;
+      l->flags |= PERLIO_F_TRUNCATE|PERLIO_F_CANWRITE;
       break;
      default:
       errno = EINVAL;
@@ -801,7 +931,10 @@ PerlIOBase_pushed(PerlIO *f, const char *mode)
         l->flags |= PERLIO_F_CANREAD|PERLIO_F_CANWRITE;
         break;
        case 'b':
-        l->flags |= PERLIO_F_BINARY;
+        l->flags &= ~PERLIO_F_CRLF;
+        break;
+       case 't':
+        l->flags |= PERLIO_F_CRLF;
         break;
       default:
        errno = EINVAL;
@@ -814,10 +947,12 @@ PerlIOBase_pushed(PerlIO *f, const char *mode)
    if (l->next)
     {
      l->flags |= l->next->flags &
-                 (PERLIO_F_CANREAD|PERLIO_F_CANWRITE|
-                   PERLIO_F_TRUNCATE|PERLIO_F_APPEND|PERLIO_F_BINARY);
+                 (PERLIO_F_CANREAD|PERLIO_F_CANWRITE|PERLIO_F_TRUNCATE|PERLIO_F_APPEND);
     }
   }
+ PerlIO_debug(__FUNCTION__ " f=%p %s %s fl=%08x (%s)\n",
+              f,PerlIOBase(f)->tab->name,(omode) ? omode : "(Null)",
+              l->flags,PerlIO_modestr(f,temp)); 
  return 0;
 }
 
@@ -855,9 +990,10 @@ IV
 PerlIOBase_close(PerlIO *f)
 {
  IV code = 0;
+ PerlIO *n = PerlIONext(f);
  if (PerlIO_flush(f) != 0)
   code = -1;
- if (PerlIO_close(PerlIONext(f)) != 0)
+ if (n && (*PerlIOBase(n)->tab->Close)(n) != 0)
   code = -1;
  PerlIOBase(f)->flags &= ~(PERLIO_F_CANREAD|PERLIO_F_CANWRITE|PERLIO_F_OPEN);
  return code;
@@ -888,7 +1024,10 @@ PerlIOBase_clearerr(PerlIO *f)
 {
  if (f && *f)
   {
-   PerlIOBase(f)->flags &= ~PERLIO_F_ERROR;
+   PerlIO *n = PerlIONext(f);
+   PerlIOBase(f)->flags &= ~(PERLIO_F_ERROR|PERLIO_F_EOF);
+   if (n)
+    PerlIO_clearerr(n);
   }
 }
 
@@ -947,9 +1086,18 @@ PerlIOUnix_oflags(const char *mode)
   }
  if (*mode == 'b')
   {
-   oflags |= O_BINARY;
-   mode++; 
-  }   
+   oflags |=  O_BINARY;
+   oflags &= ~O_TEXT;
+   mode++;
+  }
+ else if (*mode == 't')
+  {
+   oflags |=  O_TEXT;
+   oflags &= ~O_BINARY;
+   mode++;
+  }
+ /* Always open in binary mode */
+ oflags |= O_BINARY;
  if (*mode || oflags == -1)
   {
    errno = EINVAL;
@@ -1097,7 +1245,7 @@ PerlIOUnix_close(PerlIO *f)
 PerlIO_funcs PerlIO_unix = {
  "unix",
  sizeof(PerlIOUnix),
- 0,
+ PERLIO_K_RAW,
  PerlIOUnix_fileno,
  PerlIOUnix_fdopen,
  PerlIOUnix_open,
@@ -1138,12 +1286,29 @@ PerlIOStdio_fileno(PerlIO *f)
  return fileno(PerlIOSelf(f,PerlIOStdio)->stdio);
 }
 
+const char *
+PerlIOStdio_mode(const char *mode,char *tmode)
+{
+ const char *ret = mode;
+ if (O_BINARY != O_TEXT)
+  {
+   ret = (const char *) tmode;
+   while (*mode)
+    {
+     *tmode++ = *mode++;
+    }
+   *tmode++ = 'b';
+   *tmode = '\0';   
+  }
+ return ret;
+}
 
 PerlIO *
 PerlIOStdio_fdopen(PerlIO_funcs *self, int fd,const char *mode)
 {
  PerlIO *f = NULL;
  int init = 0;
+ char tmode[8];
  if (*mode == 'I')
   {
    init = 1;
@@ -1168,7 +1333,9 @@ PerlIOStdio_fdopen(PerlIO_funcs *self, int fd,const char *mode)
       }
     }
    else
-    stdio = fdopen(fd,mode);
+    {
+     stdio = fdopen(fd,mode = PerlIOStdio_mode(mode,tmode));
+    }
    if (stdio)
     {
      PerlIOStdio *s = PerlIOSelf(PerlIO_push(f = PerlIO_allocate(),self,mode),PerlIOStdio);
@@ -1198,7 +1365,10 @@ PerlIOStdio_open(PerlIO_funcs *self, const char *path,const char *mode)
  FILE *stdio = fopen(path,mode);
  if (stdio)
   {
-   PerlIOStdio *s = PerlIOSelf(PerlIO_push(f = PerlIO_allocate(),self,mode),PerlIOStdio);
+   char tmode[8];
+   PerlIOStdio *s = PerlIOSelf(PerlIO_push(f = PerlIO_allocate(), self, 
+                               (mode = PerlIOStdio_mode(mode,tmode))),
+                               PerlIOStdio);
    s->stdio  = stdio;
   }
  return f;
@@ -1208,7 +1378,8 @@ int
 PerlIOStdio_reopen(const char *path, const char *mode, PerlIO *f)
 {
  PerlIOStdio *s = PerlIOSelf(f,PerlIOStdio);
- FILE *stdio = freopen(path,mode,s->stdio);
+ char tmode[8];
+ FILE *stdio = freopen(path,(mode = PerlIOStdio_mode(mode,tmode)),s->stdio);
  if (!s->stdio)
   return -1;
  s->stdio = stdio;
@@ -1278,8 +1449,12 @@ PerlIOStdio_tell(PerlIO *f)
 IV
 PerlIOStdio_close(PerlIO *f)
 {
+ int optval, optlen = sizeof(int);
  FILE *stdio = PerlIOSelf(f,PerlIOStdio)->stdio;
- return fclose(stdio);
+ return(
+   (getsockopt(PerlIO_fileno(f), SOL_SOCKET, SO_TYPE, (char *)&optval, &optlen) < 0) ? 
+       fclose(stdio) :
+       close(PerlIO_fileno(f)));
 }
 
 IV
@@ -1423,7 +1598,7 @@ PerlIOStdio_set_ptrcnt(PerlIO *f,STDCHAR *ptr,SSize_t cnt)
 PerlIO_funcs PerlIO_stdio = {
  "stdio",
  sizeof(PerlIOStdio),
- 0,
+ PERLIO_K_BUFFERED,
  PerlIOStdio_fileno,
  PerlIOStdio_fdopen,
  PerlIOStdio_open,
@@ -1500,15 +1675,23 @@ PerlIOBuf_fdopen(PerlIO_funcs *self, int fd, const char *mode)
    init = 1;
    mode++;
   }
+ if (O_BINARY != O_TEXT)
+  {
+   int code = PerlLIO_setmode(fd, O_BINARY);
+   PerlIO_debug(__FUNCTION__ " %s fd=%d m=%s c=%d\n",tab->name,fd,mode,code); 
+  }
  f = (*tab->Fdopen)(tab,fd,mode);
  if (f)
   {
-   /* Initial stderr is unbuffered */
-   if (!init || fd != 2)
+   PerlIOBuf *b = PerlIOSelf(PerlIO_push(f,self,mode),PerlIOBuf);
+   b->posn = PerlIO_tell(PerlIONext(f));
+   if (init && fd == 2)
     {
-     PerlIOBuf *b = PerlIOSelf(PerlIO_push(f,self,NULL),PerlIOBuf);
-     b->posn = PerlIO_tell(PerlIONext(f));
-    }
+     /* Initial stderr is unbuffered */
+     PerlIOBase(f)->flags |= PERLIO_F_UNBUF;
+    } 
+   PerlIO_debug(__FUNCTION__ " %s f=%p fd=%d m=%s fl=%08x\n",
+                self->name,f,fd,mode,PerlIOBase(f)->flags);
   }
  return f;
 }
@@ -1520,7 +1703,7 @@ PerlIOBuf_open(PerlIO_funcs *self, const char *path, const char *mode)
  PerlIO *f = (*tab->Open)(tab,path,mode);
  if (f)
   {
-   PerlIOBuf *b = PerlIOSelf(PerlIO_push(f,self,NULL),PerlIOBuf);
+   PerlIOBuf *b = PerlIOSelf(PerlIO_push(f,self,mode),PerlIOBuf);
    b->posn = PerlIO_tell(PerlIONext(f));
   }
  return f;
@@ -1659,35 +1842,33 @@ PerlIOBuf_fill(PerlIO *f)
 SSize_t
 PerlIOBuf_read(PerlIO *f, void *vbuf, Size_t count)
 {
- PerlIOBuf *b = PerlIOSelf(f,PerlIOBuf);
- STDCHAR *buf = (STDCHAR *) vbuf;
+ PerlIOBuf *b  = PerlIOSelf(f,PerlIOBuf);
+ STDCHAR *buf  = (STDCHAR *) vbuf;
  if (f)
   {
-   Size_t got = 0;
    if (!b->ptr)
     PerlIO_get_base(f);
    if (!(PerlIOBase(f)->flags & PERLIO_F_CANREAD))
     return 0;
    while (count > 0)
     {
-     SSize_t avail = (b->end - b->ptr);
-     if ((SSize_t) count < avail)
-      avail = count;
-     if (avail > 0)
+     SSize_t avail = PerlIO_get_cnt(f);
+     SSize_t take  = (count < avail) ? count : avail;
+     if (take > 0)
       {
-       Copy(b->ptr,buf,avail,STDCHAR);
-       got     += avail;
-       b->ptr  += avail;
-       count   -= avail;
-       buf     += avail;
+       STDCHAR *ptr = PerlIO_get_ptr(f);
+       Copy(ptr,buf,take,STDCHAR);
+       PerlIO_set_ptrcnt(f,ptr+take,(avail -= take));
+       count   -= take;
+       buf     += take;
       }
-     if (count && (b->ptr >= b->end))
+     if (count > 0  && avail <= 0)
       {
        if (PerlIO_fill(f) != 0)
         break;
       }
     }
-   return got;
+   return (buf - (STDCHAR *) vbuf);
   }
  return 0;
 }
@@ -1780,6 +1961,8 @@ PerlIOBuf_write(PerlIO *f, const void *vbuf, Size_t count)
    if (b->ptr >= (b->buf + b->bufsiz))
     PerlIO_flush(f);
   }
+ if (PerlIOBase(f)->flags & PERLIO_F_UNBUF)
+  PerlIO_flush(f);
  return written;
 }
 
@@ -1902,7 +2085,7 @@ PerlIOBuf_set_ptrcnt(PerlIO *f, STDCHAR *ptr, SSize_t cnt)
 PerlIO_funcs PerlIO_perlio = {
  "perlio",
  sizeof(PerlIOBuf),
- 0,
+ PERLIO_K_BUFFERED,
  PerlIOBase_fileno,
  PerlIOBuf_fdopen,
  PerlIOBuf_open,
@@ -1929,27 +2112,295 @@ PerlIO_funcs PerlIO_perlio = {
 };
 
 /*--------------------------------------------------------------------------------------*/
-/* crlf - translation currently just a copy of perlio to prove
-   that extra buffering which real one will do is not an issue.
+/* crlf - translation
+   On read translate CR,LF to "\n" we do this by overriding ptr/cnt entries
+   to hand back a line at a time and keeping a record of which nl we "lied" about.
+   On write translate "\n" to CR,LF
  */
 
+typedef struct
+{
+ PerlIOBuf     base;         /* PerlIOBuf stuff */
+ STDCHAR       *nl;           /* Position of crlf we "lied" about in the buffer */
+} PerlIOCrlf;
+
+IV
+PerlIOCrlf_pushed(PerlIO *f, const char *mode)
+{
+ IV code;
+ PerlIOBase(f)->flags |= PERLIO_F_CRLF;
+ code = PerlIOBase_pushed(f,mode);
+ PerlIO_debug(__FUNCTION__ " f=%p %s %s fl=%08x\n",
+              f,PerlIOBase(f)->tab->name,(mode) ? mode : "(Null)",
+              PerlIOBase(f)->flags); 
+ return code;
+}
+
+
+SSize_t
+PerlIOCrlf_unread(PerlIO *f, const void *vbuf, Size_t count)
+{
+ PerlIOCrlf *c = PerlIOSelf(f,PerlIOCrlf);
+ if (c->nl)
+  {
+   *(c->nl) = 0xd;
+   c->nl = NULL;
+  }
+ if (!(PerlIOBase(f)->flags & PERLIO_F_CRLF))
+  return PerlIOBuf_unread(f,vbuf,count);
+ else
+  {
+   const STDCHAR *buf = (const STDCHAR *) vbuf+count;  
+   PerlIOBuf *b = PerlIOSelf(f,PerlIOBuf);
+   SSize_t unread = 0;
+   if (PerlIOBase(f)->flags & PERLIO_F_WRBUF)
+    PerlIO_flush(f);
+   if (!b->buf)
+    PerlIO_get_base(f);
+   if (b->buf)
+    {
+     if (!(PerlIOBase(f)->flags & PERLIO_F_RDBUF))
+      {
+       b->end = b->ptr = b->buf + b->bufsiz;
+       PerlIOBase(f)->flags |= PERLIO_F_RDBUF;
+      }
+     while (count > 0 && b->ptr > b->buf)
+      {
+       int ch = *--buf;
+       if (ch == '\n')
+        {
+         if (b->ptr - 2 >= b->buf)
+          {
+           *--(b->ptr) = 0xa;
+           *--(b->ptr) = 0xd;
+           unread++;
+           count--;
+          }
+         else
+          {
+           buf++;
+           break;
+          }
+        }
+       else
+        {
+         *--(b->ptr) = ch;
+         unread++;
+         count--;
+        }
+      }
+    }
+   return unread;
+  }
+}
+
+SSize_t
+PerlIOCrlf_get_cnt(PerlIO *f)
+{
+ PerlIOBuf *b = PerlIOSelf(f,PerlIOBuf);
+ if (!b->buf)
+  PerlIO_get_base(f);
+ if (PerlIOBase(f)->flags & PERLIO_F_RDBUF)
+  {
+   PerlIOCrlf *c = PerlIOSelf(f,PerlIOCrlf);
+   if ((PerlIOBase(f)->flags & PERLIO_F_CRLF) && !c->nl)
+    {
+     STDCHAR *nl   = b->ptr;
+    scan:
+     while (nl < b->end && *nl != 0xd)
+      nl++;
+     if (nl < b->end && *nl == 0xd)
+      {
+     test:
+       if (nl+1 < b->end)
+        {
+         if (nl[1] == 0xa)
+          {
+           *nl   = '\n';
+           c->nl = nl;
+          }
+         else
+          {
+           /* Not CR,LF but just CR */
+           nl++;
+           goto scan;
+          }
+        }
+       else
+        {
+         /* Blast - found CR as last char in buffer */
+         if (b->ptr < nl)
+          {
+           /* They may not care, defer work as long as possible */
+           return (nl - b->ptr);
+          }
+         else
+          {
+           int code;
+           dTHX;
+           b->ptr++;               /* say we have read it as far as flush() is concerned */
+           b->buf++;               /* Leave space an front of buffer */
+           b->bufsiz--;            /* Buffer is thus smaller */
+           code = PerlIO_fill(f);  /* Fetch some more */
+           b->bufsiz++;            /* Restore size for next time */
+           b->buf--;               /* Point at space */
+           b->ptr = nl = b->buf;   /* Which is what we hand off */
+           b->posn--;              /* Buffer starts here */
+           *nl = 0xd;              /* Fill in the CR */
+           if (code == 0)
+            goto test;             /* fill() call worked */
+           /* CR at EOF - just fall through */
+          }
+        }
+      }
+    }
+   return (((c->nl) ? (c->nl+1) : b->end) - b->ptr);
+  }
+ return 0;
+}
+
+void
+PerlIOCrlf_set_ptrcnt(PerlIO *f, STDCHAR *ptr, SSize_t cnt)
+{
+ PerlIOBuf *b  = PerlIOSelf(f,PerlIOBuf);
+ PerlIOCrlf *c = PerlIOSelf(f,PerlIOCrlf);
+ IV flags = PerlIOBase(f)->flags;
+ if (!b->buf)
+  PerlIO_get_base(f);
+ if (!ptr)
+  {
+   if (c->nl)
+    ptr = c->nl+1;
+   else
+    {
+     ptr = b->end;
+     if ((flags & PERLIO_F_CRLF) && ptr > b->buf && ptr[-1] == 0xd)
+      ptr--;
+    }
+   ptr -= cnt;
+  }
+ else
+  {
+   /* Test code - delete when it works ... */
+   STDCHAR *chk;
+   if (c->nl)
+    chk = c->nl+1;
+   else
+    {
+     chk = b->end;
+     if ((flags & PERLIO_F_CRLF) && chk > b->buf && chk[-1] == 0xd)
+      chk--;
+    }
+   chk -= cnt;
+   
+   if (ptr != chk)
+    {
+     dTHX;
+     Perl_croak(aTHX_ "ptr wrong %p != %p fl=%08x nl=%p e=%p for %d",
+                ptr, chk, flags, c->nl, b->end, cnt);    
+    }
+  }
+ if (c->nl)
+  {
+   if (ptr > c->nl)
+    {
+     /* They have taken what we lied about */
+     *(c->nl) = 0xd;
+     c->nl = NULL;
+     ptr++;
+    }
+  }
+ b->ptr = ptr;
+ PerlIOBase(f)->flags |= PERLIO_F_RDBUF;
+}
+
+SSize_t
+PerlIOCrlf_write(PerlIO *f, const void *vbuf, Size_t count)
+{
+ if (!(PerlIOBase(f)->flags & PERLIO_F_CRLF))
+  return PerlIOBuf_write(f,vbuf,count);
+ else
+  {
+   PerlIOBuf *b = PerlIOSelf(f,PerlIOBuf); 
+   const STDCHAR *buf  = (const STDCHAR *) vbuf;
+   const STDCHAR *ebuf = buf+count;
+   if (!b->buf)
+    PerlIO_get_base(f);
+   if (!(PerlIOBase(f)->flags & PERLIO_F_CANWRITE))
+    return 0;
+   while (buf < ebuf)
+    {
+     STDCHAR *eptr = b->buf+b->bufsiz;
+     PerlIOBase(f)->flags |= PERLIO_F_WRBUF;
+     while (buf < ebuf && b->ptr < eptr)
+      {
+       if (*buf == '\n')
+        {
+         if ((b->ptr + 2) > eptr)
+          {
+           /* Not room for both */
+           PerlIO_flush(f);
+           break;
+          }
+         else
+          {
+           *(b->ptr)++ = 0xd; /* CR */
+           *(b->ptr)++ = 0xa; /* LF */
+           buf++;
+           if (PerlIOBase(f)->flags & PERLIO_F_LINEBUF)
+            {
+             PerlIO_flush(f);
+             break;
+            }
+          }
+        }
+       else
+        {
+         int ch = *buf++;
+         *(b->ptr)++ = ch;
+        }
+       if (b->ptr >= eptr)
+        {
+         PerlIO_flush(f);
+         break;
+        }
+      }
+    }
+   if (PerlIOBase(f)->flags & PERLIO_F_UNBUF)
+    PerlIO_flush(f);
+   return (buf - (STDCHAR *) vbuf);
+  }
+}
+
+IV
+PerlIOCrlf_flush(PerlIO *f)
+{
+ PerlIOCrlf *c = PerlIOSelf(f,PerlIOCrlf);
+ if (c->nl)
+  {
+   *(c->nl) = 0xd;
+   c->nl = NULL;
+  }
+ return PerlIOBuf_flush(f);
+}
+
 PerlIO_funcs PerlIO_crlf = {
  "crlf",
- sizeof(PerlIOBuf),
- 0,
+ sizeof(PerlIOCrlf),
+ PERLIO_K_BUFFERED|PERLIO_K_CANCRLF,
  PerlIOBase_fileno,
  PerlIOBuf_fdopen,
  PerlIOBuf_open,
  PerlIOBuf_reopen,
- PerlIOBase_pushed,
- PerlIOBase_noop_ok,
- PerlIOBuf_read,
- PerlIOBuf_unread,
- PerlIOBuf_write,
+ PerlIOCrlf_pushed,
+ PerlIOBase_noop_ok,   /* popped */
+ PerlIOBuf_read,       /* generic read works with ptr/cnt lies ... */
+ PerlIOCrlf_unread,    /* Put CR,LF in buffer for each '\n' */
+ PerlIOCrlf_write,     /* Put CR,LF in buffer for each '\n' */
  PerlIOBuf_seek,
  PerlIOBuf_tell,
  PerlIOBuf_close,
- PerlIOBuf_flush,
+ PerlIOCrlf_flush,
  PerlIOBuf_fill,
  PerlIOBase_eof,
  PerlIOBase_error,
@@ -1958,8 +2409,8 @@ PerlIO_funcs PerlIO_crlf = {
  PerlIOBuf_get_base,
  PerlIOBuf_bufsiz,
  PerlIOBuf_get_ptr,
- PerlIOBuf_get_cnt,
- PerlIOBuf_set_ptrcnt,
+ PerlIOCrlf_get_cnt,
+ PerlIOCrlf_set_ptrcnt,
 };
 
 #ifdef HAS_MMAP
@@ -1972,7 +2423,6 @@ typedef struct
  Mmap_t                mptr;        /* Mapped address */
  Size_t                len;          /* mapped length */
  STDCHAR       *bbuf;        /* malloced buffer if map fails */
-
 } PerlIOMmap;
 
 static size_t page_size = 0;
@@ -2240,7 +2690,7 @@ PerlIOMmap_close(PerlIO *f)
 PerlIO_funcs PerlIO_mmap = {
  "mmap",
  sizeof(PerlIOMmap),
- 0,
+ PERLIO_K_BUFFERED,
  PerlIOBase_fileno,
  PerlIOBuf_fdopen,
  PerlIOBuf_open,