RESOLVED FIXED Bug 44318
style correction in markup.cpp 
https://bugs.webkit.org/show_bug.cgi?id=44318
Summary style correction in markup.cpp
Ryosuke Niwa
Reported 2010-08-19 18:58:27 PDT
style correction suggested by abarth.
Attachments
cleanup (27.68 KB, patch)
2010-08-19 19:00 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
fixed mac build (27.52 KB, patch)
2010-08-20 12:17 PDT, Ryosuke Niwa 		abarth: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2010-08-19 19:00:51 PDT
Created attachment 64922 [details] cleanup
Ryosuke Niwa
Comment 2 2010-08-19 19:02:10 PDT
(In reply to comment #1) > Created an attachment (id=64922) [details] > cleanup svn diff did a terrible job here because of } but hopefully what I'm doing in appendStartMarkup is just outdenting the entire switch and renaming variables el, attr, and attrs.
Ryosuke Niwa
Comment 3 2010-08-19 19:02:43 PDT
> svn diff did a terrible job here because of } but hopefully what I'm doing in appendStartMarkup is just outdenting the entire switch and renaming variables el, attr, and attrs. hopefully *clear that*
Adam Barth
Comment 4 2010-08-19 19:11:27 PDT
Comment on attachment 64922 [details] cleanup Thanks Ryosuke. These diffs can be hard to read sometimes, but what I could verify looks correct and for the rest, I trust you.
Eric Seidel (no email)
Comment 5 2010-08-19 19:12:41 PDT
Attachment 64922 [details] did not build on mac: Build output: http://queues.webkit.org/results/3764378
Ryosuke Niwa
Comment 6 2010-08-20 12:17:06 PDT
Created attachment 64970 [details] fixed mac build
Adam Barth
Comment 7 2010-08-20 12:36:30 PDT
Comment on attachment 64970 [details] fixed mac build Yay. This one builds. :)
Ryosuke Niwa
Comment 8 2010-08-20 12:45:03 PDT
Committed r65749: <http://trac.webkit.org/changeset/65749>
Note You need to log in before you can comment on or make changes to this bug.