Created attachment 43130 [details] Patch submitted On the webkit-dev e-mail list, Darin Adler said that "using namespace" statements are not permitted in header files (see this report's URL for the e-mail). He said we should clarify that -- hence this report. A patch to the Coding Style Guidelines is attached. Thanks.
Comment on attachment 43130 [details] Patch submitted Need a </li>.
Created attachment 43136 [details] New patch: added some closing </li>'s The surrounding sections did not have closing </li>'s, which is what I was going by. (The entire document is inconsistent in this regard.) I am attaching a new patch with closing </li>'s for the three list items in the "using namespace" section. Thanks.
Comment on attachment 43136 [details] New patch: added some closing </li>'s Marking previous attachment as a patch
Comment on attachment 43136 [details] New patch: added some closing </li>'s Someone should run the file through an HTML validator and fix issues they find. r=me on this change
Comment on attachment 43130 [details] Patch submitted Be sure to mark your old patches as obsolete when uploading replacement patches. Some tools, like bugzilla-tool post-diff or bugzilla-tool post-commits will do this automatically for you.
Comment on attachment 43136 [details] New patch: added some closing </li>'s Rejecting patch 43136 from commit-queue. Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1']" exit_code: 1 Running build-dumprendertree Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests Testing 11621 test cases. fast/workers/worker-terminate.html -> crashed Exiting early after 1 failures. 8405 tests run. 361.78s total testing time 8404 test cases (99%) succeeded 1 test case (<1%) crashed 3 test cases (<1%) had stderr output
Comment on attachment 43136 [details] New patch: added some closing </li>'s Not your fault. :( Bug 31461 again.
Comment on attachment 43136 [details] New patch: added some closing </li>'s Rejecting patch 43136 from commit-queue. Failed to run "WebKitTools/Scripts/build-webkit" exit_code: 1 Last 500 characters of output: ld/WebCore.build/Release/WebCore.build/Objects-normal/i386/JSCompositionEvent.o /Users/eseidel/Projects/CommitQueue/WebKitBuild/Release/DerivedSources/WebCore/JSCompositionEvent.cpp normal i386 c++ com.apple.compilers.gcc.4_2 Distributed-CompileC /Users/eseidel/Projects/CommitQueue/WebKitBuild/WebCore.build/Release/WebCore.build/Objects-normal/i386/JSEventCustom.o /Users/eseidel/Projects/CommitQueue/WebCore/bindings/js/JSEventCustom.cpp normal i386 c++ com.apple.compilers.gcc.4_2 (2 failures)
Comment on attachment 43136 [details] New patch: added some closing </li>'s I'm in the process of moving the commit-queue to a new machine and it's had hiccups. Sorry. :(
Comment on attachment 43136 [details] New patch: added some closing </li>'s Clearing flags on attachment: 43136 Committed r50978: <http://trac.webkit.org/changeset/50978>
All reviewed patches have been landed. Closing bug.
(In reply to comment #4) > (From update of attachment 43136 [details]) > Someone should run the file through an HTML validator and fix issues they find. > > r=me on this change I did this while preparing the patch, and there was only one issue. The <style> tag specifying a style unique to this page (the style for code excerpts) occurs in the <body> rather than the <head>. This is partly a consequence of how the site is structured in source control with the <body> of each page being included, the <head> lying in another file, etc. So I didn't try addressing it in this patch. By the way, the document type for the page is HTML 4.01 strict, which doesn't really require closing <li>'s. If we want to switch to XHTML for the site, perhaps that should be made as a decision. It would be like deciding on a style guideline for the HTML in addition to the code.
(In reply to comment #5) > (From update of attachment 43130 [details]) > Be sure to mark your old patches as obsolete when uploading replacement > patches. Some tools, like bugzilla-tool post-diff or bugzilla-tool > post-commits will do this automatically for you. I wanted to give the reviewer the option of selecting either patch, given the extra information I supplied with the second patch. Is there a better way for me to do that? Also, I apologize in advance if I shouldn't be adding comments to a closed bug report. Let me know if there is a better way to handle that as well. Thanks for your help as I learn the ropes.
(In reply to comment #12) > By the way, the document type for the page is HTML 4.01 strict, which doesn't > really require closing <li>'s. I was surprised to learn this. > If we want to switch to XHTML for the site, perhaps that should be made as a decision. Absolutely not! My desire to close an <li> element was not driven at all by a desire to use XHTML. I just have a habit of closing elements that are not self-closing. My ignorance -- sorry.