Hi all,
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: https://bugs.webkit.org/show_bug.cgi?id=164761
<https://bugs.webkit.org/show_bug.cgi?id=164761>). 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).
If there is a strong preference to use the name TemporaryChange instead of
SetForScope, I’ll rename the class back to its original name.
My personal preference is still for the name SetForScope, but my feelings are
not strongly tied to one name or the other.
- Saam
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-dev
