Code clean-up: Remove tabs from suite code
Categories
(SeaMonkey :: General, enhancement)
Tracking
(seamonkey2.49esr wontfix, seamonkey2.63 wontfix, seamonkey2.53 fixed, seamonkey2.57esr fixed)
People
(Reporter: frg, Assigned: frg)
References
(Blocks 2 open bugs)
Details
Attachments
(3 files)
77.09 KB,
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
10.58 KB,
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
1.23 KB,
patch
|
frg
:
review+
frg
:
approval-comm-esr60+
|
Details | Diff | Splinter Review |
Even after Bug 1530955 we still have a lot of tabs and some trailing blanks left.
+++ This bug was initially created as a clone of Bug #1530955 +++
+++ This bug was initially created as a clone of Bug #1015741 +++
The language files for SeaMonkey contain extra whitespace in entities, at end of lines and at end of files, which makes them harder to parse/use in scripts. It would be great if this excess whitespace were removed.
(Probably also in other directories/file types, but if that's also wanted/needed it should happen in a separate bug.)
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Assignee | ||
Comment 2•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment on attachment 9054712 [details] [diff] [review] 1540438-1-suite-mailnews-whitespace.patch >+++ b/suite/mailnews/components/addrbook/content/abCardViewOverlay.js > function cvSetNode(node, text) > { >+ if ( node ) >+ { >+ if ( !node.hasChildNodes() ) >+ { >+ var textNode = document.createTextNode(text); >+ node.appendChild(textNode); >+ } >+ else if ( node.childNodes.length == 1 ) >+ node.childNodes[0].nodeValue = text; Whilst here can we remove the extra space within () >+ if ( text ) >+ visible = true; >+ else >+ visible = false; Ditto > function cvSetVisible(node, visible) > { >+ if ( visible ) Ditto >+++ b/suite/mailnews/components/addrbook/content/addressbook.js >+ var menuitem = top.document.getElementById(menuitemID); >+ if ( menuitem ) Remove extra spaces in () >+ printEngineWindow = window.openDialog("chrome://messenger/content/msgPrintEngine.xul", >+ "", >+ "chrome,dialog=no,all", >+ 1, [printUrl], statusFeedback, doPrintPreview, msgType); Should be lined up with " in (" >+++ b/suite/mailnews/content/mail3PaneWindowCommands.js > var FolderPaneController = > { >+ supportsCommand: function(command) >+ { >+ switch ( command ) Remove extra spaces in () >+ isCommandEnabled: function(command) >+ { >+ switch ( command ) Ditto >+ case "button_shiftDelete": >+ if ( command == "cmd_delete" ) Ditto > if (folder.server.type == "nntp") { >+ if ( command == "cmd_delete" ) { Ditto >+ doCommand: function(command) >+ { > // if the user invoked a key short cut then it is possible that we got here for a command which is > // really disabled. kick out if the command should be disabled. > if (!this.isCommandEnabled(command)) return; > >+ switch ( command ) Ditto >+ supportsCommand: function(command) >+ { > >+ switch ( command ) Ditto > isCommandEnabled: function(command) > { > var enabled = new Object(); > enabled.value = false; > var checkStatus = new Object(); > > switch ( command ) Ditto > case "cmd_printpreview": >+ if ( GetNumSelectedMessages() == 1 && gDBView) Ditto >+ onEvent: function(event) >+ { >+ // on blur events set the menu item texts back to the normal values >+ if ( event == 'blur' ) Ditto > { > goSetMenuValue('cmd_undo', 'valueDefault'); > goSetMenuValue('cmd_redo', 'valueDefault'); > } These four lines are not correctly indented. > function GetNumSelectedMessages() > { > try { > return gDBView.numSelected; > } > catch (ex) { These seem to be incorrectly indented too. > function SetFocusThreadPaneIfNotOnMessagePane() > { > var focusedElement = WhichPaneHasFocus(); > > if((focusedElement != GetThreadTree()) && > (focusedElement != GetMessagePane())) Missing space after if and next line will need indenting further after. > function MsgNextMessage() > { >+ GoNextMessage(nsMsgNavigationType.nextMessage, false ); Extra space before ) >+++ b/suite/mailnews/content/messageWindow.js >+ doCommand: function(command) >+ { > // if the user invoked a key short cut then it is possible that we got here for a command which is > // really disabled. kick out if the command should be disabled. > if (!this.isCommandEnabled(command)) return; > > var navigationType = nsMsgNavigationType.nextUnreadMessage; > >+ switch ( command ) Remove extra spaces within () >+++ b/suite/mailnews/content/msgMail3PaneWindow.js >+ var numFrames = window.frames.length; >+ for(var i = 0; i < numFrames; i++) Missing space after for r/a=me with those fixed.
Comment on attachment 9054713 [details] [diff] [review] 1540438-2-suite-whitespace.patch >+++ b/suite/base/content/unix/platformCommunicatorOverlay.xul >+ <!-- close --> >+ <menuitem id="menu_close" label="&closeCmd.label;" key="key_close" accesskey="&closeCmd.accesskey;" command="cmd_close"/> >+ <key id="key_close" key="&closeCmd.key;" command="cmd_close" modifiers="accel"/> >+ <key id="key_closeWindow" key="&closeCmd.key;" command="cmd_closeWindow" modifiers="accel,shift"/> Remove extra space before key= >+ <key id="key_quit" key="&quitApplicationCmd.key;" command="cmd_quit" modifiers="accel"/> Remove extra space before key= >+ <!-- Edit Menu --> >+ <key id="key_redo" key="&redoCmd.key;" command="cmd_redo" modifiers="accel"/> Remove extra space before key= r/a=me with those fixed.
Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/e1343db925fa
Part 1: SeaMonkey tab removal in mail and news. r=IanN
https://hg.mozilla.org/comm-central/rev/0a8b3f8e67f7
Part 2: SeaMonkey tab removal in general suite parts. r=IanN
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 6•5 years ago
|
||
https://hg.mozilla.org/releases/comm-esr60/rev/8de0c91224f065369f21cada0e45f40a24607fc4
Part 1: SeaMonkey tab removal in mail and news. r=IanN a=IanN
https://hg.mozilla.org/releases/comm-esr60/rev/fbaf0fac419f1d4ab0abb0bcc0b6ca96f64b732c
Part 2: SeaMonkey tab removal in general suite parts. r=IanN a=IanN
Comment 7•5 years ago
•
|
||
Just noticed an ambiguous if/then indentation. I think this is what you intended (rather than braces around a larger chunk), not 100% sure.
Assignee | ||
Comment 8•5 years ago
|
||
Comment on attachment 9056426 [details] [diff] [review] 1540438_indent_fixup.patch Thanks Ben, you are right missed this one when cleaning up the patch. Fortunately only the indention is bad. I will put it in later.
Pushed by frgrahl@gmx.net: https://hg.mozilla.org/comm-central/rev/459233271154 Fix misleading if/then indent. r=frg
Assignee | ||
Comment 10•5 years ago
|
||
Comment on attachment 9056426 [details] [diff] [review] 1540438_indent_fixup.patch https://hg.mozilla.org/releases/comm-esr60/rev/88c79646ff8a
Assignee | ||
Updated•3 years ago
|
Description
•