*** empty log message ***
[dana/urxvt.git] / src / rxvtperl.xs
index 0e78f77..a3768a6 100644 (file)
@@ -31,6 +31,8 @@
 #include <cstddef>
 #include <cstdarg>
 
+#include "unistd.h"
+
 #include "iom.h"
 #include "rxvt.h"
 #include "keyboard.h"
@@ -39,7 +41,7 @@
 
 #include "perlxsi.c"
 
-#if defined(HAVE_SCROLLBARS) || defined(MENUBAR)
+#ifdef HAVE_SCROLLBARS
 # define GRAB_CURSOR THIS->leftptr_cursor
 #else
 # define GRAB_CURSOR None
 #undef ROW
 #define ROW(n) THIS->row_buf [LINENO (n)]
 
-/////////////////////////////////////////////////////////////////////////////
-
-static SV *
-taint (SV *sv)
-{
-  SvTAINT (sv);
-  return sv;
-}
-
-static SV *
-taint_if (SV *sv, SV *src)
-{
-  if (SvTAINTED (src))
-    SvTAINT (sv);
+#define ENABLE_PERL_FRILLS 1
 
-  return sv;
-}
+/////////////////////////////////////////////////////////////////////////////
 
 static wchar_t *
 sv2wcs (SV *sv)
@@ -347,8 +335,8 @@ void overlay::swap ()
       text_t *t1 = text [y];
       rend_t *r1 = rend [y];
 
-      text_t *t2 = ROW(y + ov_y - THIS->view_start).t + ov_x;
-      rend_t *r2 = ROW(y + ov_y - THIS->view_start).r + ov_x;
+      text_t *t2 = ROW(y + ov_y + THIS->view_start).t + ov_x;
+      rend_t *r2 = ROW(y + ov_y + THIS->view_start).r + ov_x;
 
       for (int x = ov_w; x--; )
         {
@@ -405,23 +393,30 @@ rxvt_perl_interp::~rxvt_perl_interp ()
 }
 
 void
-rxvt_perl_interp::init ()
+rxvt_perl_interp::init (rxvt_term *term)
 {
   if (!perl)
     {
+      rxvt_push_locale (""); // perl init destroys current locale
+
       perl_environ = rxvt_environ;
       swap (perl_environ, environ);
 
       char *argv[] = {
         "",
-        "-T",
-        "-edo '" LIBDIR "/urxvt.pm' or ($@ and die $@) or exit 1",
+        "-e"
+        "BEGIN {"
+        "   urxvt->bootstrap;"
+        "   unshift @INC, '" LIBDIR "';"
+        "}"
+        ""
+        "use urxvt;"
       };
 
       perl = perl_alloc ();
       perl_construct (perl);
 
-      if (perl_parse (perl, xs_init, 3, argv, (char **)NULL)
+      if (perl_parse (perl, xs_init, 2, argv, (char **)NULL)
           || perl_run (perl))
         {
           rxvt_warn ("unable to initialize perl-interpreter, continuing without.\n");
@@ -432,6 +427,15 @@ rxvt_perl_interp::init ()
         }
 
       swap (perl_environ, environ);
+
+      rxvt_pop_locale ();
+    }
+
+  if (perl)
+    {
+      // runs outside of perls ENV
+      term->perl.self = (void *)newSVptr ((void *)term, "urxvt::term");
+      hv_store ((HV *)SvRV ((SV *)term->perl.self), "_overlay", 8, newRV_noinc ((SV *)newHV ()), 0);
     }
 }
 
@@ -446,191 +450,191 @@ ungrab (rxvt_term *THIS)
     }
 }
 
