[Webkit-unassigned] [Bug 7668] ImageDiff is not endian safe

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=7668


macdome at opendarwin.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|webkit-                     |macdome at opendarwin.org
                   |unassigned at opendarwin.org   |
             Status|NEW                         |ASSIGNED
   Attachment #7050|                            |review?
               Flag|                            |




------- Comment #1 from macdome at opendarwin.org  2006-03-13 12:35 PDT -------
Created an attachment (id=7050)
 --> (http://bugzilla.opendarwin.org/attachment.cgi?id=7050&action=view)
use CG and at least makes the checksum code more robust

So I'm not sure this patch is a final total solution, but it does seem to fix
at least the checksum code.  It seems tha NSBitmapImageRep generally holding
the data in a big-endian format in its buffer (likely because that's how it
read it out of the png).  This foces CG to store the data in host-endian
format, which seems to make MD5 more happy (since it seems to do it's own
swizzling for whatever reason).  I also improved both ImageDiff and the MD5
code to ignore any extra padding data at the end of rows.  ImageDiff is not yet
truely endian safe, but it seems to work.  There are other larger pixel dump
issues now to solve next (like the spacing around ")" on some machines.


-- 
Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are the assignee for the bug, or are watching the assignee.