Re: [ros-dev] [ros-diffs] [mkupfer] 39871: Sascha Clausen <r4v3r AT hotmail DOT de> - Draw bullet for menu radio group. - See issue #4193 for details.

[Ged]

Looking at the patch only this looks wrong to me.
Assuming this is radio button it should use the marlett font.

Ged.

-----Original Message-----
From: [e-mail] [mailto:[e-mail]] On Behalf Of [e-mail]
Sent: 04 March 2009 16:30
To: [e-mail]
Subject: [ros-diffs] [mkupfer] 39871: Sascha Clausen <r4v3r AT hotmail DOT de> - Draw bullet for menu radio group. - See issue #4193 for details.

Author: mkupfer
Date: Wed Mar  4 19:29:57 2009
New Revision: 39871

URL: http://svn.reactos.org/svn/reactos?rev=39871&view=rev
Log:
Sascha Clausen <r4v3r AT hotmail DOT de>
- Draw bullet for menu radio group.
- See issue #4193 for details.

Modified:
    trunk/reactos/dll/win32/user32/windows/draw.c

Modified: trunk/reactos/dll/win32/user32/windows/draw.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/user32/windows/dr...
==============================================================================
--- trunk/reactos/dll/win32/user32/windows/draw.c [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/user32/windows/draw.c [iso-8859-1] Wed Mar  4 19:29:57 2009
@@ -1276,11 +1276,12 @@
             yc = myr.top  + SmallDiam - SmallDiam/2;
             i = 234*SmallDiam/750;
             i = i < 1 ? 1 : i;
-            myr.left   = xc - i+i/2;
+            myr.left   = xc - i/2;
             myr.right  = xc + i/2;
-            myr.top    = yc - i+i/2;
+            myr.top    = yc - i/2;
             myr.bottom = yc + i/2;
-            Pie(dc, myr.left, myr.top, myr.right, myr.bottom, xe, ye, xe, ye);
+            // if the start and the end point are equal, Pie() only draws a single line, so start one pixel lower
+            Pie(dc, myr.left, myr.top, myr.right, myr.bottom, xe, ye+1, xe, ye);
             break;

case DFCS_MENUCHECK: