[webkit-dev] Status of WTFURL
Maciej Stachowiak
mjs at apple.com
Wed Oct 13 00:54:42 PDT 2010
On Oct 12, 2010, at 11:47 PM, Adam Barth wrote:
> Hi webkit-dev,
>
> I'd like to gauge interest in the WTFURL work [1]. Currently, there
> are two WTFURL patches that have been up for review for over a month
> without a single comment:
>
> https://bugs.webkit.org/show_bug.cgi?id=45085
> https://bugs.webkit.org/show_bug.cgi?id=45088
>
> That leads me to believe this work is not desired by the project.
Reviewed both. I didn't know that you'd returned to this project or that you had patches pending. All you had to do was ask.
> 1) I recruit folks to review the patches. To date, I've been waiting
> for Apple folks to review these patches on the theory that they might
> like to have input on this topic. If that's not necessary/desirable,
> I can expand my reviewer recruitment.
Asking people directly tends to work well.
>
> 2) I can spam the review queue with all the WTFURL patches so that
> reviewer can see how the whole thing is going to work. Typically, we
> advise folks to land things like this in small, buildable chunks,
> which is what I've been doing. However, if there's a month latency
> between chunks, that means the project will never be finished.
It is kind of hard to review the pieces without context or tests. I assume correctness issues will be flushed out later through comprehensive testing. And I can spot surface-level style issues reviewing one patch at a time. But it's hard to evaluate the design of the code without understanding the big picture. Maybe that is better done once enough code is landed that it works.
>
> [1] By way of background, Chromium uses a different URL parsing
> library than other WebKit ports. In the interests of not fracturing
> the WebKit platform, I've been working on implementing a URL parsing
> library, called WTFURL, that meets the needs of both the WebKit and
> Chromium projects.
I support your goals for this project.
Regards,
Maciej
More information about the webkit-dev
mailing list