[webkit-dev] Functionality of create_activation and tear_off_activation

2013-06-18 Thread Abhishek Bichhawat
Hi, We are working on a research project involving webkit. We are currently using Webkit-r122160 and trying to understand the functionality of two of the opcodes  (op_create_activation and op_tear_off_activation), which are a part of the classical interpreter. Although, the classical

[webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Darin Adler
Lets bike shed! For some time, functions with names like fooIfExists and ensureFoo have been bothering me. I find both names kind of opaque and unpleasant. Here’s an example: StyleResolver* styleResolverIfExists(); StyleResolver* ensureStyleResolver() What do you think of these names

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Emil A Eklund
On Tue, Jun 18, 2013 at 6:38 PM, Darin Adler da...@apple.com wrote: What do you think of these names instead? StyleResolver* optionalStyleResolver(); StyleResolver requiredStyleResolver(); +1, much clearer and the pointer vs reference makes it even more so. Perhaps enough so that the

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Ryosuke Niwa
On Tue, Jun 18, 2013 at 6:38 PM, Darin Adler da...@apple.com wrote: Lets bike shed! For some time, functions with names like fooIfExists and ensureFoo have been bothering me. I find both names kind of opaque and unpleasant. Here’s an example: StyleResolver* styleResolverIfExists();

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Darin Adler
On Jun 18, 2013, at 7:05 PM, Ryosuke Niwa rn...@webkit.org wrote: requiredStyleResolver sounds as if it's a special (required) type of a style resolver as opposed to the caller requiring it. That is true. Why don't we call it requireStyleResolver() instead? Functions with return values

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Darin Adler
On Jun 18, 2013, at 7:05 PM, Ryosuke Niwa rn...@webkit.org wrote: Why don't we call it requireStyleResolver() instead? I’m warming to this idea. Maybe we can use “require” as a term of art, analogous to the way we use “create”, to mean “create if not already created”. -- Darin

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Simon Fraser
On Jun 18, 2013, at 7:11 PM, Darin Adler da...@apple.com wrote: On Jun 18, 2013, at 7:05 PM, Ryosuke Niwa rn...@webkit.org wrote: Why don't we call it requireStyleResolver() instead? I’m warming to this idea. Maybe we can use “require” as a term of art, analogous to the way we use

[webkit-dev] There is a linking error when build webkit on windows 7

2013-06-18 Thread siqinbilige
Hi, I wan to build webkit on Windows. My environment is : Windows 7 Ultimate (Service Pack1) 64bit (japanese) Intel(R) Core(TM) i7 CPU / 12.0GB Memory VC++ 2010 Express Microsoft Windows SDK for windows 7(7.1) Safari 5.1.7 When I run build-webkit, first failed with libs. 1LINK :

Re: [webkit-dev] There is a linking error when build webkit on windows 7

2013-06-18 Thread Brent Fulgham
Hi SiqinBilige: On Jun 18, 2013, at 8:03 PM, siqinbilige siqinbil...@gmail.com wrote: I wan to build webkit on Windows. My environment is : Windows 7 Ultimate (Service Pack1) 64bit (japanese) Intel(R) Core(TM) i7 CPU / 12.0GB Memory VC++ 2010 Express Microsoft Windows SDK for windows

Re: [webkit-dev] There is a linking error when build webkit on windows 7

2013-06-18 Thread ビリゴ スチン
Hi brent, Thank you for your response. All of that should be fine... When I run build-webkit, first failed with libs. 1LINK : fatal error LNK1181: 入力ファイル 'libicuuc.lib' を開けません。 First clue above... 1Base64.obj : error LNK2019: 未解決の外部シンボル _u_charDirection_46 が関数 enum

Re: [webkit-dev] There is a linking error when build webkit on windows 7

2013-06-18 Thread Brent Fulgham
Hi Siqinbilige: On Jun 18, 2013, at 8:31 PM, ビリゴ スチン siqinbil...@gmail.com wrote: Yes, There is a WebKitLibraries/win/lib32 folder and also have WebKitLibraries/win/lib folder. first, there is one file WebKitSystemInterface.lib. I copied the all of the files which including libicuuc.lib

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Maciej Stachowiak
If the semantic is essentially that of a getter that just happens to lazily create what it gets on demand, then I don't think require or required is needed. It can just be named as a getter. If the side effect is very important and especially if clients ever call the function only for its side

Re: [webkit-dev] For your consideration: Naming scheme for fooIfExists/ensureFoo

2013-06-18 Thread Ryosuke Niwa
On Tue, Jun 18, 2013 at 7:20 PM, Simon Fraser simon.fra...@apple.comwrote: On Jun 18, 2013, at 7:11 PM, Darin Adler da...@apple.com wrote: On Jun 18, 2013, at 7:05 PM, Ryosuke Niwa rn...@webkit.org wrote: Why don't we call it requireStyleResolver() instead? I’m warming to this idea. Maybe