# HG changeset patch
# User Sam Lantinga <slouken@libsdl.org>
# Date 1260180504 0
# Node ID b403f790df6547e9fe9deb531f946450622a6b83
# Parent  9d129e1d0782fce684538c4036938ea76677d505
Hey, those automated tests are coming in handy!  Fixed GDI rendering semantics

diff -r 9d129e1d0782 -r b403f790df65 src/video/win32/SDL_gdirender.c
--- a/src/video/win32/SDL_gdirender.c	Mon Dec 07 09:44:55 2009 +0000
+++ b/src/video/win32/SDL_gdirender.c	Mon Dec 07 10:08:24 2009 +0000
@@ -743,6 +743,10 @@
     status = Polyline(data->current_hdc, points, 2);
     DeleteObject(pen);
 
+    /* Need to close the endpoint of the line */
+    SetPixel(data->current_hdc, x2, y2,
+             RGB(renderer->r, renderer->g, renderer->b));
+
     if (!status) {
         WIN_SetError("FillRect()");
         return -1;
@@ -764,8 +768,8 @@
 
     rc.left = rect->x;
     rc.top = rect->y;
-    rc.right = rect->x + rect->w + 1;
-    rc.bottom = rect->y + rect->h + 1;
+    rc.right = rect->x + rect->w;
+    rc.bottom = rect->y + rect->h;
 
     /* Should we cache the brushes? .. it looks like GDI does for us. :) */
     brush = CreateSolidBrush(RGB(renderer->r, renderer->g, renderer->b));