Summary: | [Qt] PasteboardQt.cpp has coding-style errors | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Anders Bakken <agbakken> | ||||||||
Component: | WebKit Qt | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | benjamin, commit-queue, levin, peojala11 | ||||||||
Priority: | P5 | Keywords: | Qt, QtTriaged | ||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | All | ||||||||||
Attachments: |
|
Description
Anders Bakken
2010-05-26 13:33:24 PDT
Created attachment 57151 [details]
Patch that fixes the coding style issues
Qt, QtTriaged Comment on attachment 57151 [details] Patch that fixes the coding style issues > No new tests. (OOPS!) Ditto. Created attachment 57948 [details]
Patch
Comment on attachment 57948 [details]
Patch
I don't understand this change:
47 QApplication::clipboard()->setMimeData(md, m_selectionMode ?
148 QClipboard::Selection : QClipboard::Clipboard);
148 QClipboard::Selection : QClipboard::Clipboard);
do we have a style rule about wrapping? I don't think so. I would rather use a temporary for that line to make it more clear instead of such strange indent.
Comment on attachment 57948 [details]
Patch
Please consider Eric's comment.
Anders, what is the status of this? Created attachment 78310 [details]
Patch
Comment on attachment 78310 [details] Patch Clearing flags on attachment: 78310 Committed r75339: <http://trac.webkit.org/changeset/75339> All reviewed patches have been landed. Closing bug. |