Thanks for the link to the prototype code!
If I understand it right and the DatabaseManager can not have any other
relationship with ScriptExecutionContext but 1-1 and their lifetime is the
same then it's not clear to me what benefit could be had by effectively
splitting a class in two. The fact t
On Wed, Dec 9, 2009 at 4:11 PM, Dmitry Titov wrote:
> On Wed, Dec 9, 2009 at 12:58 PM, Eric Uhrhane wrote:
>>
>> I've pulled the database-related members out of Document and made a
>> new class for them, DatabaseManager. An instance of that is owned by
>> each ScriptExecutionContext. There are
On Wed, Dec 9, 2009 at 4:47 PM, Adam Treat wrote:
> On Wednesday 09 December 2009 07:03:51 pm Peter Kasting wrote:
> > And I don't. Who is right? More importantly, how will you prevent us
> from
> > starting this debate on a bug?
>
> I won't? If you as a reviewer consider the indentation of ca
On Wednesday 09 December 2009 07:03:51 pm Peter Kasting wrote:
> But even what is trivial is a judgement call. In general people don't
> disagree about issues where they believe disagreement is a waste of time.
...
> And I don't. Who is right? More importantly, how will you prevent us from
> sta
At first glance these "meta-guidelines" seem fine. If they are going
to be written down somewhere, a couple minor comments:
On Dec 9, 2009, Maciej Stachowiak wrote:
> I think something like
> the following should be our "meta-guidelines" for when to change the
> style guide:
It would be good to
On Wed, Dec 9, 2009 at 12:58 PM, Eric Uhrhane wrote:
>
> I've pulled the database-related members out of Document and made a
> new class for them, DatabaseManager. An instance of that is owned by
> each ScriptExecutionContext. There are two flavors,
> DocumentDatabaseManager and WorkerDatabase
On Wed, Dec 9, 2009 at 3:42 PM, Adam Treat wrote:
> Pedantry over trivial issues wastes time.
>
But even what is trivial is a judgement call. In general people don't
disagree about issues where they believe disagreement is a waste of time.
> A major problem: as you already noted, my common sen
On Wednesday 09 December 2009 06:33:20 pm Peter Kasting wrote:
> On Wed, Dec 9, 2009 at 3:19 PM, Adam Treat wrote:
> > With every new rule to the style guide I fear we
> > lose the latter and become ever more pedantic about often trivial issues.
>
> I see no problem with pedantry.
Pedantry over t
On Wed, Dec 9, 2009 at 3:19 PM, Adam Treat wrote:
> With every new rule to the style guide I fear we
> lose the latter and become ever more pedantic about often trivial issues.
>
I see no problem with pedantry.
Rather, I'd prefer to think of the style guidelines as just that:
> guidelines.
> In
On Wednesday 09 December 2009 04:02:07 pm Maciej Stachowiak wrote:
> I have thought over your comments about not changing the style
> guidelines at a whim.
>
> I think you make a very good point: the most important thing about the
> style guidelines is that there is one way to do things, and that's
Hi,
Kim and I have been looking into DOM/JavaScript touch event support (see
https://bugs.webkit.org/show_bug.cgi?id=32114 ). After some research we
concluded that the best option for the interface is the de-facto standard
shipped by the iPhone, Android and Palm-Pre.
So we took the BSD license
On Wed, Dec 9, 2009 at 1:02 PM, Maciej Stachowiak wrote:
> That's my thinking on the matter, perhaps others have other opinions.
> Regards,
> Maciej
+1. Very sound rules of thumb; please preserve them somewhere! FWIW,
these are basically the principles we followed at Oracle; the only
additional p
For what it's worth, I completely agree as well.
Can we take this and put it in the style guide? That way it's perfectly
clear to people under what circumstances we would consider changing it and
we can even cite specific reason numbers in the discussions?
On Wed, Dec 9, 2009 at 1:05 PM, Darin A
On Dec 9, 2009, at 1:02 PM, Maciej Stachowiak wrote:
> That's my thinking on the matter, perhaps others have other opinions.
Perhaps no surprise to the rest of you, but I agree with every word Maciej said.
-- Darin
___
webkit-dev mailing list
webk
Hi Adam,
On Dec 9, 2009, at 11:25 AM, Adam Treat wrote:
Yep, that does distinguish. That being said, my personal preference
would be
either to keep the current rule and change the code to meet it or
even better
just eliminate the rule altogether since the community hasn't been
following
Hello webkit folks!
This is just a heads-up that I'm working on implementing access to the
SQL database from worker context [I see
https://bugs.webkit.org/show_bug.cgi?id=22725, now that I look for
it]. I'll give a quick outline as to how I'm doing it, and if you
want to tell me that there's a be
On Dec 9, 2009, at 10:11 AM, Chris Marrin wrote:
On Dec 9, 2009, at 9:41 AM, Maciej Stachowiak wrote:
I meant 4 spaces from the switch (i.e. 2 additional spaces from the
case label).
switch (x) {
case foo: {
fooFunc();
}
case bar:
barFunc();
}
Ok, the example above seems to be mi
Looked at the bots again this morning and filed:
https://bugs.webkit.org/show_bug.cgi?id=32339 about a media test which
still seems to be failing on SL
https://bugs.webkit.org/show_bug.cgi?id=32340 about the Windows
fast/js/method-check.html crash.
-eric
On Tue, Dec 8, 2009 at 1:24 PM, Nikolas Zi
On Dec 9, 2009, at 11:25 AM, Adam Treat wrote:
> That speaks negatively to the real world value of the rule IMHO.
A lot of the code predates the rule, so I wouldn’t judge the effectiveness of
the rule too harshly.
-- Darin
___
webkit-dev mailing
On Wed, Dec 9, 2009 at 10:11 AM, Chris Marrin wrote:
> 1) Case labels always have the same indentation as their switch (today's
>> rule)
>
> If excessive indentation really is that big of a concern (which I don't
> think it is) I would rather see the current rule (rule 1) used.
>
> From the last
On Wednesday 09 December 2009 02:04:07 pm Darin Adler wrote:
> On Dec 9, 2009, at 10:57 AM, Adam Treat wrote:
> > Ok, well FWIW I disagree that the current rule makes things hard to read
> > and I do not like the idea of changing it. I object on the same grounds
> > as the other recent styling chan
On Dec 9, 2009, at 10:57 AM, Adam Treat wrote:
> Ok, well FWIW I disagree that the current rule makes things hard to read and
> I do not like the idea of changing it. I object on the same grounds as the
> other recent styling change proposals as well as substantively that it makes
> anything ea
On 08.12.2009, at 23:44, Darin Fisher wrote:
Sounds right to me since it mirrors the solution used to enable XHR
from workers.
Sounds good to me, too.
- WBR, Alexey Proskuryakov
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://list
On Wednesday 09 December 2009 01:03:23 pm Chris Marrin wrote:
> > What is wrong with keeping the current rule?
>
> As I pointed out in the previous thread, I feel like it makes the code
> harder to read, and got several responses of agreement. Also most of
> the switch statements in the code curren
On Dec 9, 2009, at 9:41 AM, Maciej Stachowiak wrote:
On Dec 9, 2009, at 7:26 AM, Chris Marrin wrote:
I saw another patch get rejected today because of switch statement
indentation. We discussed this last week, and I saw a lot of
support for my proposal of indenting case labels from thei
On Dec 9, 2009, at 8:13 AM, Adam Treat wrote:
On Wednesday 09 December 2009 10:26:24 am Chris Marrin wrote:
I saw another patch get rejected today because of switch statement
indentation. We discussed this last week, and I saw a lot of support
for my proposal of indenting case labels from thei
On Dec 9, 2009, at 7:26 AM, Chris Marrin wrote:
I saw another patch get rejected today because of switch statement
indentation. We discussed this last week, and I saw a lot of support
for my proposal of indenting case labels from their switch. But the
discussion did not end in resolution
On Dec 9, 2009, at 4:41 AM, Eddy Bruël wrote:
> However, when I trace my plugin, I notice that NPN_IdentifierIsString()
> always returns true, even if I do an array access. It looks like the array
> index gets converted to a string before it is passed to any of my callbacks.
> This doesn't seem
On Wednesday 09 December 2009 10:26:24 am Chris Marrin wrote:
> I saw another patch get rejected today because of switch statement
> indentation. We discussed this last week, and I saw a lot of support
> for my proposal of indenting case labels from their switch. But the
> discussion did not end in
Hi WebKit folks,
If you are not using git for WebKit, you can stop reading now.
With many helps from bunch of people, I added support of decoding images in
git binary patches to svn-apply and bugzilla's pretty printer. This means
you can post git format patches to bugzilla even if your patch cont
I saw another patch get rejected today because of switch statement
indentation. We discussed this last week, and I saw a lot of support
for my proposal of indenting case labels from their switch. But the
discussion did not end in resolution. To summarize, here are the
options mentioned:
Hi everybody,
I'm currently in the process of building a scriptable plugin using
NPAPI/NPRuntime,
and I'm trying to make the thing work with both ordinary property access and
array
access.
To this end, I have the following check in the implementation of my
hasProperty() / getProperty()
if (NPN_I
32 matches
Mail list logo