-bool
-rxvt_perl_interp::invoke (rxvt_term *term, hook_type htype, ...)
+static void
+swap_overlays (rxvt_term *term)
 {
-  if (!perl)
-    return false;
+  HV *hv = (HV *)SvRV (*hv_fetch ((HV *)SvRV ((SV *)term->perl.self), "_overlay", 8, 0));
 
-  if (htype == HOOK_INIT) // first hook ever called
-    {
-      term->perl.self = (void *)newSVptr ((void *)term, "urxvt::term");
-      hv_store ((HV *)SvRV ((SV *)term->perl.self), "_overlay", 8, newRV_noinc ((SV *)newHV ()), 0);
-    }
-  else if (!term->perl.self)
-    return false; // perl not initialized for this instance
-  else if (htype == HOOK_DESTROY)
+  if (HvKEYS (hv))
     {
-      // handled later
-    }
-  else
-    {
-      if (htype == HOOK_REFRESH_BEGIN || htype == HOOK_REFRESH_END)
-        {
-          HV *hv = (HV *)SvRV (*hv_fetch ((HV *)SvRV ((SV *)term->perl.self), "_overlay", 8, 0));
-
-          if (HvKEYS (hv))
-            {
-              hv_iterinit (hv);
+      hv_iterinit (hv);
 
-              while (HE *he = hv_iternext (hv))
-                ((overlay *)SvIV (hv_iterval (hv, he)))->swap ();
-            }
+      while (HE *he = hv_iternext (hv))
+        ((overlay *)SvIV (hv_iterval (hv, he)))->swap ();
+    }
+}
 
-        }
+bool
+rxvt_perl_interp::invoke (rxvt_term *term, hook_type htype, ...)
+{
+  if (!perl || !term->perl.self)
+    return false;
 
-      if (!should_invoke [htype])
-        return false;
-    }
+  // pre-handling of some events
+  if (htype == HOOK_REFRESH_END)
+    swap_overlays (term);
 
   swap (perl_environ, environ);
 
