Re: [webkit-dev] Naming preference: SetForScope vs. TemporaryChange

2016-12-22 Thread Yusuke SUZUKI
Personally I like the name "SetForScope" since the name "scope" states that this value change is tied to C++ scope. On Fri, Dec 23, 2016 at 11:32 JF Bastien wrote: > "Scope" seems to capture the C++ nature better. > > "Temporary" isn't clear on how long it'll last IMO. > > >

Re: [webkit-dev] Naming preference: SetForScope vs. TemporaryChange

2016-12-22 Thread JF Bastien
"Scope" seems to capture the C++ nature better. "Temporary" isn't clear on how long it'll last IMO. On Thu, Dec 22, 2016 at 6:28 PM Saam barati wrote: > Hi all, > > Recently, Yusuke found that JSC and WTF both had the exact same RAII > helper data structure. JSC called it

[webkit-dev] Naming preference: SetForScope vs. TemporaryChange

2016-12-22 Thread Saam barati
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 ). Yusuke went to