RESOLVED FIXED Bug 120040
Save three bytes per CStringBuffer object 
https://bugs.webkit.org/show_bug.cgi?id=120040
Summary Save three bytes per CStringBuffer object
Ryosuke Niwa
Reported 2013-08-19 17:27:12 PDT
Merge https://chromium.googlesource.com/chromium/blink/+/894ae8eafdb64912aefd8f9c809f4ccda84f3b89 sizeof(CStringBuffer) was rounded up to 8 on account of struct size and alignment rules. This is clearly not what was intended. Also tidy up includes.
Attachments
Patch (3.02 KB, patch)
2013-08-23 13:45 PDT, Benjamin Poulain 		darin: review+
gtk-ews: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Benjamin Poulain
Comment 1 2013-08-23 13:45:48 PDT
Created attachment 209506 [details] Patch
Darin Adler
Comment 2 2013-08-23 13:47:40 PDT
Comment on attachment 209506 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=209506&action=review > Source/WTF/wtf/text/CString.h:49 > + char* mutableData() { return reinterpret_cast<char*>(this + 1); } Might need to use reinterpret_cast_ptr for the benefit of some ARM-based platforms.
kov's GTK+ EWS bot
Comment 3 2013-08-23 15:13:40 PDT
Comment on attachment 209506 [details] Patch Attachment 209506 [details] did not pass gtk-wk2-ews (gtk-wk2): Output: http://webkit-queues.appspot.com/results/1543435
Benjamin Poulain
Comment 4 2013-08-24 21:38:24 PDT
Committed r154565: <http://trac.webkit.org/changeset/154565>
Note You need to log in before you can comment on or make changes to this bug.