Marcus Meissner : riched20: rcUpdate cannot be NULL in ME_PaintContent ( Coverity).

Alexandre Julliard julliard at winehq.org
Wed Jan 28 08:03:09 CST 2009


Module: wine
Branch: master
Commit: 43eb9ab9d7fd9b3c63b0ca5a768ffcd08041733c
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=43eb9ab9d7fd9b3c63b0ca5a768ffcd08041733c

Author: Marcus Meissner <marcus at jet.franken.de>
Date:   Wed Jan 28 09:35:07 2009 +0100

riched20: rcUpdate cannot be NULL in ME_PaintContent (Coverity).

---

 dlls/riched20/paint.c |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/dlls/riched20/paint.c b/dlls/riched20/paint.c
index 0c027e4..30548fe 100644
--- a/dlls/riched20/paint.c
+++ b/dlls/riched20/paint.c
@@ -74,15 +74,12 @@ void ME_PaintContent(ME_TextEditor *editor, HDC hDC, BOOL bOnlyNew, const RECT *
     if (!bOnlyNew || (item->member.para.nFlags & MEPF_REPAINT))
     {
       /* Draw the pargraph if any of the paragraph is in the update region. */
-      BOOL bPaint = (rcUpdate == NULL);
-      if (rcUpdate)
-        bPaint = ys < rcUpdate->bottom && ye > rcUpdate->top;
-      if (bPaint)
+      if (ys < rcUpdate->bottom && ye > rcUpdate->top)
       {
         ME_DrawParagraph(&c, item);
         /* Clear the repaint flag if the whole paragraph is in the
          * update region. */
-        if (!rcUpdate || (rcUpdate->top <= ys && rcUpdate->bottom >= ye))
+        if (rcUpdate->top <= ys && rcUpdate->bottom >= ye)
           item->member.para.nFlags &= ~MEPF_REPAINT;
       }
     }




More information about the wine-cvs mailing list