-  try
-    {
-      dSP;
-      va_list ap;
+  bool event_consumed;
 
-      va_start (ap, htype);
+  if (htype == HOOK_INIT || htype == HOOK_DESTROY // must be called always
+      || term->perl.should_invoke [htype])
+    try
+      {
+        dSP;
+        va_list ap;
 
-      ENTER;
-      SAVETMPS;
+        va_start (ap, htype);
 
-      PUSHMARK (SP);
+        ENTER;
+        SAVETMPS;
 
-      XPUSHs (sv_2mortal (newSVterm (term)));
-      XPUSHs (sv_2mortal (newSViv (htype)));
+        PUSHMARK (SP);
 
-      for (;;) {
-        data_type dt = (data_type)va_arg (ap, int);
+        XPUSHs (sv_2mortal (newSVterm (term)));
+        XPUSHs (sv_2mortal (newSViv (htype)));
 
-        switch (dt)
-          {
-            case DT_INT:
-              XPUSHs (sv_2mortal (newSViv (va_arg (ap, int))));
-              break;
+        for (;;) {
+          data_type dt = (data_type)va_arg (ap, int);
 
-            case DT_LONG:
-              XPUSHs (sv_2mortal (newSViv (va_arg (ap, long))));
-              break;
+          switch (dt)
+            {
+              case DT_INT:
+                XPUSHs (sv_2mortal (newSViv (va_arg (ap, int))));
+                break;
 
-            case DT_STR:
-              XPUSHs (taint (sv_2mortal (newSVpv (va_arg (ap, char *), 0))));
-              break;
+              case DT_LONG:
+                XPUSHs (sv_2mortal (newSViv (va_arg (ap, long))));
+                break;
 
-            case DT_STR_LEN:
-              {
-                char *str = va_arg (ap, char *);
-                int len = va_arg (ap, int);
+              case DT_STR:
+                XPUSHs (sv_2mortal (newSVpv (va_arg (ap, char *), 0)));
+                break;
 
-                XPUSHs (taint (sv_2mortal (newSVpvn (str, len))));
-              }
-              break;
+              case DT_STR_LEN:
+                {
+                  char *str = va_arg (ap, char *);
+                  int len = va_arg (ap, int);
 
-            case DT_WCS_LEN:
-              {
-                wchar_t *wstr = va_arg (ap, wchar_t *);
-                int wlen = va_arg (ap, int);
+                  XPUSHs (sv_2mortal (newSVpvn (str, len)));
+                }
+                break;
 
-                XPUSHs (taint (sv_2mortal (wcs2sv (wstr, wlen))));
-              }
-             break;
+              case DT_WCS_LEN:
+                {
+                  wchar_t *wstr = va_arg (ap, wchar_t *);
+                  int wlen = va_arg (ap, int);
 
-            case DT_XEVENT:
-              {
-                XEvent *xe = va_arg (ap, XEvent *);
-                HV *hv = newHV ();
+                  XPUSHs (sv_2mortal (wcs2sv (wstr, wlen)));
+                }
+               break;
+
+              case DT_XEVENT:
+                {
+                  XEvent *xe = va_arg (ap, XEvent *);
+                  HV *hv = newHV ();
 
 #           define set(name, sv) hv_store (hv, # name,  sizeof (# name) - 1, sv, 0)
 #           define setiv(name, val) hv_store (hv, # name,  sizeof (# name) - 1, newSViv (val), 0)
 #           undef set
 
-                setiv (type,       xe->type);
-                setiv (send_event, xe->xany.send_event);
-                setiv (serial,     xe->xany.serial);
-
-                switch (xe->type)
-                  {
-                    case KeyPress:
-                    case KeyRelease:
-                    case ButtonPress:
-                    case ButtonRelease:
-                    case MotionNotify:
-                      setiv (time,   xe->xmotion.time);
-                      setiv (x,      xe->xmotion.x);
-                      setiv (y,      xe->xmotion.y);
-                      setiv (row,    xe->xmotion.y / term->fheight);
-                      setiv (col,    xe->xmotion.x / term->fwidth);
-                      setiv (x_root, xe->xmotion.x_root);
-                      setiv (y_root, xe->xmotion.y_root);
-                      setiv (state,  xe->xmotion.state);
-                      break;
-                  }
-
-                switch (xe->type)
-                  {
-                    case KeyPress:
-                    case KeyRelease:
-                      setiv (keycode, xe->xkey.keycode);
-                      break;
-
-                    case ButtonPress:
-                    case ButtonRelease:
-                      setiv (button,  xe->xbutton.button);
-                      break;
-
-                    case MotionNotify:
-                      setiv (is_hint, xe->xmotion.is_hint);
-                      break;
-                  }
-
-                XPUSHs (sv_2mortal (newRV_noinc ((SV *)hv)));
-              }
-              break;
+                  setiv (type,       xe->type);
+                  setiv (send_event, xe->xany.send_event);
+                  setiv (serial,     xe->xany.serial);
+
+                  switch (xe->type)
+                    {
+                      case KeyPress:
+                      case KeyRelease:
+                      case ButtonPress:
+                      case ButtonRelease:
+                      case MotionNotify:
+                        setiv (time,   xe->xmotion.time);
+                        setiv (x,      xe->xmotion.x);
+                        setiv (y,      xe->xmotion.y);
+                        setiv (row,    xe->xmotion.y / term->fheight);
+                        setiv (col,    xe->xmotion.x / term->fwidth);
+                        setiv (x_root, xe->xmotion.x_root);
+                        setiv (y_root, xe->xmotion.y_root);
+                        setiv (state,  xe->xmotion.state);
+                        break;
+                    }
+
+                  switch (xe->type)
+                    {
+                      case KeyPress:
+                      case KeyRelease:
+                        setiv (keycode, xe->xkey.keycode);
+                        break;
+
+                      case ButtonPress:
+                      case ButtonRelease:
+                        setiv (button,  xe->xbutton.button);
+                        break;
+
+                      case MotionNotify:
+                        setiv (is_hint, xe->xmotion.is_hint);
+                        break;
+                    }
+
+                  XPUSHs (sv_2mortal (newRV_noinc ((SV *)hv)));
+                }
+                break;
+
+              case DT_END:
+                goto call;
+
+              default:
+                rxvt_fatal ("FATAL: unable to pass data type %d\n", dt);
+            }
+        }
 
-            case DT_END:
-              {
-                va_end (ap);
-
-                PUTBACK;
-                int count = call_pv ("urxvt::invoke", G_ARRAY | G_EVAL);
-                SPAGAIN;
-
-                if (count)
-                  {
-                    SV *status = POPs;
-                    count = SvTRUE (status);
-                  }
-
-                PUTBACK;
-                FREETMPS;
-                LEAVE;
-
-                if (SvTRUE (ERRSV))
-                  {
-                    rxvt_warn ("perl hook %d evaluation error: %s", htype, SvPV_nolen (ERRSV));
-                    ungrab (term); // better lose the grab than the session
-                  }
-
-                if (htype == HOOK_DESTROY)
-                  {
-                    clearSVptr ((SV *)term->perl.self);
-                    SvREFCNT_dec ((SV *)term->perl.self);
-                  }
-
-                swap (perl_environ, environ);
-                return count;
-              }
+      call:
+        va_end (ap);
+
+        PUTBACK;
+        int count = call_pv ("urxvt::invoke", G_ARRAY | G_EVAL);
+        SPAGAIN;
+
+        if (count)
+          {
+            SV *status = POPs;
+            count = SvTRUE (status);
+          }
+
+        PUTBACK;
+        FREETMPS;
+        LEAVE;
 
-            default:
-              rxvt_fatal ("FATAL: unable to pass data type %d\n", dt);
+        if (SvTRUE (ERRSV))
+          {
+            rxvt_warn ("perl hook %d evaluation error: %s", htype, SvPV_nolen (ERRSV));
+            ungrab (term); // better lose the grab than the session
           }
+
+        event_consumed = !!count;
       }
-    }
-  catch (...)
+    catch (...)
+      {
+        swap (perl_environ, environ);
+        throw;
+      }
+  else
+    event_consumed = false;
+
+  // post-handling of some events
+  if (htype == HOOK_REFRESH_BEGIN)
+    swap_overlays (term);
+  else if (htype == HOOK_DESTROY)
     {
-      swap (perl_environ, environ);
-      throw;
+      clearSVptr ((SV *)term->perl.self);
+      SvREFCNT_dec ((SV *)term->perl.self);
     }
+
+  swap (perl_environ, environ);
+
+  return event_consumed;
 }
 
 /////////////////////////////////////////////////////////////////////////////
@@ -659,42 +663,107 @@ BOOT:
 # undef def
 
   HV *stash = gv_stashpv ("urxvt", 1);
-# define export_const_iv(name) newCONSTSUB (stash, # name, newSViv (name));
-  export_const_iv (DEFAULT_RSTYLE);
-  export_const_iv (OVERLAY_RSTYLE);
-  export_const_iv (RS_Bold);
-  export_const_iv (RS_Italic);
-  export_const_iv (RS_Blink);
-  export_const_iv (RS_RVid);
-  export_const_iv (RS_Uline);
-
-  export_const_iv (CurrentTime);
-  export_const_iv (ShiftMask);
-  export_const_iv (LockMask);
-  export_const_iv (ControlMask);
-  export_const_iv (Mod1Mask);
-  export_const_iv (Mod2Mask);
-  export_const_iv (Mod3Mask);
-  export_const_iv (Mod4Mask);
-  export_const_iv (Mod5Mask);
-  export_const_iv (Button1Mask);
-  export_const_iv (Button2Mask);
-  export_const_iv (Button3Mask);
-  export_const_iv (Button4Mask);
-  export_const_iv (Button5Mask);
-  export_const_iv (AnyModifier);
-
-  export_const_iv (EVENT_NONE);
-  export_const_iv (EVENT_READ);
-  export_const_iv (EVENT_WRITE);
+  struct {
+    const char *name;
+    IV iv;
+  } *civ, const_iv[] = {
+#   define const_iv(name) { # name, (IV)name }
+    const_iv (DEFAULT_RSTYLE),
+    const_iv (OVERLAY_RSTYLE),
+    const_iv (RS_Bold),
+    const_iv (RS_Italic),
+    const_iv (RS_Blink),
+    const_iv (RS_RVid),
+    const_iv (RS_Uline),
+
+    const_iv (CurrentTime),
+    const_iv (ShiftMask),
+    const_iv (LockMask),
+    const_iv (ControlMask),
+    const_iv (Mod1Mask),
+    const_iv (Mod2Mask),
+    const_iv (Mod3Mask),
+    const_iv (Mod4Mask),
+    const_iv (Mod5Mask),
+    const_iv (Button1Mask),
+    const_iv (Button2Mask),
+    const_iv (Button3Mask),
+    const_iv (Button4Mask),
+    const_iv (Button5Mask),
+    const_iv (AnyModifier),
+
+    const_iv (EVENT_NONE),
+    const_iv (EVENT_READ),
+    const_iv (EVENT_WRITE),
+
+    const_iv (NoEventMask),
+    const_iv (KeyPressMask),
+    const_iv (KeyReleaseMask),
+    const_iv (ButtonPressMask),
+    const_iv (ButtonReleaseMask),
+    const_iv (EnterWindowMask),
+    const_iv (LeaveWindowMask),
+    const_iv (PointerMotionMask),
+    const_iv (PointerMotionHintMask),
+    const_iv (Button1MotionMask),
+    const_iv (Button2MotionMask),
+    const_iv (Button3MotionMask),
+    const_iv (Button4MotionMask),
+    const_iv (Button5MotionMask),
+    const_iv (ButtonMotionMask),
+    const_iv (KeymapStateMask),
+    const_iv (ExposureMask),
+    const_iv (VisibilityChangeMask),
+    const_iv (StructureNotifyMask),
+    const_iv (ResizeRedirectMask),
+    const_iv (SubstructureNotifyMask),
+    const_iv (SubstructureRedirectMask),
+    const_iv (FocusChangeMask),
+    const_iv (PropertyChangeMask),
+    const_iv (ColormapChangeMask),
+    const_iv (OwnerGrabButtonMask),
+
+    const_iv (KeyPress),
+    const_iv (KeyRelease),
+    const_iv (ButtonPress),
+    const_iv (ButtonRelease),
+    const_iv (MotionNotify),
+    const_iv (EnterNotify),
+    const_iv (LeaveNotify),
+    const_iv (FocusIn),
+    const_iv (FocusOut),
+    const_iv (KeymapNotify),
+    const_iv (Expose),
+    const_iv (GraphicsExpose),
+    const_iv (NoExpose),
+    const_iv (VisibilityNotify),
+    const_iv (CreateNotify),
+    const_iv (DestroyNotify),
+    const_iv (UnmapNotify),
+    const_iv (MapNotify),
+    const_iv (MapRequest),
+    const_iv (ReparentNotify),
+    const_iv (ConfigureNotify),
+    const_iv (ConfigureRequest),
+    const_iv (GravityNotify),
+    const_iv (ResizeRequest),
+    const_iv (CirculateNotify),
+    const_iv (CirculateRequest),
+    const_iv (PropertyNotify),
+    const_iv (SelectionClear),
+    const_iv (SelectionRequest),
+    const_iv (SelectionNotify),
+    const_iv (ColormapNotify),
+    const_iv (ClientMessage),
+    const_iv (MappingNotify),
+  };
+
+  for (civ = const_iv + sizeof (const_iv) / sizeof (const_iv [0]);
+       civ-- > const_iv; )
+    newCONSTSUB (stash, (char *)civ->name, newSViv (civ->iv));
 }
 
 void
