On Wed, Jan 20, 2010 at 11:58 PM, Maciej Stachowiak <m...@apple.com> wrote: > On Jan 20, 2010, at 8:18 AM, Darin Adler wrote: >> Hi folks. >> >> We’ve never formalized this, but I believe that patches tagged with a >> particular platform name such as >> >> [Qt] Add new API for fluffy bunnies >> >> should be limited to one particular platform’s code. If the patch changes >> more than a trivial bit of platform-independent code, even if the change is >> only for the benefit of a signle platform, I suggest we not use the platform >> name prefix. > > Makes sense to me. I would also like to suggest that people should *not* use > the bracket convention for things besides platform, like [dom] or > [style-queue]. Some people (e.g. me) use queries to split bracket-prefix > patches into a separate list, and it's better not to hide general-purpose > patches.
Guilty as charged. I'll avoid this practice in the future. Adam _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev