[Webkit-unassigned] [Bug 96066] [BlackBerry] Removing String operator += uses in BlackBerry code

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=96066





--- Comment #12 from Sean Wang <xuewen.wang at torchmobile.com.cn>  2012-09-10 03:53:33 PST ---
(In reply to comment #10)
> > Source/WebKit/blackberry/WebKitSupport/AboutData.cpp:86
> > +    page.append("</td></tr>");
> 
> Can you put a blank line after each <tr>...</tr> block, or use "page.append("<tr>..."); page.append(...); page.append("</tr>");" to put them all on the same line, or something?  (I like putting them all on the same line, but it may be against the coding style.  I'm not sure.) It's hard to read where one cell stops and the next starts now.

I add a blank line after each <tr>...</tr>, since place multiple sentences in one line is not following webkit styles.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.