<div>Personally I like the name &quot;SetForScope&quot; since the name &quot;scope&quot; states that this value change is tied to C++ scope.</div><div><br><div class="gmail_quote"><div>On Fri, Dec 23, 2016 at 11:32 JF Bastien &lt;<a href="mailto:jfb@chromium.org">jfb@chromium.org</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_msg">&quot;Scope&quot; seems to capture the C++ nature better. </div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">&quot;Temporary&quot; isn&#39;t clear on how long it&#39;ll last IMO. </div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg"><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">On Thu, Dec 22, 2016 at 6:28 PM Saam barati &lt;<a href="mailto:sbarati@apple.com" class="gmail_msg" target="_blank">sbarati@apple.com</a>&gt; wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word" class="gmail_msg">Hi all,<div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">Recently, Yusuke found that JSC and WTF both had the exact same RAII helper data structure. JSC called it SetForScope, and WTF called it TemporaryChange. (Details here: <a href="https://bugs.webkit.org/show_bug.cgi?id=164761" class="gmail_msg" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=164761</a>). Yusuke went to replace JSC’s use of SetForScope with TemporaryChange. When I reviewed his patch, I suggested to him to rename the class to SetForScope because it was my personal preference of the two names. However, I should have first discussed this change with other WebKit developers (so I’m doing that now, retroactively).</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">If there is a strong preference to use the name TemporaryChange instead of SetForScope, I’ll rename the class back to its original name.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">My personal preference is still for the name SetForScope, but my feelings are not strongly tied to one name or the other.</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">- Saam</div></div>_______________________________________________<br class="gmail_msg"><br class="gmail_msg">webkit-dev mailing list<br class="gmail_msg"><br class="gmail_msg"><a href="mailto:webkit-dev@lists.webkit.org" class="gmail_msg" target="_blank">webkit-dev@lists.webkit.org</a><br class="gmail_msg"><br class="gmail_msg"><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br class="gmail_msg"><br class="gmail_msg"></blockquote></div></div><br><br>_______________________________________________<br class="gmail_msg"><br>webkit-dev mailing list<br class="gmail_msg"><br><a href="mailto:webkit-dev@lists.webkit.org" class="gmail_msg" target="_blank">webkit-dev@lists.webkit.org</a><br class="gmail_msg"><br><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev" rel="noreferrer" class="gmail_msg" target="_blank">https://lists.webkit.org/mailman/listinfo/webkit-dev</a><br class="gmail_msg"><br></blockquote></div></div>