Closed Bug 1053166 Opened 6 years ago Closed 6 years ago
Result Popuup typo in autocomplete .xml by open Popup
Bug 707039 renamed openResultPopup to openPopup in XPFE autocomplete.xml. Due to a typo, one instance in a comment was missed: http://mxr.mozilla.org/comm-central/source/mozilla/xpfe/components/autocomplete/resources/content/autocomplete.xml#557
> http://mxr.mozilla.org/comm-central/source/mozilla/xpfe/components/ > autocomplete/resources/content/autocomplete.xml#557 Permalink: http://mxr.mozilla.org/comm-central/source/mozilla/xpfe/components/autocomplete/resources/content/autocomplete.xml?rev=b88aab9f4e80#557
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #8487023 - Flags: review?(jh)
This patch does two things: Removes trailing whitespace in blank lines. Removes trailing whitespace in comments only. This shouldn't affect blame too much.
Attachment #8487027 - Flags: review?(jh)
Attachment #8487023 - Flags: review?(jh) → review+
Comment on attachment 8487027 [details] [diff] [review] Patch v2.0 remove trailing white space in blank lines and comments only Personally I don't mind having trailing whitespace removed; I even enforce it at my job and with private stuff. But my rules don't apply here strictly (unfortunately I'd say) so let's have Neil decide whether it's OK to do this here, not do it at all, or have its own bug.
Attachment #8487027 - Flags: review?(jh) → review?(neil)
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.32
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/comm-central/rev/63e59ba6b2d5 Fix typo in autocomplete.xml comment, s/openResultPopuup/openPopup/ r=InvisibleSmiley DONTBUILD NPOTDB https://hg.mozilla.org/comm-central/rev/3e9828cc49e7 Part 2 Remove trailing whitespace in blank lines and comments r=Neil DONTBUILD NPOTDB
You need to log in before you can comment on or make changes to this bug.