RESOLVED FIXED 134337
HIDGamepad should use CFIndex when looping 
https://bugs.webkit.org/show_bug.cgi?id=134337
Summary HIDGamepad should use CFIndex when looping
Dean Jackson
Reported 2014-06-25 22:42:57 PDT
HIDGamepad should use CFIndex when looping
Attachments
Patch (1.43 KB, patch)
2014-06-25 22:45 PDT, Dean Jackson 		mitz: review+
DetailsFormatted DiffDiff
Patch (1.42 KB, patch)
2014-06-25 22:55 PDT, Dean Jackson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dean Jackson
Comment 1 2014-06-25 22:45:46 PDT
Created attachment 233884 [details] Patch
mitz
Comment 2 2014-06-25 22:48:14 PDT
Comment on attachment 233884 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=233884&action=review > Source/WebCore/platform/mac/HIDGamepad.cpp:79 > + CFIndex count = CFArrayGetCount(elements); > + for (CFIndex i = 0; i < count; ++i) { Our usual style for this is for (CFIndex i = 0, count = CFArrayGetCount(elements); i < count; ++i)
Dean Jackson
Comment 3 2014-06-25 22:55:43 PDT
Created attachment 233887 [details] Patch
WebKit Commit Bot
Comment 4 2014-06-25 23:35:48 PDT
Comment on attachment 233887 [details] Patch Clearing flags on attachment: 233887 Committed r170469: <http://trac.webkit.org/changeset/170469>
Note You need to log in before you can comment on or make changes to this bug.