dropping fields (test)

["Jay 'Eraserhead' Felice" <eraserhead@xxxxxxxxxxxxxx>]

Second attemt to send this message, sorry if there are duplicates.

Sorry to get back to you guys on the bug-hunting so late, but a bunch of things
broke over the weekend, including my ability to send and receive mail *sigh*

I looked into the field dropping problems, and the only possibility looks 
like a bug with SSP not properly setting the screen's Modified Data Tag 
(mdt).

Below is a patch which essentially causes the emulator to ignore whether the
Modified Data Tag is set and send all of the fields anyway.  I'd like feedback
on whether this works (I'm 99% sure it will work, the question is whether
that's the right solution).

The patch should apply without errors to both 0.17.x and 0.16.x.  If this
solves the problem, I'll add a proper patch to CVS.

I didn't have the time then to look into other issues then, but I'm looking
into them now.  There are a few updates in the 0.17.x tree which have to
do with the python language support that I did just before looking into
the bugs...  it fixes some compile issues and gets the rest of the python
support maybe 50% done.

-Jay 'Eraserhead' Felice

--- session.c.orig      Sun Jul 15 22:54:42 2001
+++ session.c   Sun Jul 15 22:56:02 2001
@@ -333,8 +333,7 @@
    switch (This->read_opcode) {
    case CMD_READ_INPUT_FIELDS:
       TN5250_ASSERT(aidcode != 0);
-      if (tn5250_dbuffer_mdt(dbuffer) 
-           && tn5250_dbuffer_send_data_for_aid_key(dbuffer,aidcode)) {
+      if (tn5250_dbuffer_send_data_for_aid_key(dbuffer,aidcode)) {
         field = dbuffer->field_list;
         if (field != NULL) {
            do {
@@ -346,14 +345,12 @@
       break;
 
    case CMD_READ_IMMEDIATE:
-      if (tn5250_dbuffer_mdt(dbuffer)) {
-        field = dbuffer->field_list;
-        if (field != NULL) {
-           do {
-              tn5250_session_send_field (This, &field_buf, field);
-              field = field->next;
-           } while (field != dbuffer->field_list);
-        }
+      field = dbuffer->field_list;
+      if (field != NULL) {
+        do {
+           tn5250_session_send_field (This, &field_buf, field);
+           field = field->next;
+        } while (field != dbuffer->field_list);
       }
       break;
 
+---
| This is the LINUX5250 Mailing List!
| To submit a new message, send your mail to LINUX5250@midrange.com.
| To subscribe to this list send email to LINUX5250-SUB@midrange.com.
| To unsubscribe from this list send email to LINUX5250-UNSUB@midrange.com.
| Questions should be directed to the list owner/operator: david@midrange.com
+---