Re: [webkit-dev] coding style and comments

2011-01-31 Thread Maciej Stachowiak
On Jan 31, 2011, at 7:18 PM, Peter Kasting wrote: > This thread has probably gone the way of all webkit-dev threads on comments > or ChangeLog files -- people's opinions vary, it turns into a bikeshed, and > nothing really changes about how we code. Repeat in a year. Well, even though we didn

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Aaron Boodman
On Mon, Jan 31, 2011 at 7:18 PM, Peter Kasting wrote: > P.S. I agree with you about assertions being better than comments to > document pre- (and post-) conditions (where possible). Me too (where possible). - a ___ webkit-dev mailing list webkit-dev@li

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Peter Kasting
This thread has probably gone the way of all webkit-dev threads on comments or ChangeLog files -- people's opinions vary, it turns into a bikeshed, and nothing really changes about how we code. Repeat in a year. w.r.t. ImageDecoder specifically, as I mentioned before I do agree that there are som

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Maciej Stachowiak
On Jan 31, 2011, at 5:48 PM, Aaron Boodman wrote: > On Mon, Jan 31, 2011 at 5:23 PM, Ryosuke Niwa wrote: >> On Mon, Jan 31, 2011 at 4:54 PM, Aaron Boodman wrote: >>> >>> It seems like the one line patch to C just broke A. It had a >>> dependency on the behavior of C that was worth documenting.

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Maciej Stachowiak
On Jan 31, 2011, at 1:06 PM, Peter Kasting wrote: > On Mon, Jan 31, 2011 at 12:47 PM, Maciej Stachowiak wrote: > On Jan 31, 2011, at 11:01 AM, Peter Kasting wrote: > I think people who favor comments tend to produce a lot of exactly this kind > of comment. Except in some cases its verbose multi

Re: [webkit-dev] Using Visual Studio 2010 for Apple's Windows WebKit port