-set_should_invoke (int htype, int value)
-       CODE:
-        rxvt_perl.should_invoke [htype] = value;
-
-void
 warn (const char *msg)
        CODE:
         rxvt_warn ("%s", msg);
@@ -704,20 +773,8 @@ fatal (const char *msg)
        CODE:
         rxvt_fatal ("%s", msg);
 
-SV *
-untaint (SV *sv)
-       CODE:
-        RETVAL = newSVsv (sv);
-        SvTAINTED_off (RETVAL);
-        OUTPUT:
-        RETVAL
-
-bool
-safe ()
-       CODE:
-        RETVAL = !rxvt_tainted ();
-        OUTPUT:
-        RETVAL
+void
+_exit (int status)
 
 NV
 NOW ()
@@ -823,6 +880,100 @@ _new (...)
 void
 rxvt_term::destroy ()
 
+#if ENABLE_PERL_FRILLS
+
+void
+rxvt_term::XListProperties (U32 window)
+       PPCODE:
+{
+       int count;
+       Atom *props = XListProperties (THIS->display->display, (Window)window, &count);
+
+        EXTEND (SP, count);
+        while (count--)
+          PUSHs (newSVuv ((U32)props [count]));
+        
+        XFree (props);
+}
+
+void
+rxvt_term::XGetWindowProperty (U32 window, U32 property)
+       PPCODE:
+{
+        Atom type;
+        int format;
+        unsigned long nitems;
+        unsigned long bytes_after;
+        unsigned char *prop;
+       XGetWindowProperty (THIS->display->display, (Window)window, (Atom)property,
+                            0, 1<<30, 0, AnyPropertyType,
+                            &type, &format, &nitems, &bytes_after, &prop);
+        if (type != None)
+          {
+            EXTEND (SP, 3);
+            PUSHs (newSVuv ((U32)type));
+            PUSHs (newSViv (format));
+            PUSHs (newSVpvn ((char *)prop, nitems * format / 8));
+            XFree (prop);
+          }
+}
+
+void
+rxvt_term::XChangeWindowProperty (U32 window, U32 property, U32 type, int format, SV *data)
+       CODE:
+{
+       STRLEN len;
+        char *data_ = SvPVbyte (data, len);
+
+       XChangeProperty (THIS->display->display, (Window)window, (Atom)property,
+                         type, format, PropModeReplace,
+                         (unsigned char *)data, len * 8 / format);
+}
+
+void
+rxvt_term::XDeleteProperty (U32 window, U32 property)
+       CODE:
+        XDeleteProperty (THIS->display->display, (Window)window, (Atom)property);
+
+U32
+rxvt_term::DefaultRootWindow ()
+       CODE:
+        RETVAL = (U32)THIS->display->root;
+        OUTPUT:
+        RETVAL
+
+U32
+rxvt_term::XCreateSimpleWindow (U32 parent, int x, int y, unsigned int width, unsigned int height)
+       CODE:
+        RETVAL = XCreateSimpleWindow (THIS->display->display, (Window)parent,
+                                      x, y, width, height, 0,
+                                      THIS->pix_colors_focused[Color_border],
+                                      THIS->pix_colors_focused[Color_border]);
+       OUTPUT:
+        RETVAL
+
+void
+rxvt_term::XReparentWindow (U32 window, U32 parent, int x = 0, int y = 0)
+       CODE:
+        XReparentWindow (THIS->display->display, (Window)window, (Window)parent, x, y);
+
+void
+rxvt_term::XMapWindow (U32 window)
+       CODE:
+        XMapWindow (THIS->display->display, (Window)window);
+
+void
+rxvt_term::XUnmapWindow (U32 window)
+       CODE:
+        XUnmapWindow (THIS->display->display, (Window)window);
+
+#endif
+
+void
+rxvt_term::set_should_invoke (int htype, int inc)
+       CODE:
+        THIS->perl.should_invoke [htype] += inc;
+
 void
 rxvt_term::grab_button (int button, U32 modifiers)
        CODE:
