Fix coding style of Intl code according to Darin's comments in https://bugs.webkit.org/show_bug.cgi?id=147604 This patch does three things: 1. Rename pointers and references to ExecState from "exec" to "state". 2. Pass parameters by references instead of pointers if the parameters are required. 3. Remove the word "get" from the names of functions that don't return values through out arguments. This should make future Intl patches smaller.
Created attachment 265950 [details] Patch
Comment on attachment 265950 [details] Patch Hmmm, I know Darin said to use "state" instead of "exec" for the ExecState*, but I wonder if we should do that. Using "exec" to mean "ExecState*" is universally practiced in JSC code. Changing to use "state" in just some places would actually make it harder to read.
(In reply to comment #2) > Hmmm, I know Darin said to use "state" instead of "exec" for the ExecState*, > but I wonder if we should do that. Using "exec" to mean "ExecState*" is > universally practiced in JSC code. Changing to use "state" in just some > places would actually make it harder to read. If you look you will see that we are starting to change "exec" to "state" in much new code, at least in WebCore.
(In reply to comment #3) > (In reply to comment #2) > > Hmmm, I know Darin said to use "state" instead of "exec" for the ExecState*, > > but I wonder if we should do that. Using "exec" to mean "ExecState*" is > > universally practiced in JSC code. Changing to use "state" in just some > > places would actually make it harder to read. > > If you look you will see that we are starting to change "exec" to "state" in > much new code, at least in WebCore.
Sigh ... such a rookie mistake (clicked Save before adding comment) ... (In reply to comment #3) > If you look you will see that we are starting to change "exec" to "state" in > much new code, at least in WebCore. OK. I'm good with this if this is the new direction we're taking.
There might be people who don’t agree with me, but I think it’s a good change that we should eventually do everywhere.
My vote would be to rename all ExecState to CallFrame. And we have a natural name for the variable: (CallFrame* callFrame)
Comment on attachment 265950 [details] Patch Clearing flags on attachment: 265950 Committed r192831: <http://trac.webkit.org/changeset/192831>
All reviewed patches have been landed. Closing bug.