Move some structs into Connection.cpp
Created attachment 287293 [details] Patch
Comment on attachment 287293 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=287293&action=review > Source/WebKit2/Platform/IPC/Connection.cpp:189 > + uint64_t syncRequestID; Could we initialize this to 0 here? > Source/WebKit2/Platform/IPC/Connection.cpp:196 > + bool didReceiveReply; Could we initialize this to false here? > Source/WebKit2/Platform/IPC/Connection.cpp:202 > + PendingSyncReply() > + : syncRequestID(0) > + , didReceiveReply(false) > + { > + } Would be nice if this was just "= default". > Source/WebKit2/Platform/IPC/Connection.cpp:206 > + , didReceiveReply(0) This says 0 but should say false, but instead could be omitted.
(In reply to comment #2) > Comment on attachment 287293 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=287293&action=review > > > Source/WebKit2/Platform/IPC/Connection.cpp:189 > > + uint64_t syncRequestID; > > Could we initialize this to 0 here? > > > Source/WebKit2/Platform/IPC/Connection.cpp:196 > > + bool didReceiveReply; > > Could we initialize this to false here? > > > Source/WebKit2/Platform/IPC/Connection.cpp:202 > > + PendingSyncReply() > > + : syncRequestID(0) > > + , didReceiveReply(false) > > + { > > + } > > Would be nice if this was just "= default". Fixed all of these, thanks! > > > Source/WebKit2/Platform/IPC/Connection.cpp:206 > > + , didReceiveReply(0) > > This says 0 but should say false, but instead could be omitted.
Committed r205129: <http://trac.webkit.org/changeset/205129>