@@ -899,7 +1050,7 @@ rxvt_term::locale_encode (SV *str)
 
         free (wstr);
 
-        RETVAL = taint_if (newSVpv (mbstr, 0), str);
+        RETVAL = newSVpv (mbstr, 0);
         free (mbstr);
 }
        OUTPUT:
@@ -916,7 +1067,7 @@ rxvt_term::locale_decode (SV *octets)
         wchar_t *wstr = rxvt_mbstowcs (data, len);
         rxvt_pop_locale ();
 
-        RETVAL = taint_if (wcs2sv (wstr), octets);
+        RETVAL = wcs2sv (wstr);
         free (wstr);
 }
        OUTPUT:
@@ -935,7 +1086,7 @@ rxvt_term::locale_decode (SV *octets)
 #define TERM_OFFSET_mapped     TERM_OFFSET(mapped)
 #define TERM_OFFSET_saveLines  TERM_OFFSET(saveLines)
 #define TERM_OFFSET_total_rows TERM_OFFSET(total_rows)
-#define TERM_OFFSET_nsaved     TERM_OFFSET(nsaved)
+#define TERM_OFFSET_top_row    TERM_OFFSET(top_row)
 
 int
 rxvt_term::width ()
@@ -951,7 +1102,7 @@ rxvt_term::width ()
            mapped     = TERM_OFFSET_mapped
            saveLines  = TERM_OFFSET_saveLines
            total_rows = TERM_OFFSET_total_rows
