[ros-diffs] [aandrejevic] 60900: [NTVDM] Instead of repeating the BOP, simply handle both 0x51 and 0x62 as the same function.

[aandrejevic]

Author: aandrejevic
Date: Sat Nov  9 16:15:24 2013
New Revision: 60900

URL: http://svn.reactos.org/svn/reactos?rev=60900&view=rev
Log:
[NTVDM]
Instead of repeating the BOP, simply handle both 0x51 and 0x62 as the same 
function.


Modified:
    branches/ntvdm/subsystems/ntvdm/dos.c

Modified: branches/ntvdm/subsystems/ntvdm/dos.c
URL: 
http://svn.reactos.org/svn/reactos/branches/ntvdm/subsystems/ntvdm/dos.c?rev=60900&r1=60899&r2=60900&view=diff
==============================================================================
--- branches/ntvdm/subsystems/ntvdm/dos.c       [iso-8859-1] (original)
+++ branches/ntvdm/subsystems/ntvdm/dos.c       [iso-8859-1] Sat Nov  9 
16:15:24 2013
@@ -2330,6 +2330,8 @@
 
         /* Internal - Get Current Process ID (Get PSP Address) */
         case 0x51:
+        /* Get Current PSP Address */
+        case 0x62:
         {
             /*
              * Identical to the documented AH=62h.
@@ -2393,24 +2395,6 @@
                 setAX(ERROR_INVALID_FUNCTION);
             }
 
-            break;
-        }
-
-        /* Get Current PSP Address */
-        case 0x62:
-        {
-            /*
-             * Identical to the undocumented AH=51h.
-             * See Ralf Brown: http://www.ctyme.com/intr/rb-3140.htm
-             * for more information.
-             */
-            setAH(0x51); // Call the internal function.
-            /*
-             * Instead of calling ourselves really recursively as in:
-             * DosInt21h(Stack);
-             * prefer resetting the CF flag to let the BOP repeat.
-             */
-            setCF(1);
             break;
         }