commit 0818e1aa2df04f0e336a5055e7d7495b1a042206
author Kevin O'Connor <kevin@koconnor.net> Sat May 16 18:00:19 2009 -0400
committer Kevin O'Connor <kevin@koconnor.net> Sat May 16 18:00:19 2009 -0400
tree c166140ac135e16519ec5f3eb0c620a2aee0e1d1
parent f94ec43133b795049432945e4e6f10b5f0ac423b

VGA: Split biosfn_get_cursor_pos function.

Create biosfn_get_cursor_pos and biosfn_get_cursor_shape.

vgasrc/vga.c

diff --git a/vgasrc/vga.c b/vgasrc/vga.c
index 15cca33..d389692 100644
--- a/vgasrc/vga.c
+++ b/vgasrc/vga.c
@@ -131,6 +131,15 @@
     outb(CL, crtc_addr + 1);
 }
 
+static u16
+biosfn_get_cursor_shape(u8 page)
+{
+    if (page > 7)
+        return 0;
+    // FIXME should handle VGA 14/16 lines
+    return GET_BDA(cursor_type);
+}
+
 // -------------------------------------------------------------------
 static void
 biosfn_set_cursor_pos(u8 page, u16 cursor)
@@ -165,19 +174,13 @@
     outb(address & 0x00ff, crtc_addr + 1);
 }
 
-// -------------------------------------------------------------------
-static void
-biosfn_get_cursor_pos(u8 page, u16 *shape, u16 *pos)
+static u16
+biosfn_get_cursor_pos(u8 page)
 {
-    // Default
-    *shape = 0;
-    *pos = 0;
     if (page > 7)
-        return;
-
+        return 0;
     // FIXME should handle VGA 14/16 lines
-    *shape = GET_BDA(cursor_type);
-    *pos = GET_BDA(cursor_pos[page]);
+    return GET_BDA(cursor_pos[page]);
 }
 
 // -------------------------------------------------------------------
@@ -193,8 +196,7 @@
         return;
 
     // Get pos curs pos for the right page
-    u16 cursor, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &cursor);
+    u16 cursor = biosfn_get_cursor_pos(page);
 
     u16 address;
     if (GET_GLOBAL(vmode_g->class) == TEXT) {
@@ -627,8 +629,7 @@
         return;
 
     // Get the cursor pos for the page
-    u16 cursor, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &cursor);
+    u16 cursor = biosfn_get_cursor_pos(page);
     u8 xcurs = cursor & 0x00ff;
     u8 ycurs = (cursor & 0xff00) >> 8;
 
@@ -773,8 +774,7 @@
         return;
 
     // Get the cursor pos for the page
-    u16 cursor, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &cursor);
+    u16 cursor = biosfn_get_cursor_pos(page);
     u8 xcurs = cursor & 0x00ff;
     u8 ycurs = (cursor & 0xff00) >> 8;
 
@@ -787,7 +787,7 @@
         void *address_far = (void*)(SCREEN_MEM_START(nbcols, nbrows, page)
                                     + (xcurs + ycurs * nbcols) * 2);
 
-        dummy = ((u16)attr << 8) + car;
+        u16 dummy = ((u16)attr << 8) + car;
         memset16_far(GET_GLOBAL(vmode_g->sstart), address_far, dummy, count * 2);
         return;
     }
@@ -824,8 +824,7 @@
         return;
 
     // Get the cursor pos for the page
-    u16 cursor, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &cursor);
+    u16 cursor = biosfn_get_cursor_pos(page);
     u8 xcurs = cursor & 0x00ff;
     u8 ycurs = (cursor & 0xff00) >> 8;
 
@@ -1021,8 +1020,7 @@
         return;
 
     // Get the cursor pos for the page
-    u16 cursor, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &cursor);
+    u16 cursor = biosfn_get_cursor_pos(page);
     u8 xcurs = cursor & 0x00ff;
     u8 ycurs = (cursor & 0xff00) >> 8;
 
@@ -1051,7 +1049,7 @@
     case '\t':
         do {
             biosfn_write_teletype(' ', page, attr, flag);
-            biosfn_get_cursor_pos(page, &dummy, &cursor);
+            cursor = biosfn_get_cursor_pos(page);
             xcurs = cursor & 0x00ff;
             ycurs = (cursor & 0xff00) >> 8;
         } while (xcurs % 8 == 0);
@@ -1588,8 +1586,7 @@
                     u16 seg, u8 *offset_far)
 {
     // Read curs info for the page
-    u16 oldcurs, dummy;
-    biosfn_get_cursor_pos(page, &dummy, &oldcurs);
+    u16 oldcurs = biosfn_get_cursor_pos(page);
 
     // if row=0xff special case : use current cursor position
     if (row == 0xff) {
@@ -1965,7 +1962,8 @@
 static void
 handle_1003(struct bregs *regs)
 {
-    biosfn_get_cursor_pos(regs->bh, &regs->cx, &regs->dx);
+    regs->cx = biosfn_get_cursor_shape(regs->bh);
+    regs->dx = biosfn_get_cursor_pos(regs->bh);
 }
 
 // Read light pen pos (unimplemented)