[webkit-changes] cvs commit: WebKit/WebCoreSupport.subproj WebTextRenderer.m

[Maciej]

mjs         05/12/29 23:05:55

  Modified:    .        ChangeLog
               WebCoreSupport.subproj WebTextRenderer.m
  Log:
          Reviewed by Eric, committed by Maciej.
  
          - fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=6288
            HEAD build broken 12/29/2005
  
          * WebCoreSupport.subproj/WebTextRenderer.m:
          (overrideLayoutOperation): Use previous character from the buffer
          instead of ch which may be uninitialized on the first iteration.
  
  Revision  Changes    Path
  1.3435    +11 -0     WebKit/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebKit/ChangeLog,v
  retrieving revision 1.3434
  retrieving revision 1.3435
  diff -u -r1.3434 -r1.3435
  --- ChangeLog	30 Dec 2005 05:11:38 -0000	1.3434
  +++ ChangeLog	30 Dec 2005 07:05:47 -0000	1.3435
  @@ -1,3 +1,14 @@
  +2005-12-30  Mitz Pettel  <opendarwin.org at mitzpettel.com>
  +
  +        Reviewed by Eric, committed by Maciej.
  +        
  +        - fix for http://bugzilla.opendarwin.org/show_bug.cgi?id=6288
  +          HEAD build broken 12/29/2005
  +
  +        * WebCoreSupport.subproj/WebTextRenderer.m:
  +        (overrideLayoutOperation): Use previous character from the buffer
  +        instead of ch which may be uninitialized on the first iteration.
  +
   2005-12-29  Maciej Stachowiak  <mjs at apple.com>
   
           Reviewed by Eric.
  
  
  
  1.212     +1 -1      WebKit/WebCoreSupport.subproj/WebTextRenderer.m
  
  Index: WebTextRenderer.m
  ===================================================================
  RCS file: /cvs/root/WebKit/WebCoreSupport.subproj/WebTextRenderer.m,v
  retrieving revision 1.211
  retrieving revision 1.212
  diff -u -r1.211 -r1.212
  --- WebTextRenderer.m	29 Dec 2005 11:30:00 -0000	1.211
  +++ WebTextRenderer.m	30 Dec 2005 07:05:54 -0000	1.212
  @@ -286,7 +286,7 @@
                               padding -= params->padPerSpace;
                           }
                       }
  -                    if (offset != 0 && !isSpace(ch) && style->wordSpacing)
  +                    if (offset != 0 && !isSpace(*((UniChar *)(((char *)characters)+offset) - 1)) && style->wordSpacing)
                           width += style->wordSpacing;
                   }
               }