RESOLVED FIXED 7618
use unsigned bitfields when storing enums in them, to avoid mangling on win32 (was causing all layout to be RTL) 
https://bugs.webkit.org/show_bug.cgi?id=7618
Summary use unsigned bitfields when storing enums in them, to avoid mangling on win32...
Maciej Stachowiak
Reported 2006-03-05 17:53:19 PST
In MSVC, enums are signed, as are bitfields containing them. This often makes values at the extreme end of the enum range go negative, so they fail equality comparisons. I fixed this by always using unsigned for bitfields that hold enums, unless the enum actually uses negative values.
Attachments
take 1 (kinda ugly) (87.89 KB, patch)
2006-03-05 17:55 PST, Maciej Stachowiak 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2006-03-05 17:55:24 PST
Created attachment 6880 [details] take 1 (kinda ugly)
Darin Adler
Comment 2 2006-03-05 20:17:57 PST
Comment on attachment 6880 [details] take 1 (kinda ugly) r=me I didn't spot any problems. The trick with a change like this is to do enough testing.
Darin Adler
Comment 3 2006-03-05 20:23:36 PST
Maciej landed this.
Note You need to log in before you can comment on or make changes to this bug.