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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Sep 10 03:53:14 PDT 2012


--- 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.

More information about the webkit-unassigned mailing list