-           nsaved     = TERM_OFFSET_nsaved
+           top_row    = TERM_OFFSET_top_row
        CODE:
         RETVAL = *(int *)((char *)THIS + ix);
         OUTPUT:
@@ -1030,27 +1181,29 @@ rxvt_term::vt ()
         OUTPUT:
         RETVAL
 
+void
+rxvt_term::vt_emask_add (U32 emask)
+       CODE:
+        THIS->vt_emask_perl |= emask;
+        THIS->vt_select_input ();
+
 U32
 rxvt_term::rstyle (U32 new_rstyle = THIS->rstyle)
        CODE:
-{
         RETVAL = THIS->rstyle;
         THIS->rstyle = new_rstyle;
-}
         OUTPUT:
        RETVAL
 
 int
-rxvt_term::view_start (int newval = -1)
+rxvt_term::view_start (int newval = 1)
+       PROTOTYPE: $;$
        CODE:
 {
         RETVAL = THIS->view_start;
 
-        if (newval >= 0)
-          {
-            THIS->view_start = min (newval, THIS->nsaved);
-            THIS->scr_changeview (RETVAL);
-          }
+        if (newval <= 0)
+          THIS->scr_changeview (max (newval, THIS->top_row));
 }
         OUTPUT:
        RETVAL
@@ -1064,7 +1217,7 @@ void
 rxvt_term::ROW_t (int row_number, SV *new_text = 0, int start_col = 0, int start_ofs = 0, int max_len = MAX_COLS)
        PPCODE:
 {
-        if (!IN_RANGE_EXC (row_number, -THIS->nsaved, THIS->nrow))
+        if (!IN_RANGE_EXC (row_number, THIS->top_row, THIS->nrow))
           XSRETURN_EMPTY;
 
         line_t &l = ROW(row_number);
@@ -1076,7 +1229,7 @@ rxvt_term::ROW_t (int row_number, SV *new_text = 0, int start_col = 0, int start
             for (int col = 0; col < THIS->ncol; col++)
               wstr [col] = l.t [col];
 
-            XPUSHs (taint (sv_2mortal (wcs2sv (wstr, THIS->ncol))));
+            XPUSHs (sv_2mortal (wcs2sv (wstr, THIS->ncol)));
 
             delete [] wstr;
           }
@@ -1107,7 +1260,7 @@ void
 rxvt_term::ROW_r (int row_number, SV *new_rend = 0, int start_col = 0, int start_ofs = 0, int max_len = MAX_COLS)
        PPCODE:
 {
-        if (!IN_RANGE_EXC (row_number, -THIS->nsaved, THIS->nrow))
+        if (!IN_RANGE_EXC (row_number, THIS->top_row, THIS->nrow))
           XSRETURN_EMPTY;
 
         line_t &l = ROW(row_number);
@@ -1147,7 +1300,7 @@ int
 rxvt_term::ROW_l (int row_number, int new_length = -1)
        CODE:
 {
-        if (!IN_RANGE_EXC (row_number, -THIS->nsaved, THIS->nrow))
+        if (!IN_RANGE_EXC (row_number, THIS->top_row, THIS->nrow))
           XSRETURN_EMPTY;
 
         line_t &l = ROW(row_number);
@@ -1163,7 +1316,7 @@ bool
 rxvt_term::ROW_is_longer (int row_number, int new_is_longer = -1)
        CODE:
 {
-        if (!IN_RANGE_EXC (row_number, -THIS->nsaved, THIS->nrow))
+        if (!IN_RANGE_EXC (row_number, THIS->top_row, THIS->nrow))
           XSRETURN_EMPTY;
 
         line_t &l = ROW(row_number);
@@ -1207,7 +1360,7 @@ rxvt_term::special_encode (SV *string)
 
        rxvt_pop_locale ();
 
-        RETVAL = taint_if (wcs2sv (rstr, r - rstr), string);
+        RETVAL = wcs2sv (rstr, r - rstr);
 
         delete [] rstr;
 }
@@ -1243,7 +1396,7 @@ rxvt_term::special_decode (SV *text)
           else
             *r++ = *s;
 
-        RETVAL = taint_if (wcs2sv (rstr, r - rstr), text);
+        RETVAL = wcs2sv (rstr, r - rstr);
 
         delete [] rstr;
 }
@@ -1275,7 +1428,7 @@ rxvt_term::_resource (char *name, int index, SV *newval = 0)
           croak ("requested out-of-bound resource %s+%d,", name, index - rs->value);
 
         if (GIMME_V != G_VOID)
-          XPUSHs (THIS->rs [index] ? sv_2mortal (taint (newSVpv (THIS->rs [index], 0))) : &PL_sv_undef);
+          XPUSHs (THIS->rs [index] ? sv_2mortal (newSVpv (THIS->rs [index], 0)) : &PL_sv_undef);
 
         if (newval)
           {
@@ -1292,8 +1445,6 @@ rxvt_term::_resource (char *name, int index, SV *newval = 0)
 
 const char *
 rxvt_term::x_resource (const char *name)
-       CLEANUP:
-        SvTAINTED_on (ST (0));
 
 bool
 rxvt_term::option (U32 optval, int set = -1)
@@ -1363,7 +1514,7 @@ rxvt_term::screen_cur (...)
 
         if (items == 3)
           {
-            rc.row = clamp (SvIV (ST (1)), -THIS->nsaved, THIS->nrow - 1);
+            rc.row = clamp (SvIV (ST (1)), THIS->top_row, THIS->nrow - 1);
             rc.col = clamp (SvIV (ST (2)), 0, THIS->ncol - 1);
 
             if (ix)
@@ -1378,6 +1529,16 @@ rxvt_term::cur_charset ()
        OUTPUT:
         RETVAL
 
+#void
+#rxvt_term::selection_clear ()
+
+void
+rxvt_term::selection_make (U32 eventtime, bool rect = false)
+       CODE:
+        THIS->selection.op = SELECTION_CONT;
+        THIS->selection.rect = rect;
+        THIS->selection_make (eventtime);
+
 int
 rxvt_term::selection_grab (U32 eventtime)
 
@@ -1387,7 +1548,7 @@ rxvt_term::selection (SV *newtext = 0)
 {
         if (GIMME_V != G_VOID)
           XPUSHs (THIS->selection.text
-                  ? taint (sv_2mortal (wcs2sv (THIS->selection.text, THIS->selection.len)))
+                  ? sv_2mortal (wcs2sv (THIS->selection.text, THIS->selection.len))
                   : &PL_sv_undef);
 
         if (newtext)