Re: [LINUX5250] destructive backspace patch (was: I'm back! ( for backspace))

[Brian McGrogan <brian@xxxxxxxxxx>]

Hello,

I have the same problem.  My backspace did nothing, when it should have
sent a destructive delete.  Here is a patch I wrote which solves this
issue, which was applied to 0.16.4 source but it should work on newer
source with little tuning, let me know if you have problems.

-brian




--- ../../tn5250-0.16.4/src/display.c   Wed Mar 20 03:50:35 2002
+++ display.c   Sun Sep 15 21:41:06 2002
@@ -661,7 +694,7 @@
    /* Make sure this is a valid data character for this field type. */
    if (!tn5250_field_valid_char(field, ch)) {
       TN5250_LOG (("Inhibiting: invalid character for field type.\n"));
-      tn5250_display_inhibit(This);
+      //tn5250_display_inhibit(This);
       return;
    }
    /* Are we at the last character of the field? */
@@ -863,21 +896,22 @@

    /* FIXME: Translate from terminal key via keyboard map to 5250 key. */

-   if (tn5250_display_inhibited(This)) {
-      if (key != K_SYSREQ && key != K_RESET) {
-        tn5250_display_beep (This);
-        return;
-      }
-   }
+//   if (tn5250_display_inhibited(This)) {
+//      if (key != K_SYSREQ && key != K_RESET) {
+//      tn5250_display_beep (This);
+//      return;
+//      }
+//   }

    /* In the case we are in the field exit required state, we inhibit on
     * everything except left arrow, backspace, field exit, field+, and
     * field- */
   if ((tn5250_display_indicators (This) & TN5250_DISPLAY_IND_FER) != 0) {
       switch (key) {
       case K_LEFT:
       case K_BACKSPACE:
         tn5250_display_indicator_clear (This, TN5250_DISPLAY_IND_FER);
+        tn5250_display_kf_delete (This);   // send a DELETE
         return;

       case K_UP:
@@ -909,6 +943,7 @@

    case K_BACKSPACE:
       tn5250_display_kf_backspace (This);
+      tn5250_display_kf_delete (This); // send a delete
       break;

    case K_LEFT:


On Thu, 28 Aug 2003 Phillip.Watts@xxxxxxxxx wrote:

>
>
> Got another request from production.
>
> Their definition of backspace is it backs over characters
> until it reaches the left end of the field then goes to the right
> end of the field above.
>
> My backspace does nothing.
> I notice it works fine on console tn5250 but not xt5250.
> Keymapping suggestion?  in Xdefaults?
> Thanx.
>
>
> _______________________________________________
> This is the Linux 5250 Development Project (LINUX5250) mailing list
> To post a message email: LINUX5250@xxxxxxxxxxxx
> To subscribe, unsubscribe, or change list options,
> visit: http://lists.midrange.com/mailman/listinfo/linux5250
> or email: LINUX5250-request@xxxxxxxxxxxx
> Before posting, please take a moment to review the archives
> at http://archive.midrange.com/linux5250.
>
>