2011-01-31 Thread Jeremy Orlow
How hard will the transition be? If it's going to take a lot of time and cause a lot of churn anyway, would this be a good time to try and make that port use GYP or CMake? (I assume the answer is probably no, but figured it was worth asking anyway. :-) J On Mon, Jan 31, 2011 at 12:46 PM, Adam R

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Aaron Boodman
On Mon, Jan 31, 2011 at 5:48 PM, Aaron Boodman wrote: > void doA() { >  // We don't need to frobulate here because doC() already did that. > } I meant for doA() to call doB() obviously. - a ___ webkit-dev mailing list webkit-dev@lists.webkit.org http:/

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Aaron Boodman
On Mon, Jan 31, 2011 at 5:23 PM, Ryosuke Niwa wrote: > On Mon, Jan 31, 2011 at 4:54 PM, Aaron Boodman wrote: >> >> It seems like the one line patch to C just broke A. It had a >> dependency on the behavior of C that was worth documenting. Now you >> have changed C and the behavior of A is probabl

[webkit-dev] Implementing the element

2011-01-31 Thread Leandro Graciá Gil
Hi, We have recently noticed a patch to implement the element in WebKit. Since this is an important new feature, I thought we should have a discussion about the best way to implement it. Here's the actual specification for it: http://www.whatwg.org/specs/web-apps/current-work/complete/commands.ht

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Ryosuke Niwa
On Mon, Jan 31, 2011 at 4:54 PM, Aaron Boodman wrote: > It seems like the one line patch to C just broke A. It had a > dependency on the behavior of C that was worth documenting. Now you > have changed C and the behavior of A is probably wrong (or at least > wasteful). > Not necessarily. X' migh

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Aaron Boodman
On Mon, Jan 31, 2011 at 12:47 AM, Ryosuke Niwa wrote: > How can we ensure that all comments are up to do date?  For example, suppose > function A calls B, and B calls C.  Then in the call site of A, I comment > "Because A does X, we do Y."  Now suppose for the moment that the behavior X > of A is

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Peter Kasting
On Mon, Jan 31, 2011 at 12:47 PM, Maciej Stachowiak wrote: > On Jan 31, 2011, at 11:01 AM, Peter Kasting wrote: > I think people who favor comments tend to produce a lot of exactly this > kind of comment. Except in some cases its verbose multiline comments that > exceed the number of actual lines

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Maciej Stachowiak
On Jan 31, 2011, at 11:01 AM, Peter Kasting wrote: > On Sun, Jan 30, 2011 at 4:47 PM, Maciej Stachowiak wrote: > Well, I didn't mean to pick on the authors of this file. This is the > impression I get from a lot of code that some call "well-commented", by which > they mean "lots of comments".

Re: [webkit-dev] Using Visual Studio 2010 for Apple's Windows WebKit port

2011-01-31 Thread Adam Roben
On Jan 31, 2011, at 3:41 PM, Peter Kasting wrote: > On Mon, Jan 31, 2011 at 12:30 PM, Adam Roben wrote: > Please let me (and the list) know if this change will cause you trouble, and > if there's something we can do to make the transition easier. > > This may make life hard on Chromium as right

Re: [webkit-dev] Using Visual Studio 2010 for Apple's Windows WebKit port

2011-01-31 Thread Peter Kasting
On Mon, Jan 31, 2011 at 12:30 PM, Adam Roben wrote: > Please let me (and the list) know if this change will cause you trouble, > and if there's something we can do to make the transition easier. > This may make life hard on Chromium as right now we don't support building with VS2010. We are wor

[webkit-dev] Using Visual Studio 2010 for Apple's Windows WebKit port

2011-01-31 Thread Adam Roben
Hi all- We'd like to switch Apple's Windows WebKit port to build with Visual Studio 2010 sometime in the next 6-8 months, and to drop support for building with Visual Studio 2005 at the same time. The biggest consequence of this will be that anyone wishing to build Apple's Windows port will ha

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Peter Kasting
On Sun, Jan 30, 2011 at 4:47 PM, Maciej Stachowiak wrote: > Well, I didn't mean to pick on the authors of this file. This is the > impression I get from a lot of code that some call "well-commented", by > which they mean "lots of comments". > I agree that the comments you pointed out are pretty

[webkit-dev] Use of 'inf' values in WebKit renderer classes

2011-01-31 Thread W. James MacLean
I'm trying to understand how float 'inf' values are used in WebKit rendering. I'm particularly interested in a case where a very large coordinate for a rect, which gets converted to 'inf' since its true value is too large to fit in a float, is retained. The value of 'inf' can't really be used for

Re: [webkit-dev] about QtWebKit documentation

2011-01-31 Thread Benjamin
On Fri, Jan 28, 2011 at 5:29 PM, Joe Mason wrote: > I’m not clear from your question what you’re trying to do. If you’re just > trying to write an app using the Qt port of WebKit, you should ask on > webkit-h...@lists.webkit.org. This list is only for developing WebKit > itself. > > > > The int

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Ryosuke Niwa
2011/1/31 Konstantin Tokarev > > You can document A as function calling B, B as function calling C, and keep > documentation of C up to date when it's behavior changes > I don't see how that can substitute my comment that "Because A does X, do Y". Saying "do Y because we call A" isn't useful at

Re: [webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Konstantin Tokarev
31.01.2011, 11:47, "Ryosuke Niwa" : > How can we ensure that all comments are up to do date?  For example, suppose > function A calls B, and B calls C.  Then in the call site of A, I comment > "Because A does X, we do Y."  Now suppose for the moment that the behavior X > of A is implemented by

[webkit-dev] Fwd: coding style and comments

2011-01-31 Thread Ryosuke Niwa
On Fri, Jan 28, 2011 at 10:21 AM, Peter Kasting wrote: > On Fri, Jan 28, 2011 at 10:14 AM, Alexey Proskuryakov wrote: > >> Do you have any specific mechanism in mind for keeping global comments >> accurate? > > > No more than I have for keeping API usage or function implementations > correct; tha

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Konstantin Tokarev
31.01.2011, 03:41, "Ryan Leavengood" : > Maybe the solution here is better documentation outside the source > code. I hope some of the more experienced WebKit developers can agree > that there are parts of the code that are harder for new developers to > dig into. Some high-level docs that are k

Re: [webkit-dev] coding style and comments

2011-01-31 Thread Ryosuke Niwa
On Sun, Jan 30, 2011 at 4:41 PM, Ryan Leavengood wrote: > > Maybe the solution here is better documentation outside the source > code. I hope some of the more experienced WebKit developers can agree > that there are parts of the code that are harder for new developers to > dig into. Can't agree m