1454764 - (rustfmt) [meta] rustfmt mozilla-central

<!doctype html> <html lang="en"> <head> <base href="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764"> <meta charset="UTF-8"> <meta property="og:type" content="website"> <meta property="og:title" content="1454764 - (rustfmt) [meta] rustfmt mozilla-central"> <meta property="og:url" content="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764"> <meta property="og:description" content="REOPENED (nobody) in Developer Infrastructure - Lint and Formatting. Last updated 2022-10-11."> <meta name="twitter:label1" value="Type"> <meta name="twitter:data1" value="enhancement"> <meta name="twitter:label2" value="Priority"> <meta name="twitter:data2" value="--"> <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1"> <meta name="color-scheme" content="dark light"> <meta name="generator" content="Bugzilla 20241119.1"> <meta name="bugzilla-global" content="dummy" id="bugzilla-global" data-bugzilla="{"api_token":"","config":{"basepath":"\/"},"constant":{"COMMENT_COLS":80},"param":{"maxattachmentsize":"10240","maxusermatches":"50","splinter_base":"\/page.cgi?id=splinter.html&ignore=\/","use_markdown":"1"},"string":{"TextEditor":{"command_bold":"Bold","command_bulleted_list":"Bulleted list","command_code":"Code","command_heading":"Heading","command_italic":"Italic","command_link":"Link","command_numbered_list":"Numbered list","command_quote":"Quote","comment_editor":"Comment Editor","edit":"Edit","etiquette_link":{"href":"page.cgi?id=etiquette.html","text":"Etiquette"},"guidelines_link":{"href":"page.cgi?id=bug-writing.html","text":"Bug Writing Guidelines"},"loading":"Loading…","markdown_link":{"href":"https:\/\/guides.github.com\/features\/mastering-markdown\/","text":"Markdown supported"},"preview":"Preview","preview_error":"Preview could not be loaded. Please try again later.","text_editor":"Text Editor","toolbar_label":"Markdown text-formatting toolbar"},"bug":"bug","bug_type_required":"You must select a Type for this bug","component_required":"You must select a Component for this bug","description_required":"You must enter a Description for this bug","short_desc_required":"You must enter a Summary for this bug","version_required":"You must select a Version for this bug"},"user":{"is_new":true,"login":""}}"> <meta name="google-site-verification" content="JYXIuR9cAlV7fLmglSrc_4UaJS6Wzh5Mdxiorqu5AQc"> <title>1454764 - (rustfmt) [meta] rustfmt mozilla-central</title> <link href="/static/v20241119.1/skins/standard/global.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/BugModal/web/bug_modal.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/js/jquery/plugins/contextMenu/contextMenu.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/BMO/web/styles/bug_modal.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/EditComments/web/styles/inline-comment-editor.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/js/jquery/ui/jquery-ui-min.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/js/jquery/ui/jquery-ui-structure-min.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/js/jquery/ui/jquery-ui-theme-min.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/skins/lib/prism.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/Needinfo/web/styles/needinfo.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/Review/web/styles/badge.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/extensions/Review/web/styles/review.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/skins/standard/text-editor.css" rel="stylesheet" type="text/css"> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/jquery/jquery-min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/jquery/ui/jquery-ui-min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/jquery/plugins/contextMenu/contextMenu-min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/jquery/plugins/devbridgeAutocomplete/devbridgeAutocomplete-min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/global.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/util.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/widgets.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb">BUGZILLA.value_descs = JSON.parse('{\"bug_status\":{},\"resolution\":{\"\":\"---\"}}'); var tracking_flags_str = "{\"comments\":{\"cf_tracking_firefox134\":{\"?\":\"[Tracking Requested - why for this release]:\"},\"cf_tracking_firefox_esr128\":{\"?\":\"[Tracking Requested - why for this release]:\"},\"cf_tracking_firefox132\":{\"?\":\"[Tracking Requested - why for this release]:\"},\"cf_tracking_firefox133\":{\"?\":\"[Tracking Requested - why for this release]:\"},\"cf_tracking_firefox_esr115\":{\"?\":\"[Tracking Requested - why for this release]:\"},\"cf_tracking_firefox_relnote\":{\"?\":\"Release Note Request (optional, but appreciated)\\n[Why is this notable]:\\n[Affects Firefox for Android]:\\n[Suggested wording]:\\n[Links (documentation, blog post, etc)]:\"}},\"types\":[\"tracking\"],\"flags\":{\"project\":{\"cf_accessibility_severity\":\"---\"},\"tracking\":{\"cf_tracking_thunderbird_esr128\":\"---\",\"cf_status_firefox_esr115\":\"---\",\"cf_tracking_firefox_esr128\":\"---\",\"cf_status_thunderbird_esr115\":\"---\",\"cf_status_firefox134\":\"---\",\"cf_tracking_firefox_esr115\":\"---\",\"cf_status_firefox133\":\"---\",\"cf_tracking_firefox132\":\"---\",\"cf_tracking_thunderbird_esr115\":\"---\",\"cf_tracking_firefox133\":\"---\",\"cf_status_firefox132\":\"---\",\"cf_status_firefox_esr128\":\"---\",\"cf_status_thunderbird_esr128\":\"---\",\"cf_tracking_firefox134\":\"---\",\"cf_tracking_firefox_relnote\":\"---\"}}}"; var TrackingFlags = $.parseJSON(tracking_flags_str); BUGZILLA.bug_id = 1454764; BUGZILLA.bug_title = '1454764 - (rustfmt) [meta] rustfmt mozilla-central'; BUGZILLA.bug_summary = '[meta] rustfmt mozilla-central'; BUGZILLA.bug_url = 'https:\/\/bugzilla.mozilla.org\/show_bug.cgi?id=1454764'; BUGZILLA.bug_keywords = 'meta', BUGZILLA.bug_secure = false; BUGZILLA.user = { id: 0, login: '', is_insider: false, is_timetracker: false, can_tag: false, can_triage: false, timezone: 'America\/Los_Angeles', settings: { quote_replies: 'quoted_reply', zoom_textareas: true, remember_collapsed: true, inline_attachments: true, autosize_comments: false } }; review_suggestions = { _mentors: [ ], 'Lint and Formatting': [ ], _end: 1 }; static_component = 'Lint and Formatting'; </script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/text-editor.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/BugModal/web/autosize.min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/ProdCompSearch/web/js/prod_comp_search.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/BugModal/web/attachments_overlay.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/BugModal/web/bug_modal.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/BugModal/web/comments.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/ComponentWatching/web/js/overlay.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/bugzilla-readable-status-min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/field.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/comments.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/TrackingFlags/web/js/flags.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/BMO/web/js/firefox-crash-table.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/MozChangeField/web/js/severity-s1-priority-p1.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/MozChangeField/web/js/clear-tracking-priority-s1.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/MozChangeField/web/js/set-tracking-severity-s1.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/lib/prism.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/GoogleAnalytics/web/js/analytics.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/js/lib/md5.min.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/Review/web/js/badge.js"></script> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/Review/web/js/review.js"></script> <link href="/static/v20241119.1/skins/lib/fontawesome.min.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/skins/lib/fontawesome-brands.min.css" rel="stylesheet" type="text/css"> <link href="/static/v20241119.1/skins/lib/fontawesome-solid.min.css" rel="stylesheet" type="text/css"> <link rel="search" type="application/opensearchdescription+xml" title="Bugzilla@Mozilla" href="/search_plugin.cgi"> <link rel="shortcut icon" href="/extensions/BMO/web/images/favicon.ico"> <link rel="canonical" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764"> <link rel="shorturl" href="https://bugzilla.mozilla.org/1454764"> <meta name="google-analytics" content="UA-36116321-3" data-location="https://bugzilla.mozilla.org/bug/show-modal" data-title="1454764 - (rustfmt) [meta] rustfmt mozilla-central"> <script async src="https://www.google-analytics.com/analytics.js"></script> <meta name="robots" content="noarchive"> <meta http-equiv="X-Translated-By" content="Google"> <meta http-equiv="X-Translated-To" content="iw"> <script type="text/javascript" src="https://www.gstatic.com/_/translate_http/_/js/k=translate_http.tr.en_GB.1hbgkFx4Qn8.O/am=DgY/d=1/rs=AN8SPfqlmAPxwfG457BPbRXwNq39oSMGHg/m=corsproxy" data-sourceurl="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764"></script> <link href="https://fonts.googleapis.com/css2?family=Material+Symbols+Outlined:opsz,wght,FILL,GRAD@20..48,100..700,0..1,-50..200" rel="stylesheet"> <script type="text/javascript" src="https://www.gstatic.com/_/translate_http/_/js/k=translate_http.tr.en_GB.1hbgkFx4Qn8.O/am=DgY/d=1/exm=corsproxy/ed=1/rs=AN8SPfqlmAPxwfG457BPbRXwNq39oSMGHg/m=phishing_protection" data-phishing-protection-enabled="false" data-forms-warning-enabled="true" data-source-url="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764"></script> <meta name="robots" content="none"> </head> <body class="bugzilla-mozilla-org skin-standard bug_modal"> <script type="text/javascript" src="https://www.gstatic.com/_/translate_http/_/js/k=translate_http.tr.en_GB.1hbgkFx4Qn8.O/am=DgY/d=1/exm=corsproxy,phishing_protection/ed=1/rs=AN8SPfqlmAPxwfG457BPbRXwNq39oSMGHg/m=navigationui" data-environment="prod" data-proxy-url="https://bugzilla-mozilla-org.translate.goog" data-proxy-full-url="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" data-source-url="https://bugzilla.mozilla.org/show_bug.cgi?id=1454764" data-source-language="pl" data-target-language="iw" data-display-language="en-GB" data-detected-source-language="" data-is-source-untranslated="false" data-source-untranslated-url="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://bugzilla.mozilla.org/show_bug.cgi?id%3D1454764&anno=2" data-client="tr"></script> <div id="wrapper"> <header id="header" role="banner" aria-label="Global Header"> <div class="inner" role="none"><button type="button" class="iconic ghost" id="open-menu-drawer" aria-label="Open Site Menu"> <span class="icon" aria-hidden="true" data-icon="menu"></span> </button> <div id="header-external-links" class="dropdown" role="none"><button type="button" id="header-external-menu-button" class="dropdown-button minor" aria-label="Show Mozilla Menu" aria-expanded="false" aria-haspopup="true" aria-controls="header-external-menu"> <img src="/static/v20241119.1/extensions/BMO/web/images/moz-fav-one-color-white-rgb.svg" width="32" height="32" alt=""> </button> <ul class="dropdown-content right" id="header-external-menu" role="menu" aria-label="Mozilla Menu" style="display:none;"> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://www.mozilla.org/" role="menuitem"> <span class="label" role="none">Mozilla Home</span> </a></li> <li role="separator"></li> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://www.mozilla.org/privacy/websites/" role="menuitem"> <span class="label" role="none">Privacy</span> </a></li> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://www.mozilla.org/privacy/websites/%23cookies" role="menuitem"> <span class="label" role="none">Cookies</span> </a></li> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://www.mozilla.org/about/legal/" role="menuitem"> <span class="label" role="none">Legal</span> </a></li> </ul> </div> <h1 id="header-title" class="title" role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/home?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Go to home page"> <span aria-label="Go to Bugzilla Home Page">Bugzilla</span> </a></h1> <form id="header-search" class="quicksearch" action="/buglist.cgi" data-no-csrf role="search" aria-label="Search Bugs"><button type="button" class="iconic ghost" id="show-searchbox" aria-label="Search Bugs"> <span class="icon" aria-hidden="true" data-icon="search"></span> </button> <div class="searchbox-outer dropdown" role="combobox" aria-label="Quick Search" aria-haspopup="listbox" aria-owns="header-search-dropdown" aria-expanded="false"><span class="icon" aria-hidden="true" data-icon="search"></span> <input id="quicksearch_top" class="dropdown-button" name="quicksearch" autocomplete="off" value="" accesskey="s" placeholder="Search Bugs" title="Enter a bug number or some search terms" role="searchbox" aria-controls="header-search-dropdown" aria-label="Search Terms"> <div id="header-search-dropdown" class="dropdown-content dropdown-panel right" role="listbox" style="display: none;"> <div id="header-search-dropdown-wrapper" role="none"> <section id="header-search-dropdown-help" role="group" aria-label="Help"> <footer role="none"><a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=quicksearch.html&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Quick Search Tips</a> <a href="https://bugzilla-mozilla-org.translate.goog/query.cgi?format=advanced&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Advanced Search</a> </footer> </section> </div> </div> </div> </form> <nav id="header-nav" role="menubar" aria-label="Site Links"> <ul class="links" role="none"> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/describecomponents.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Browse bugs by component" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="category"></span> <span class="label" role="none">Browse</span> </a></li> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/query.cgi?format=advanced&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Search bugs using various criteria" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="pageview"></span> <span class="label" role="none">Advanced Search</span> </a></li> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/enter_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="File a new bug" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="add_box"></span> <span class="label" role="none">New Bug</span> </a></li> </ul> <div class="dropdown" role="none"><button type="button" id="header-tools-menu-button" class="header-button dropdown-button minor" title="More tools…" role="menuitem" aria-label="Show More Tools Menu" aria-expanded="false" aria-haspopup="true" aria-controls="header-tools-menu"> <span class="icon" aria-hidden="true" data-icon="more_horiz"></span> </button> <ul class="dropdown-content left" id="header-tools-menu" role="menu" aria-label="More Tools Menu" style="display:none;"> <li role="none"><a href="https://bugzilla-mozilla-org.translate.goog/report.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="analytics"></span> <span class="label" role="none">Reports</span> </a></li> <li role="separator"></li> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://bmo.readthedocs.io/en/latest/" target="_blank" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="help"></span> <span class="label" role="none">Documentation</span> </a></li> </ul> </div> </nav> <ul id="header-login" class="links" role="none"> <li id="mini_login_container_top" role="none"><a id="login_link_top" href="https://bugzilla-mozilla-org.translate.goog/index.cgi?GoAheadAndLogIn=1&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" class="show_mini_login_form header-button" data-qs-suffix="_top" role="button"> <span class="icon" aria-hidden="true" data-icon="login"></span> <span class="label" role="none">Log In</span> </a> <div id="mini_login_top" class="mini-popup mini_login bz_default_hidden"> <form method="post" action="/github.cgi"><input type="hidden" name="github_token" value="lCX4BVrSRt3L43v2v0dDMSTF1w8nplCUZpWklwDYqgEoO8aJO5n6gcn2B9JA69XfAFbHXL7CFabXPzPZLZEtkfhztBAdyOFCNYZIgyrnOMo4J6cRUdTsTMElrxlQBsWjl8S49xjbkU1pVbwAPxSPhe9ZXJFtNVEFqyB9Y1jKUbFB5jWWlgeZol72OST736Dy7bf7TLKHBBEbaHcWqXiU9gOaiQNUpdLHN3nY7n7l7WBiVZaonBFo0tq7a1bVdfM3"> <input type="hidden" name="target_uri" value="https://bugzilla.mozilla.org/show_bug.cgi"> <button type="submit"> <i class="fab fa-github"></i> Log In with GitHub </button> </form> <div class="method-separator"> or </div> <form action="/show_bug.cgi?id=1454764" method="POST" data-qs-suffix="_top"><input id="Bugzilla_login_top" class="bz_login" name="Bugzilla_login" title="Login" placeholder="Email" aria-label="Email" type="email" required> <input class="bz_password" id="Bugzilla_password_top" name="Bugzilla_password" type="password" title="Password" placeholder="Password" aria-label="Password" required> <input class="bz_password bz_default_hidden bz_mini_login_help" type="text" id="Bugzilla_password_dummy_top" value="password" title="Password"> <span class="remember-outer"> <input type="checkbox" id="Bugzilla_remember_top" name="Bugzilla_remember" value="on" class="bz_remember" checked> <label for="Bugzilla_remember_top">Remember me</label> </span> <input type="hidden" name="Bugzilla_login_token" value="1732463935-0KqpldaEYElA8-BWcHh7PXMZrKjvZKxNvHrbz8Ju260"> <input type="submit" name="GoAheadAndLogIn" value="Log In" id="log_in_top" class="check_mini_login_fields" data-qs-suffix="_top"> <a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#" id="hide_mini_login_top" aria-label="Close" class="close-button hide_mini_login_form" data-qs-suffix="_top"> <span class="icon" aria-hidden="true"></span> </a> </form> <div class="footer"><a href="https://bugzilla-mozilla-org.translate.goog/createaccount.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Create an Account</a> · <a id="forgot_link_top" href="https://bugzilla-mozilla-org.translate.goog/index.cgi?GoAheadAndLogIn=1&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#forgot" class="show_forgot_form" data-qs-suffix="_top">Forgot Password</a> </div> </div> <div id="forgot_form_top" class="mini-popup mini_forgot bz_default_hidden"> <form action="/token.cgi" method="post"><input type="email" name="loginname" size="20" placeholder="Email" aria-label="Email" required> <input id="forgot_button_top" value="Reset Password" type="submit"> <input type="hidden" name="a" value="reqpw"> <input type="hidden" id="token_top" name="token" value="1732463935-B1Li5j9mzSAg3y23faAbVIKtpWmGaSx7oBdC7c4uDEU"> <a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#" class="close-button hide_forgot_form" aria-label="Close" data-qs-suffix="_top"> <span class="icon" aria-hidden="true"></span> </a> </form> </div></li> </ul> </div><dialog id="menu-drawer" inert aria-label="Site Menu"> <div class="drawer-inner" role="none"> <div class="header" role="none"><button type="button" class="iconic ghost" id="close-menu-drawer" aria-label="Close Site Menu"> <span class="icon" aria-hidden="true" data-icon="close"></span> </button> </div> <ul role="menu" aria-label="Site Links"> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/describecomponents.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Browse bugs by component" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="category"></span> <span class="label" role="none">Browse</span> </a></li> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/query.cgi?format=advanced&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Search bugs using various criteria" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="pageview"></span> <span class="label" role="none">Advanced Search</span> </a></li> <li role="none"><a class="header-button" href="https://bugzilla-mozilla-org.translate.goog/enter_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="File a new bug" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="add_box"></span> <span class="label" role="none">New Bug</span> </a></li> <li role="none"><a href="https://bugzilla-mozilla-org.translate.goog/report.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="analytics"></span> <span class="label" role="none">Reports</span> </a></li> <li role="separator"></li> <li role="none"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://bmo.readthedocs.io/en/latest/" target="_blank" role="menuitem"> <span class="icon" aria-hidden="true" data-icon="help"></span> <span class="label" role="none">Documentation</span> </a></li> </ul> </div> </dialog> </header> <main id="bugzilla-body" tabindex="-1"> <aside id="message-container" role="complementary"> <noscript> <div class="noscript"> <div class="inner"> <p>Please enable JavaScript in your browser to use all the features on this site.</p> </div> </div> </noscript> </aside> <div id="main-inner"> <div role="toolbar" id="page-toolbar"> <div role="group" class="buttons"><button type="button" id="copy-summary" class="secondary separate-dropdown-button-main" title="Copy bug number and summary to your clipboard">Copy Summary</button> <div id="copy-menu-dropdown" class="dropdown"> <button type="button" id="copy-menu-btn" aria-haspopup="true" aria-label="View" aria-expanded="false" aria-controls="copy-menu" class="dropdown-button secondary separate-dropdown-button-arrow" title="More options for copy">▾</button> <ul class="dropdown-content left" id="copy-menu" role="menu" style="display:none;"> <li role="presentation"><a id="copy-markdown-summary" role="menuitem" tabindex="-1">Markdown</a></li> <li role="presentation"><a id="copy-markdown-bug-number" role="menuitem" tabindex="-1">Markdown (bug number)</a></li> <li role="presentation"><a id="copy-text-summary" role="menuitem" tabindex="-1">Plain Text</a></li> <li role="presentation"><a id="copy-html-summary" role="menuitem" tabindex="-1">HTML</a></li> </ul> </div> <div id="clip-container" style="display:none"> <input type="text" id="clip"> </div> <div class="dropdown"><button type="button" id="action-menu-btn" aria-haspopup="true" aria-label="View" aria-expanded="false" aria-controls="action-menu" class="dropdown-button secondary">View ▾</button> <ul class="dropdown-content left" id="action-menu" role="menu" style="display:none;"> <li role="presentation"><a id="action-reset" role="menuitem" tabindex="-1">Reset Sections</a></li> <li role="presentation"><a id="action-expand-all" role="menuitem" tabindex="-1">Expand All Sections</a></li> <li role="presentation"><a id="action-collapse-all" role="menuitem" tabindex="-1">Collapse All Sections</a></li> <li role="separator"></li> <li role="presentation"><a id="action-history" role="menuitem" tabindex="-1">History</a></li> <li role="separator"></li> <li role="presentation"><a href="https://bugzilla-mozilla-org.translate.goog/rest/bug/1454764?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" role="menuitem" tabindex="-1">JSON</a></li> <li role="presentation"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?ctype=xml&id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" role="menuitem" tabindex="-1">XML</a></li> </ul> </div> </div> </div> <div role="status" id="io-error" style="display:none"></div> <div role="status" id="floating-message" style="display:none"> <div id="floating-message-text"></div> </div> <section class="module"> <div class="module-content"> <div id="summary-container"> <div class="field bug_modal indent" id="field-status_summary"> <div class=" container"><span id="field-value-status_summary"> <span class="bug-status-label text" data-status="open">Open</span> <span id="field-value-bug_id"> <a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Bug 1454764</a> <span class="edit-hide">(rustfmt)</span> </span> <span class="bug-time-labels"> <span class="bug-time-label">Opened <span class="rel-time" title="2018-04-17 13:08 PDT" data-time="1523995731">7 years ago</span></span> <span class="bug-time-label">Updated <span class="rel-time" title="2022-10-11 14:30 PDT" data-time="1665523825">2 years ago</span></span> </span> </span> </div> </div> <div class="field bug_modal indent edit-hide"> <div class=" container"> <h1 id="field-value-short_desc">[meta] rustfmt mozilla-central</h1> </div> </div> <div class="field bug_modal edit-show" id="field-short_desc" style="display:none"> <div class="name"><span class="required_star edit-show" style="display:none" aria-label="Required Field">*</span> <a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23short_desc" id="short_desc-help-link" class="help">Summary: </a> </div> <div class="value"><span id="field-value-short_desc"> [meta] rustfmt mozilla-central </span> </div> </div> </div> </div> </section> <section class="module" id="module-categories"> <header id="module-categories-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Categories section" data-label-collapsed="Expand Categories section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-categories-content" aria-expanded="true" aria-labeledby="module-categories-title" aria-describedby="module-categories-subtitle"></div> <h2 class="module-title" id="module-categories-title">Categories</h2> <h3 class="module-subtitle" id="module-categories-subtitle">(Developer Infrastructure :: Lint and Formatting, enhancement)</h3> </div> </header> <div class="module-content" id="module-categories-content"> <div class="fields-lhs"> <div class="field bug_modal" id="field-product"> <div class="name"><a href="https://bugzilla-mozilla-org.translate.goog/describecomponents.cgi?product=Developer+Infrastructure&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" id="product-help-link" class="help">Product: </a> </div> <div class="value"><span id="field-value-product"> <div class="name-info-outer dropdown"><span id="product-name" class="dropdown-button" tabindex="0" role="button" aria-haspopup="menu" aria-controls="product-info">Developer Infrastructure <span class="icon" aria-hidden="true">▾</span> </span> <aside id="product-info" class="name-info-popup dropdown-content right hover-display" hidden role="menu" aria-label="Product description and actions"> <header> <div class="title"> Developer Infrastructure </div> <div class="description"> Other developer workflow and infrastructure components </div> </header> <li role="separator"></li> <div class="actions"> <div> <a href="https://bugzilla-mozilla-org.translate.goog/buglist.cgi?product=Developer+Infrastructure&bug_status=__open__&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" role="menuitem" tabindex="-1">See Open Bugs in This Product</a> </div> <div> <a href="https://bugzilla-mozilla-org.translate.goog/enter_bug.cgi?product=Developer+Infrastructure&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" role="menuitem" tabindex="-1">File New Bug in This Product</a> </div> <div> <button disabled type="button" class="secondary component-watching" role="menuitem" tabindex="-1" data-product="Developer Infrastructure" data-label-watch="Watch This Product" data-label-unwatch="Unwatch This Product" data-source="BugModal">Watch This Product</button> </div> </div> </aside> </div></span> </div> </div> <div class="field bug_modal" id="field-component"> <div class="name"><a href="https://bugzilla-mozilla-org.translate.goog/describecomponents.cgi?product=Developer+Infrastructure&component=Lint+and+Formatting&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#Lint%20and%20Formatting" id="component-help-link" class="help">Component: </a> </div> <div class="value"><span id="field-value-component"> <div class="name-info-outer dropdown"><span id="component-name" class="dropdown-button" tabindex="0" role="button" aria-haspopup="menu" aria-controls="component-info">Lint and Formatting <span class="icon" aria-hidden="true">▾</span> </span> <aside id="component-info" class="name-info-popup dropdown-content right hover-display" hidden role="menu" aria-label="Component description and actions"> <header> <div class="title"> Developer Infrastructure :: Lint and Formatting </div> <div class="description"> Issues related to linting tools (e.g flake8, eslint) and code formatting tools (clang-format). Feature requests for these tools can also be filed here </div> </header> <li role="separator"></li> <div class="actions"> <div> <a href="https://bugzilla-mozilla-org.translate.goog/buglist.cgi?product=Developer+Infrastructure&component=Lint+and+Formatting&bug_status=__open__&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" role="menuitem" tabindex="-1">See Open Bugs in This Component</a> </div> <div> <a href="https://bugzilla-mozilla-org.translate.goog/buglist.cgi?product=Developer+Infrastructure&component=Lint+and+Formatting&chfield=resolution&chfieldfrom=-6m&chfieldvalue=FIXED&bug_status=__closed__&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" role="menuitem" tabindex="-1">Recently Fixed Bugs in This Component</a> </div> <div> <a href="https://bugzilla-mozilla-org.translate.goog/enter_bug.cgi?product=Developer+Infrastructure&component=Lint+and+Formatting&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" role="menuitem" tabindex="-1">File New Bug in This Component</a> </div> <div> <button disabled type="button" class="secondary component-watching" role="menuitem" tabindex="-1" data-product="Developer Infrastructure" data-component="Lint and Formatting" data-label-watch="Watch This Component" data-label-unwatch="Unwatch This Component" data-source="BugModal">Watch This Component</button> </div> </div> </aside> </div></span> </div> </div> <div class="field bug_modal edit-show" id="field-version" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23version" id="version-help-link" class="help">Version: </a> </div> <div class="value"><span id="field-value-version"> unspecified </span> </div> </div> <div class="field bug_modal edit-show" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23rep_platform" id="-help-link" class="help">Platform: </a> </div> <div class=" container"> <div class="field bug_modal inline" id="field-rep_platform"> <div class="value"><span id="field-value-rep_platform"> Unspecified </span> </div> </div> <div class="field bug_modal indent inline" id="field-op_sys"> <div class="value"><span id="field-value-op_sys"> Unspecified </span> </div> </div> <div class="field bug_modal"> <div class=" container"> </div> </div> </div> </div> </div> <div class="fields-rhs"> <div class="field bug_modal contains-buttons" id="field-bug_type"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23bug_type" id="bug_type-help-link" class="help">Type: </a> </div> <div class="value"><span id="field-value-bug_type"> <span class="bug-type-label iconic-text" data-type="enhancement"> <span class="icon" aria-hidden="true"></span>enhancement</span> </span> </div> </div> <div class="field bug_modal" id="field-importance"> <div class=" container"><span id="field-value-importance"> <div class="field bug_modal inline" id="field-priority"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23priority" id="priority-help-link" class="help">Priority: </a> </div> <div class="value"><span id="field-value-priority"> <em>Not set</em> </span> </div> </div> <div class="field bug_modal inline" id="field-bug_severity"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23bug_severity" id="bug_severity-help-link" class="help">Severity: </a> </div> <div class="value"><span id="field-value-bug_severity"> S3 </span> </div> </div></span> </div> </div> </div> </div> </section> <section class="module" id="module-tracking"> <header id="module-tracking-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Tracking section" data-label-collapsed="Expand Tracking section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-tracking-content" aria-expanded="true" aria-labeledby="module-tracking-title" aria-describedby="module-tracking-subtitle"></div> <h2 class="module-title" id="module-tracking-title">Tracking</h2> <h3 class="module-subtitle" id="module-tracking-subtitle">(Not tracked)</h3> </div> </header> <div class="module-content" id="module-tracking-content"> <div class="fields-lhs"> <div class="field bug_modal edit-hide" id="field-status-view"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugStatuses" id="status-view-help-link" class="help">Status: </a> </div> <div class=" container"><span id="field-value-status-view">REOPENED </span> </div> </div> <div class="field bug_modal edit-show" id="field-status-edit" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugStatuses" id="status-edit-help-link" class="help">Status: </a> </div> <div class=" container"><span id="field-value-status-edit"> <div class="field bug_modal inline" id="field-bug_status"> <div class="value"><span id="field-value-bug_status"> REOPENED </span> </div> </div> <div class="field bug_modal indent inline" id="field-resolution"> <div class="value"><span id="field-value-resolution"> --- </span> </div> </div> <div id="status-action-buttons"> <div id="assigned-container" style="display:none"><button type="button" class="secondary" id="mark-as-assigned-btn"> Mark as Assigned </button> </div> </div></span> </div> </div> <div class="field bug_modal edit-show" id="field-target_milestone" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23target_milestone" id="target_milestone-help-link" class="help">Milestone: </a> </div> <div class="value"><span id="field-value-target_milestone"> --- </span> </div> </div> <div class="field bug_modal edit-show" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide%23Project_Flags" id="-help-link" class="help">Project Flags: </a> </div> <div class=" container"> <div class="flags edit-show" style="display:none"> <table class="layout-table tracking-flags"> <tbody> <tr> <td class="tracking-flag-name">Accessibility Severity</td> <td class="tracking-flag-status"><input type="hidden" id="cf_accessibility_severity-dirty"> <select id="cf_accessibility_severity" name="cf_accessibility_severity"> <option value="---" id="v6045_cf_accessibility_severity" selected>--- </option> </select></td> </tr> </tbody> </table> </div> </div> </div> </div> <div class="fields-rhs"> <div class="field tracking-flags-wrapper edit-show" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide%23Tracking_Flags" id="-help-link" class="help">Tracking Flags: </a> </div> <div class=" container"> <div class="flags edit-show" style="display:none"> <table class="layout-table tracking-flags"> <tbody> <tr> <th></th> <th>Tracking</th> <th>Status</th> </tr> <tr> <td class="tracking-flag-name">relnote-firefox</td> <td class="tracking-flag-tracking"></td> <td class="tracking-flag-status"><input type="hidden" id="cf_tracking_firefox_relnote-dirty"> <select id="cf_tracking_firefox_relnote" name="cf_tracking_firefox_relnote"> <option value="---" id="v539_cf_tracking_firefox_relnote" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">thunderbird_esr115</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_thunderbird_esr115-dirty"> <select id="cf_tracking_thunderbird_esr115" name="cf_tracking_thunderbird_esr115"> <option value="---" id="v6094_cf_tracking_thunderbird_esr115" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_thunderbird_esr115-dirty"> <select id="cf_status_thunderbird_esr115" name="cf_status_thunderbird_esr115"> <option value="---" id="v6100_cf_status_thunderbird_esr115" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">thunderbird_esr128</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_thunderbird_esr128-dirty"> <select id="cf_tracking_thunderbird_esr128" name="cf_tracking_thunderbird_esr128"> <option value="---" id="v6557_cf_tracking_thunderbird_esr128" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_thunderbird_esr128-dirty"> <select id="cf_status_thunderbird_esr128" name="cf_status_thunderbird_esr128"> <option value="---" id="v6576_cf_status_thunderbird_esr128" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">firefox-esr115</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_firefox_esr115-dirty"> <select id="cf_tracking_firefox_esr115" name="cf_tracking_firefox_esr115"> <option value="---" id="v6079_cf_tracking_firefox_esr115" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_firefox_esr115-dirty"> <select id="cf_status_firefox_esr115" name="cf_status_firefox_esr115"> <option value="---" id="v6084_cf_status_firefox_esr115" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">firefox-esr128</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_firefox_esr128-dirty"> <select id="cf_tracking_firefox_esr128" name="cf_tracking_firefox_esr128"> <option value="---" id="v6501_cf_tracking_firefox_esr128" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_firefox_esr128-dirty"> <select id="cf_status_firefox_esr128" name="cf_status_firefox_esr128"> <option value="---" id="v6505_cf_status_firefox_esr128" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">firefox132</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_firefox132-dirty"> <select id="cf_tracking_firefox132" name="cf_tracking_firefox132"> <option value="---" id="v6666_cf_tracking_firefox132" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_firefox132-dirty"> <select id="cf_status_firefox132" name="cf_status_firefox132"> <option value="---" id="v6671_cf_status_firefox132" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">firefox133</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_firefox133-dirty"> <select id="cf_tracking_firefox133" name="cf_tracking_firefox133"> <option value="---" id="v6686_cf_tracking_firefox133" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_firefox133-dirty"> <select id="cf_status_firefox133" name="cf_status_firefox133"> <option value="---" id="v6691_cf_status_firefox133" selected>--- </option> </select></td> </tr> <tr> <td class="tracking-flag-name">firefox134</td> <td class="tracking-flag-tracking"><input type="hidden" id="cf_tracking_firefox134-dirty"> <select id="cf_tracking_firefox134" name="cf_tracking_firefox134"> <option value="---" id="v6773_cf_tracking_firefox134" selected>--- </option> </select></td> <td class="tracking-flag-status"><input type="hidden" id="cf_status_firefox134-dirty"> <select id="cf_status_firefox134" name="cf_status_firefox134"> <option value="---" id="v6778_cf_status_firefox134" selected>--- </option> </select></td> </tr> </tbody> </table> </div> </div> </div> </div> </div> </section> <section class="module" id="module-people"> <header id="module-people-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse People section" data-label-collapsed="Expand People section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-people-content" aria-expanded="false" aria-labeledby="module-people-title" aria-describedby="module-people-subtitle"></div> <h2 class="module-title" id="module-people-title">People</h2> <h3 class="module-subtitle" id="module-people-subtitle">(Reporter: bholley, Unassigned)</h3> </div> </header> <div class="module-content" id="module-people-content" style="display:none"> <div class="fields-lhs"> <div class="field bug_modal edit-hide" id="field-assigned_to"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23assigned_to" id="assigned_to-help-link" class="help">Assignee: </a> </div> <div class="value"><span id="field-value-assigned_to"> <i>Unassigned</i> </span> </div> </div> <div class="field bug_modal edit-show" id="field-assigned_to" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23assigned_to" id="assigned_to-help-link" class="help">Assignee: </a> </div> <div class="value"><span id="field-value-assigned_to"> <div class="set-default-container" style="display:none"><input type="checkbox" id="set-default-assignee" name="set_default_assignee" class="set-default" value="1" data-for="assigned_to"> <label for="set-default-assignee">Reset Assignee to default</label> </div></span> </div> </div> <div class="field bug_modal edit-show" id="field-bug_mentors" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23bug_mentor" id="bug_mentors-help-link" class="help">Mentors: </a> </div> <div class="value"><span id="field-value-bug_mentors">--- </span> </div> </div> <div class="field bug_modal edit-show" id="field-qa_contact" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23qa_contact" id="qa_contact-help-link" class="help">QA Contact: </a> </div> <div class="value"><span id="field-value-qa_contact"> <div class="set-default-container" style="display:none"><input type="checkbox" id="set-default-qa-contact" name="set_default_qa_contact" class="set-default" value="1" data-for="qa_contact"> <label for="set-default-qa-contact">Reset QA Contact to default</label> </div></span> </div> </div> </div> <div class="fields-rhs"> <div class="field bug_modal" id="field-reporter"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23reporter" id="reporter-help-link" class="help">Reporter: </a> </div> <div class="value"><span id="field-value-reporter"> <div class="vcard vcard_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=40" class="gravatar" width="20" height="20"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">bholley</span></a> </div></span> </div> </div> <div class="field bug_modal" id="field-triage_owner"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23triage_owner" id="triage_owner-help-link" class="help">Triage Owner: </a> </div> <div class="value"><span id="field-value-triage_owner"> <div class="vcard vcard_555858"> <img src="https://secure.gravatar.com/avatar/69ca889a9ea77292757129a6c25568a7?d=mm&size=40" class="gravatar" width="20" height="20"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=555858&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">andi</span></a> </div></span> </div> </div> <div class="field bug_modal"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23cc" id="-help-link" class="help">CC: </a> </div> <div class=" container"><span aria-owns="cc-summary cc-latch"> <span role="button" tabindex="0" id="cc-summary" data-count="27">27 people </span> </span> <div id="cc-list" style="display:none"></div> </div> </div> </div> </div> </section> <section class="module" id="module-references"> <header id="module-references-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse References section" data-label-collapsed="Expand References section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-references-content" aria-expanded="false" aria-labeledby="module-references-title" aria-describedby="module-references-subtitle"></div> <h2 class="module-title" id="module-references-title">References</h2> <h3 class="module-subtitle" id="module-references-subtitle">(Depends on 3 open bugs, Blocks 1 open bug)</h3> </div> </header> <div class="module-content" id="module-references-content" style="display:none"> <div class="fields-lhs"> <div class="field bug_modal bug-list" id="field-dependson"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23dependson" id="dependson-help-link" class="help">Depends on: </a> </div> <div class="value"><span id="field-value-dependson"> <div class="bug-list"><span class="bug-type-label iconic" title="defect" aria-label="defect" data-type="defect"> <span class="icon" aria-hidden="true"></span> </span><a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED INCOMPLETE - cargo fmt doesn't find the servo crates used in gecko" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454757&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1454757</a> </div> <div class="bug-list"><span class="bug-type-label iconic" title="task" aria-label="task" data-type="task"> <span class="icon" aria-hidden="true"></span> </span><a class="bz_bug_link bz_status_ASSIGNED" title="ASSIGNED - [rust] Reformat the tree using rustfmt" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1617369&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1617369</a>, <a class="bz_bug_link bz_status_NEW" title="NEW - rustfmt could be a bit more polite" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1652981&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1652981</a>, <a class="bz_bug_link bz_status_NEW" title="NEW - Stop treating gfx/wr as third-party code" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1685948&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1685948</a>, <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - Add support of rustfmt in mozlint" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1551078&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1551078</a> </div></span> </div> </div> <div class="field bug_modal bug-list" id="field-blocked"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23blocks" id="blocked-help-link" class="help">Blocks: </a> </div> <div class="value"><span id="field-value-blocked"> <div class="bug-list"><span class="bug-type-label iconic" title="enhancement" aria-label="enhancement" data-type="enhancement"> <span class="icon" aria-hidden="true"></span> </span><a class="bz_bug_link bz_status_NEW" title="NEW - [meta] Make the developer experience for Firefox + Rust great" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1380210&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">rust-great</a>, <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Add a new rustfmt linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1369792&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1369792</a>, <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Enforce rustfmt with a linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454777&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1454777</a> </div></span> </div> </div> <div class="field bug_modal" id="field-dependencytree"> <div class="name"> </div> <div class=" container"><span id="field-value-dependencytree"> Dependency <a href="https://bugzilla-mozilla-org.translate.goog/showdependencytree.cgi?id=1454764&hide_resolved=1&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">tree</a> / <a href="https://bugzilla-mozilla-org.translate.goog/showdependencygraph.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">graph</a> </span> </div> </div> <div class="field bug_modal bug-list edit-show" id="field-regresses" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23regresses" id="regresses-help-link" class="help">Regressions: </a> </div> <div class="value"><span id="field-value-regresses"> --- </span> </div> </div> <div class="field bug_modal bug-list edit-show" id="field-regressed_by" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23regressed_by" id="regressed_by-help-link" class="help">Regressed by: </a> </div> <div class="value"><span id="field-value-regressed_by"> --- </span> </div> </div> </div> <div class="fields-rhs"> <div class="field bug_modal edit-show" id="field-bug_file_loc" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23bug_file_loc" id="bug_file_loc-help-link" class="help">URL: </a> </div> <div class="value"><span id="field-value-bug_file_loc"> <div class="link"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" target="_blank" rel="noreferrer" title="" class="bug-url" data-safe="0"></a> </div></span> </div> </div> <div class="field bug_modal edit-show" id="field-see_also" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23see_also" id="see_also-help-link" class="help">See Also: </a> </div> <div class="value"><span id="field-value-see_also"> --- </span> </div> </div> </div> </div> </section> <section class="module" id="module-details"> <header id="module-details-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Details section" data-label-collapsed="Expand Details section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-details-content" aria-expanded="false" aria-labeledby="module-details-title" aria-describedby="module-details-subtitle"></div> <h2 class="module-title" id="module-details-title">Details</h2> <h3 class="module-subtitle" id="module-details-subtitle">(Keywords: meta)</h3> </div> </header> <div class="module-content" id="module-details-content" style="display:none"> <div class="fields-lhs"> <div class="field bug_modal edit-show" id="field-alias" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/BugFields%23alias" id="alias-help-link" class="help">Alias: </a> </div> <div class="value"><span id="field-value-alias"> rustfmt </span> </div> </div> <div class="field bug_modal" id="field-keywords"> <div class="name"><a href="https://bugzilla-mozilla-org.translate.goog/describekeywords.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" id="keywords-help-link" class="help">Keywords: </a> </div> <div class="value"><span id="field-value-keywords"> <a href="https://bugzilla-mozilla-org.translate.goog/buglist.cgi?keywords=meta&resolution=---&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">meta</a> </span> </div> </div> <div class="field bug_modal edit-show" id="field-status_whiteboard" style="display:none"> <div class="name"><a href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://wiki.mozilla.org/BMO/UserGuide/Whiteboard" id="status_whiteboard-help-link" class="help">Whiteboard: </a> </div> <div class="value"><span id="field-value-status_whiteboard">--- </span> </div> </div> <div class="field bug_modal edit-show" id="field-cf_cab_review" style="display:none"> <div class="name"> Change Request: </div> <div class="value"><span id="field-value-cf_cab_review"> --- </span> </div> </div> </div> <div class="fields-rhs"> <div class="field bug_modal edit-show" id="field-bug_flags" style="display:none"> <div class="name"> Bug Flags: </div> <div class=" container"><span id="field-value-bug_flags"> <div id="bug-flags" class="flags"> <table class="layout-table"> <tbody class="edit-show" style="display:none"> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Is this bug/feature enabled in a release by a pref flag?" for="flag_type-930">behind-pref</label></td> <td class="flag-value"><input type="hidden" id="flag_type-930-dirty"> <select id="flag_type-930" name="flag_type-930" title="Is this bug/feature enabled in a release by a pref flag?" disabled class="bug-flag"> <option value="X"></option> </select></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Flag tracking inclusion in the desktop Firefox product backlog." for="flag_type-846">firefox-backlog</label></td> <td class="flag-value"><input type="hidden" id="flag_type-846-dirty"> <select id="flag_type-846" name="flag_type-846" title="Flag tracking inclusion in the desktop Firefox product backlog." disabled class="bug-flag"> <option value="X"></option> </select></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Flag is used to track security bug bounty nominations. Mail security(at)mozilla.org to nominate a bug." for="flag_type-803">sec-bounty</label></td> <td class="flag-value"><input type="hidden" id="flag_type-803-dirty"> <select id="flag_type-803" name="flag_type-803" title="Flag is used to track security bug bounty nominations. Mail security(at)mozilla.org to nominate a bug." class="bug-flag"> <option value="X"></option> <option value="?">?</option> </select></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Flag is used to track whether the bug report is eligible for inclusion in the Bug Bounty Hall of Fame." for="flag_type-913">sec-bounty-hof</label></td> <td class="flag-value"><input type="hidden" id="flag_type-913-dirty"> <select id="flag_type-913" name="flag_type-913" title="Flag is used to track whether the bug report is eligible for inclusion in the Bug Bounty Hall of Fame." disabled class="bug-flag"> <option value="X"></option> </select></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Whether the bug has a testcase in the qa test suite or not. Set it to "in-qa-testsuite?" if the bug needs a testcase (only set this if the bug actually *needs* a testcase - not all bugs do, even layout bugs!), set it to "in-qa-testsuite+" if the bug has an appropriate testcase, and set it to "in-qa-testsuite-" if the bug doesn't need an explicit testcase (e.g. for code cleanup bugs). Only QA actively working on test cases in the component should use this keyword." for="flag_type-787">in-qa-testsuite</label></td> <td class="flag-value"><input type="hidden" id="flag_type-787-dirty"> <select id="flag_type-787" name="flag_type-787" title="Whether the bug has a testcase in the qa test suite or not. Set it to "in-qa-testsuite?" if the bug needs a testcase (only set this if the bug actually *needs* a testcase - not all bugs do, even layout bugs!), set it to "in-qa-testsuite+" if the bug has an appropriate testcase, and set it to "in-qa-testsuite-" if the bug doesn't need an explicit testcase (e.g. for code cleanup bugs). Only QA actively working on test cases in the component should use this keyword." disabled class="bug-flag"> <option value="X"></option> </select></td> <td class="flag-requestee"> <div id="requestee_type-787-container" style="display:none"> <input name="requestee_type-787" value="" class="requestee bz_autocomplete_user" id="requestee_type-787"> </div></td> <td></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="Whether the bug has a testcase in the test suite or not. Set it to "in-testsuite?" if the bug needs a testcase (only set this if the bug actually *needs* a testcase - not all bugs do, even layout bugs!), set it to "in-testsuite+" if the bug has an appropriate testcase, and set it to "in-testsuite-" if the bug doesn't need an explicit testcase (e.g. for code cleanup bugs). Only QA actively working on test cases in the component should use this keyword." for="flag_type-37">in-testsuite</label></td> <td class="flag-value"><input type="hidden" id="flag_type-37-dirty"> <select id="flag_type-37" name="flag_type-37" title="Whether the bug has a testcase in the test suite or not. Set it to "in-testsuite?" if the bug needs a testcase (only set this if the bug actually *needs* a testcase - not all bugs do, even layout bugs!), set it to "in-testsuite+" if the bug has an appropriate testcase, and set it to "in-testsuite-" if the bug doesn't need an explicit testcase (e.g. for code cleanup bugs). Only QA actively working on test cases in the component should use this keyword." disabled class="bug-flag"> <option value="X"></option> </select></td> </tr> <tr> <td class="flag-setter"></td> <td class="flag-name"><label title="qe-verify: + ➜ request to verify the bug manually qe-verify: – ➜ the bug will not/can not be verified manually" for="flag_type-864">qe-verify</label></td> <td class="flag-value"><input type="hidden" id="flag_type-864-dirty"> <select id="flag_type-864" name="flag_type-864" title="qe-verify: + ➜ request to verify the bug manually qe-verify: – ➜ the bug will not/can not be verified manually" disabled class="bug-flag"> <option value="X"></option> </select></td> </tr> </tbody> </table> </div></span> </div> </div> </div> </div> </section> <section class="module edit-show" style="display:none" id="module-crash-data"> <header id="module-crash-data-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Crash Data section" data-label-collapsed="Expand Crash Data section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-crash-data-content" aria-expanded="false" aria-labeledby="module-crash-data-title" aria-describedby="module-crash-data-subtitle"></div> <h2 class="module-title" id="module-crash-data-title">Crash Data</h2> </div> </header> <div class="module-content" id="module-crash-data-content" style="display:none"> <div class="field bug_modal edit-show" id="field-cf_crash_signature" style="display:none"> <div class="name"> Signature: </div> <div class="value"><span id="field-value-cf_crash_signature"> <em>None</em> </span> </div> </div> </div> </section> <section class="module edit-show" style="display:none" id="module-security"> <header id="module-security-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Security section" data-label-collapsed="Expand Security section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-security-content" aria-expanded="false" aria-labeledby="module-security-title" aria-describedby="module-security-subtitle"></div> <h2 class="module-title" id="module-security-title">Security</h2> <h3 class="module-subtitle" id="module-security-subtitle">(public)</h3> </div> </header> <div class="module-content" id="module-security-content" style="display:none"> <div class="groups edit-hide"> This bug is publicly visible. </div> <div class="groups edit-show" style="display:none"> </div> </div> </section> <section class="module edit-show" style="display:none" id="module-user-story" data-non-stick="1"> <header id="module-user-story-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse User Story section" data-label-collapsed="Expand User Story section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-user-story-content" aria-expanded="false" aria-labeledby="module-user-story-title" aria-describedby="module-user-story-subtitle"></div> <h2 class="module-title" id="module-user-story-title">User Story</h2> </div> </header> <div class="module-content" id="module-user-story-content" style="display:none"> <pre id="user-story"></pre> </div> </section> <section class="module" id="module-attachments"> <header id="module-attachments-header" class="module-header"> <div class="module-latch" data-label-expanded="Collapse Attachments section" data-label-collapsed="Expand Attachments section"> <div class="module-spinner" role="button" tabindex="0" aria-controls="module-attachments-content" aria-expanded="true" aria-labeledby="module-attachments-title" aria-describedby="module-attachments-subtitle"></div> <h2 class="module-title" id="module-attachments-title">Attachments</h2> <h3 class="module-subtitle" id="module-attachments-subtitle">(1 obsolete file)</h3> </div> </header> <div class="module-content" id="module-attachments-content"> <table role="table" class="responsive" id="attachments"> <tbody> <tr data-attachment-id="9002232" class=" attach-obsolete attach-patch " style="display:none"> <td class="attach-desc-td"> <div class="attach-desc"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> Format mozilla-central with rustfmt 1.0 RC. v1 </a> </div> <div><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c25" class="attach-time activity-ref"><span class="rel-time" title="2018-08-18 12:47 PDT" data-time="1534621658">6 years ago</span></a> <span class="attach-author"> <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div></span> </div> <div class="attach-info"> 2.33 MB, patch </div></td> <td> <div class="attach-flag"> <div class="vcard vcard_417288"> <span class="fn">froydnj</span> </div>: <a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c36" class="flag-name-status rel-time-title activity-ref" title="6 years ago" data-time="1534621801"> feedback+</a> </div></td> <td class="attach-actions"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" data-details="1">Details</a> | <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Diff</a> | <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Splinter Review</a></td> </tr> </tbody> </table> <footer id="attachments-footer"> <div id="attachments-actions"><button type="button" id="attachments-obsolete-btn" class="secondary">Show Obsolete</button> </div> </footer> </div> </section> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb"> init_module_visibility(); </script> <meta name="firefox-versions" content="{"FIREFOX_AURORA":"","FIREFOX_DEVEDITION":"133.0b9","FIREFOX_ESR":"128.4.0esr","FIREFOX_ESR115":"115.17.0esr","FIREFOX_ESR_NEXT":"","FIREFOX_NIGHTLY":"134.0a1","LAST_MERGE_DATE":"2024-10-28","LAST_RELEASE_DATE":"2024-10-29","LAST_SOFTFREEZE_DATE":"2024-10-24","LAST_STRINGFREEZE_DATE":"2024-10-25","LATEST_FIREFOX_DEVEL_VERSION":"133.0b9","LATEST_FIREFOX_OLDER_VERSION":"3.6.28","LATEST_FIREFOX_RELEASED_DEVEL_VERSION":"133.0b9","LATEST_FIREFOX_VERSION":"132.0.2","NEXT_MERGE_DATE":"2024-11-25","NEXT_RELEASE_DATE":"2024-11-26","NEXT_SOFTFREEZE_DATE":"2024-11-21","NEXT_STRINGFREEZE_DATE":"2024-11-22"}"> <div id="comment-actions"><button type="button" id="bottom-btn" class="secondary" aria-label="Go to Page Bottom">Bottom ↓</button> <div class="dropdown"><button type="button" id="comment-tags-btn" aria-haspopup="true" aria-label="Tags" aria-expanded="false" aria-controls="comment-tags-menu" class="dropdown-button minor">Tags ▾</button> <ul id="comment-tags-menu" role="menu" tabindex="0" class="dropdown-content left" style="display:none"> <li role="presentation"><a role="menuitem" tabindex="-1" data-comment-tag="">Reset</a></li> </ul> </div> <div class="dropdown"><button type="button" id="view-menu-btn" aria-haspopup="true" aria-label="Timeline" aria-expanded="false" aria-controls="view-menu" class="dropdown-button minor">Timeline ▾</button> <ul id="view-menu" role="menu" tabindex="0" class="dropdown-content left" style="display:none"> <li role="presentation"><a id="view-reset" role="menuitem" tabindex="-1">Reset</a></li> <li role="separator"></li> <li role="presentation"><a id="view-collapse-all" role="menuitem" tabindex="-1">Collapse All</a></li> <li role="presentation"><a id="view-expand-all" role="menuitem" tabindex="-1">Expand All</a></li> <li role="presentation"><a id="view-comments-only" role="menuitem" tabindex="-1">Comments Only</a></li> </ul> </div> </div> <div class="change-set" id="c0"> <div class="comment" data-id="13213520" data-no="0" data-tags=""> <table class="layout-table change-head reporter" id="ch-0" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-0" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-0"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c0">Description</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:08 PDT" data-time="1523995731">7 years ago</span> </div></td> </tr> <tr id="ctag-0"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-0" data-comment-id="13213520"> There are huge advantages to automatic source formatting. This is a tall mountain to climb for C++, but Rust is easier - there's less of it in the tree, and developers have had less time to become attached to their personal style fiefdoms. rustfmt isn't entirely stable, but it's pretty close, and is mostly getting work around edge cases. We could wait for it to be entirely stable, or we could just format the codebase now and then pick up any tweaks as they come. </div> </div> <div class="change-set" id="c1"> <div class="comment" data-id="13213543" data-no="1" data-tags=""> <table class="layout-table change-head reporter" id="ch-1" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-1" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-1"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c1">Comment 1</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:13 PDT" data-time="1523995991">7 years ago</span> </div></td> </tr> <tr id="ctag-1"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-1" data-comment-id="13213543"> This is what the change looks like: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/bholley/gecko/commits/rustfmt_mc">https://github.com/bholley/gecko/commits/rustfmt_mc</a> NIing a few stakeholders for comment. I believe the decision is ultimately Ehsan's. </div> <div class="activity"> <div class="change"> Flags: needinfo?(nfroyd) </div> <div class="change"> Flags: needinfo?(hsivonen) </div> <div class="change"> Flags: needinfo?(ehsan) </div> </div> </div> <div class="change-set" id="c2"> <div class="comment" data-id="13213568" data-no="2" data-tags=""> <table class="layout-table change-head " id="ch-2" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-2" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-2"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">Comment 2</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:21 PDT" data-time="1523996507">7 years ago</span> </div></td> </tr> <tr id="ctag-2"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-2" data-comment-id="13213568"> I mean, I would complain about some of the style choices, but I don't think rustfmt has a large number of knobs to tweak ala clang-format. So sure. Do you plan to enforce formatting at all with a lint? I'd be hesitant to format the whole tree and then not have a mechanism in place to ensure it stays formatted. CC'ing :ato for the geckodriver heads-up. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(nfroyd)</span> </div> </div> </div> <div class="change-set" id="c3"> <div class="comment" data-id="13213586" data-no="3" data-tags=""> <table class="layout-table change-head " id="ch-3" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546716"> <img src="https://secure.gravatar.com/avatar/1d91d22a4892f16f744b31fd4286144e?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546716"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546716&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Emilio Cobos Álvarez (:emilio)</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-3" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-3"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c3">Comment 3</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:27 PDT" data-time="1523996839">7 years ago</span> </div></td> </tr> <tr id="ctag-3"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-3" data-comment-id="13213586"> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">comment #2</a>) <span class="quote">> I mean, I would complain about some of the style choices, but I don't think > rustfmt has a large number of knobs to tweak ala clang-format. So sure. > > Do you plan to enforce formatting at all with a lint? I'd be hesitant to > format the whole tree and then not have a mechanism in place to ensure it > stays formatted.</span> +1. There's the thing about what happens when rustfmt changes. (either because of rustfmt fixes or default configuration changes or what not). My proposal would be to enforce the usage of rustfmt-format-diff[1] per-commit / per push. It worked fine when I wrote it, but last time I tried it[2] looks like it found a rustfmt bug where it formats the first line of the file without needing to. Other than fixing that, it'd provide progressive formatting without massive changesets. What do people think of something like that? [1]: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/blob/master/src/format-diff/main.rs">https://github.com/rust-lang-nursery/rustfmt/blob/master/src/format-diff/main.rs</a> [2]: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/servo/servo/pull/20617">https://github.com/servo/servo/pull/20617</a> </div> </div> <div class="change-set" id="c4"> <div class="comment" data-id="13213588" data-no="4" data-tags=""> <table class="layout-table change-head reporter" id="ch-4" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-4" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-4"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c4">Comment 4</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:27 PDT" data-time="1523996855">7 years ago</span> </div></td> </tr> <tr id="ctag-4"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-4" data-comment-id="13213588"> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">comment #2</a>) <span class="quote">> I mean, I would complain about some of the style choices, but I don't think > rustfmt has a large number of knobs to tweak ala clang-format. So sure.</span> It does have some knobs: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md">https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md</a> Emilio is also followup up with the rustfmt folks about various nits that we encountered while formatting the servo tree, so we should expect to see some improvements there. <span class="quote">> > Do you plan to enforce formatting at all with a lint? I'd be hesitant to > format the whole tree and then not have a mechanism in place to ensure it > stays formatted.</span> Yes, was just in the middle of filing a followup for that. </div> </div> <div class="change-set" id="c5"> <div class="comment" data-id="13213596" data-no="5" data-tags=""> <table class="layout-table change-head reporter" id="ch-5" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-5" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-5"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c5">Comment 5</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:30 PDT" data-time="1523997000">7 years ago</span> </div></td> </tr> <tr id="ctag-5"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-5" data-comment-id="13213596"> (In reply to Emilio Cobos Álvarez [:emilio] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c3">comment #3</a>) <span class="quote">> My proposal would be to enforce the usage of rustfmt-format-diff[1] > per-commit / per push. > > It worked fine when I wrote it, but last time I tried it[2] looks like it > found a rustfmt bug where it formats the first line of the file without > needing to. Other than fixing that, it'd provide progressive formatting > without massive changesets. > > What do people think of something like that?</span> Per our discussion on the servo side, I'm opposed to incremental formatting in this case. I think consistency is important for people to infer the correct style, and I think it also sucks for things like rename patches to suddenly become responsible for fixing up surrounding style. I think the biggest bulk of Rust code we have is the servo style system code, which has already been rustfmted. Not my call though, so I'll defer to Ehsan/Nathan. </div> </div> <div class="change-set" id="a1364_313730"> <div class="change" id="aa1364_313730"> <table class="layout-table change-head reporter" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a1364_313730" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a1364_313730"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a1364_313730">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:31 PDT" data-time="1523997095">7 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Blocks: <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Enforce rustfmt with a linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454777&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1454777</a> </div> </div> </div> <div class="change-set" id="c6"> <div class="comment" data-id="13213614" data-no="6" data-tags=""> <table class="layout-table change-head " id="ch-6" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_432813" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/7eb9a4f1258dbc22d0304a6aba6a1a9f?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_432813" id="a1364_313730"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=432813&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nick Cameron [:nrc]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-6" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-6"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c6">Comment 6</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:37 PDT" data-time="1523997420">7 years ago</span> </div></td> </tr> <tr id="ctag-6"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-6" data-comment-id="13213614"> <span class="quote">> It does have some knobs: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md">https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md</a></span> It would be great to avoid using them though - there's a benefit to the code looking like the rest of the Rust community's code, and the options are likely to be the last things to be stable and thus code using them will suffer from more formatting changes over time. We're planning to stabilise Rustfmt (release a 1.0) this quarter, which should include some mechanisms to work much better on CI, so it might be worth holding out on implementing the linting/CI stuff until we have better support in Rustfmt. </div> </div> <div class="change-set" id="c7"> <div class="comment" data-id="13213629" data-no="7" data-tags=""> <table class="layout-table change-head reporter" id="ch-7" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-7" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-7"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c7">Comment 7</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:44 PDT" data-time="1523997878">7 years ago</span> </div></td> </tr> <tr id="ctag-7"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-7" data-comment-id="13213629"> (In reply to Emilio Cobos Álvarez [:emilio] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c3">comment #3</a>) <span class="quote">> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">comment #2</a>) > > I mean, I would complain about some of the style choices, but I don't think > > rustfmt has a large number of knobs to tweak ala clang-format. So sure.</span> Also, now is a great time to complain about any of the choices, since rustfmt isn't stable yet, and any obviously-silly things can be more easily fixed now than later. So please bring them up with nrc & company. </div> </div> <div class="change-set" id="c8"> <div class="comment" data-id="13213634" data-no="8" data-tags=""> <table class="layout-table change-head " id="ch-8" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288" id="a1364_313730"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-8" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-8"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c8">Comment 8</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 13:46 PDT" data-time="1523997984">7 years ago</span> </div></td> </tr> <tr id="ctag-8"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-8" data-comment-id="13213634"> (In reply to Bobby Holley (:bholley) from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c4">comment #4</a>) <span class="quote">> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">comment #2</a>) > > I mean, I would complain about some of the style choices, but I don't think > > rustfmt has a large number of knobs to tweak ala clang-format. So sure. > > It does have some knobs: > <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md">https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md</a></span> Whoa, I guess I was mistaken about having a One True Way to format Rust code! Having just skimmed through, I think the "visual" conventions are more consistent with our C++ coding style in many cases. My impression was that the One True Style was more the "block" conventions? Do we know offhand if editors just DTRT for certain rustfmt settings? (Obviously, you can run rustfmt on your code, but it's nice if the editor just gets it right for the most part.) <span class="quote">> > Do you plan to enforce formatting at all with a lint? I'd be hesitant to > > format the whole tree and then not have a mechanism in place to ensure it > > stays formatted. > > Yes, was just in the middle of filing a followup for that.</span> \o/ As far as incremental vs. all-at-once, I think we should do all-at-once, so people don't have to remember "do I run rustfmt on this or not?" and so everything looks consistent, rather than some older, less-touched code only getting incrementally formatted as people touch it. Maybe this consideration would change if we formatted the C++ code, but we probably have a (comparatively) small enough amount of Rust that doing it all now makes more sense. </div> </div> <div class="change-set" id="c9"> <div class="comment" data-id="13213667" data-no="9" data-tags=""> <table class="layout-table change-head reporter" id="ch-9" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-9" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-9"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c9">Comment 9</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 14:03 PDT" data-time="1523998989">7 years ago</span> </div></td> </tr> <tr id="ctag-9"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-9" data-comment-id="13213667"> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c8">comment #8</a>) <span class="quote">> (In reply to Bobby Holley (:bholley) from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c4">comment #4</a>) > > (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c2">comment #2</a>) > > > I mean, I would complain about some of the style choices, but I don't think > > > rustfmt has a large number of knobs to tweak ala clang-format. So sure. > > > > It does have some knobs: > > <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md">https://github.com/rust-lang-nursery/rustfmt/blob/master/Configurations.md</a> > > Whoa, I guess I was mistaken about having a One True Way to format Rust code! > > Having just skimmed through, I think the "visual" conventions are more > consistent with our C++ coding style in many cases. My impression was that > the One True Style was more the "block" conventions?</span> Having worked with both, I've become a convert to block indentation. It makes decisions so much more straightforward, and avoids the need to reindent a zillion arguments when you change the name of a method. It's also very much the dominant Rust idiom, plus the style that we've already bulk-formatted the servo code to. I don't think we should go against the grain on this one. <span class="quote">> As far as incremental vs. all-at-once, I think we should do all-at-once, so > people don't have to remember "do I run rustfmt on this or not?" and so > everything looks consistent, rather than some older, less-touched code only > getting incrementally formatted as people touch it. Maybe this > consideration would change if we formatted the C++ code, but we probably > have a (comparatively) small enough amount of Rust that doing it all now > makes more sense.</span> +1 </div> </div> <div class="change-set" id="c10"> <div class="comment" data-id="13213786" data-no="10" data-tags=""> <table class="layout-table change-head " id="ch-10" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_47192" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/5d5f37905f36ca9989f88861a702cff3?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_47192" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=47192&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Mike Hommey [:glandium]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-10" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-10"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c10">Comment 10</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-17 14:58 PDT" data-time="1524002284">7 years ago</span> </div></td> </tr> <tr id="ctag-10"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-10" data-comment-id="13213786"> The main drawback is that rustfmt likes to change how it formats stuff quite frequently still, and you can end up in formatting wars if people change the same files and apply cargo fmt with different versions. Another is that in a few occasions, it has actually busted some of my source code (and that's not old anecdotal reference, it actually happened to me last week or so, where it simply removed #[cfg]'d code, and interestingly, I wasn't able to reproduce afterwards). </div> </div> <div class="change-set" id="c11"> <div class="comment" data-id="13217722" data-no="11" data-tags=""> <table class="layout-table change-head " id="ch-11" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_5490" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/0030426a5e04e85c33fe02480bc4ca81?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_5490" id="a1364_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=5490&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Henri Sivonen (:hsivonen)</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-11" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-11"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c11">Comment 11</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-19 01:17 PDT" data-time="1524125848">7 years ago</span> </div></td> </tr> <tr id="ctag-11"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-11" data-comment-id="13217722"> I think that in principle we should do this, but I share glandium's concern: (In reply to Mike Hommey [:glandium] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c10">comment #10</a>) <span class="quote">> The main drawback is that rustfmt likes to change how it formats stuff quite > frequently still, and you can end up in formatting wars if people change the > same files and apply cargo fmt with different versions.</span> My main concern is that rustfmt is supposed to come from rustup, but that method of obtaining it has been consistenly broken for me. I try to follow the instructions rustup gives me. Then I get: error: toolchain 'nightly-x86_64-unknown-linux-gnu' does not have the binary `cargo-fmt` Then I go back to installing it manually. Everyone installing rustfmt manually gives everyone a different snapshot. Even if the rustup method was fixed, there'd still be the problem of rustfmt changing frequently and different people having a different version. Maybe mach should manage its own copy of rustfmt like it does with clang-tidy to have a consistent rustfmt-for-m-c at a given point in time. <span class="quote">> Another is that in a > few occasions, it has actually busted some of my source code (and that's not > old anecdotal reference, it actually happened to me last week or so, where > it simply removed #[cfg]'d code, and interestingly, I wasn't able to > reproduce afterwards).</span> rustfmt breaking my code has happened to me, too, but, in fairness, it was 11 months ago: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/hsivonen/encoding_rs/blob/master/fuzz/fuzzers/fuzz_encodings.rs%23L537">https://github.com/hsivonen/encoding_rs/blob/master/fuzz/fuzzers/fuzz_encodings.rs#L537</a> On the topic of incremental vs. all at the same time, I think we should do all (non-vendored) crates at the same time. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(hsivonen)</span> </div> </div> </div> <div class="change-set" id="c12"> <div class="comment" data-id="13220393" data-no="12" data-tags=""> <table class="layout-table change-head " id="ch-12" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_480271" id="a1364_313730"> <img src="https://secure.gravatar.com/avatar/0d3b27247d1f73c7eb3fd5eeddeddfbf?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_480271" id="a1364_313730"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=480271&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">u480271</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-12" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-12"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c12">Comment 12</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-19 19:09 PDT" data-time="1524190194">7 years ago</span> </div></td> </tr> <tr id="ctag-12"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-12" data-comment-id="13220393"> As glandium said, I found rustfmt to change it's mind with each release. I did add custom settings to the crates I was working on, but in the end removed them at the insistence of nrc. I happy to use what ever settings are decided by consensus. I'm not interested in bike-shedding code formatting. <span class="quote">> My main concern is that rustfmt is supposed to come from rustup, but that method of obtaining it has been consistenly broken > for me. I try to follow the instructions rustup gives me. Then I get: > error: toolchain 'nightly-x86_64-unknown-linux-gnu' does not have the binary `cargo-fmt`</span> I had this error when I installed rustfmt via cargo and then it moved to being managed by rustup. I can't remember exactly how I fixed it, but it might have involved `cargo uninstall rustfmt` and `rustup component add rustfmt-preview`. Maybe nrc can shed more light here? </div> </div> <div class="change-set" id="a599016_39022"> <div class="change" id="aa599016_39022"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_39022" id="a599016_39022"> <img src="https://secure.gravatar.com/avatar/30fa9f928422bef2f98dd5f564def686?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_39022" id="a599016_39022"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=39022&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">(not currently active) Ted Mielczarek</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a599016_39022" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a599016_39022"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a599016_39022">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-04-24 11:32 PDT" data-time="1524594747">7 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Product: Firefox Build System → Core </div> <div class="change"> Version: Version 3 → unspecified </div> </div> </div> <div class="change-set" id="a1460790_609492"> <div class="change" id="aa1460790_609492"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_609492" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/0c9e1ed9c920c989a4fd5c8accd1041b?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_609492" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=609492&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Marion Daly [:mdaly]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a1460790_609492" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a1460790_609492"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a1460790_609492">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 10:55 PDT" data-time="1525456521">7 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Component: General → Lint and Formatting </div> <div class="change"> Product: Core → Firefox Build System </div> </div> </div> <div class="change-set" id="c13"> <div class="comment" data-id="13255346" data-no="13" data-tags=""> <table class="layout-table change-head " id="ch-13" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_609492" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/0c9e1ed9c920c989a4fd5c8accd1041b?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_609492" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=609492&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Marion Daly [:mdaly]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-13" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-13"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c13">Comment 13</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 10:55 PDT" data-time="1525456540">7 years ago</span> </div></td> </tr> <tr id="ctag-13"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-13" data-comment-id="13255346"> This belongs under lint and formatting no? </div> </div> <div class="change-set" id="c14"> <div class="comment" data-id="13255400" data-no="14" data-tags=""> <table class="layout-table change-head " id="ch-14" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_382067" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/e560e7861c9f93a235b26565ed5c4010?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_382067" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=382067&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Andrew Halberstadt [:ahal]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-14" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-14"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c14">Comment 14</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 11:21 PDT" data-time="1525458111">7 years ago</span> </div></td> </tr> <tr id="ctag-14"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-14" data-comment-id="13255400"> This looks like <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Add a new rustfmt linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1369792&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">bug 1369792</a>, but since there are more discussions and CC's here, I'll dupe that to this. There is a WIP patch in the other bug that is probably horribly bit-rotted by now. But it could be a good starting point. However that patch tries to set rustfmt up more as a linter than a formatter, which may not be what we want (though |mach lint| has a --fix argument that could be used to do formatting too). </div> <div class="activity"> <div class="change"> Status: NEW → RESOLVED </div> <div class="change"> Closed: <span class="rel-time" title="2018-05-04 11:21 PDT" data-time="1525458111">7 years ago</span> </div> <div class="change"> Resolution: --- → FIXED </div> </div> </div> <div class="change-set" id="c16"> <div class="comment" data-id="13255405" data-no="16" data-tags=""> <table class="layout-table change-head " id="ch-16" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_382067" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/e560e7861c9f93a235b26565ed5c4010?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_382067" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=382067&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Andrew Halberstadt [:ahal]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-16" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-16"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c16">Comment 16</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 11:23 PDT" data-time="1525458185">7 years ago</span> </div></td> </tr> <tr id="ctag-16"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-16" data-comment-id="13255405"> Sorry, not sure how I managed to close this. </div> <div class="activity"> <div class="change"> Status: RESOLVED → REOPENED </div> <div class="change"> Resolution: FIXED → --- </div> </div> </div> <div class="change-set" id="c17"> <div class="comment" data-id="13255448" data-no="17" data-tags=""> <table class="layout-table change-head " id="ch-17" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_39022" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/30fa9f928422bef2f98dd5f564def686?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_39022" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=39022&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">(not currently active) Ted Mielczarek</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-17" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-17"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c17">Comment 17</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 11:44 PDT" data-time="1525459458">7 years ago</span> </div></td> </tr> <tr id="ctag-17"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-17" data-comment-id="13255448"> (In reply to Marion Daly [:mdaly] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c13">comment #13</a>) <span class="quote">> This belongs under lint and formatting no?</span> I don't think so, it's not a build system issue, but we don't really have a good component for general Rust stuff in Firefox. Mostly I just want to be clear that nobody working on the build system is going to do this work. </div> </div> <div class="change-set" id="c18"> <div class="comment" data-id="13255457" data-no="18" data-tags=""> <table class="layout-table change-head " id="ch-18" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288" id="a1460790_609492"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-18" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-18"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c18">Comment 18</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 11:48 PDT" data-time="1525459680">7 years ago</span> </div></td> </tr> <tr id="ctag-18"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-18" data-comment-id="13255457"> (In reply to Ted Mielczarek [:ted.mielczarek] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c17">comment #17</a>) <span class="quote">> (In reply to Marion Daly [:mdaly] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c13">comment #13</a>) > > This belongs under lint and formatting no? > > I don't think so, it's not a build system issue, but we don't really have a > good component for general Rust stuff in Firefox. Mostly I just want to be > clear that nobody working on the build system is going to do this work.</span> kmoir was going to create a Rust/C++ "stuff" component to provide a home for this sort of thing. </div> </div> <div class="change-set" id="c19"> <div class="comment" data-id="13255461" data-no="19" data-tags=""> <table class="layout-table change-head " id="ch-19" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_382067" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/e560e7861c9f93a235b26565ed5c4010?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_382067" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=382067&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Andrew Halberstadt [:ahal]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-19" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-19"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c19">Comment 19</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-05-04 11:50 PDT" data-time="1525459827">7 years ago</span> </div></td> </tr> <tr id="ctag-19"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-19" data-comment-id="13255461"> The entire Lint and Formatting component was only stashed on Build System because there wasn't really any better place for it. As the triage-owner of this component, I think it belongs here (though if anyone feels strongly about moving it out, I don't really care that much). </div> </div> <div class="change-set" id="c20"> <div class="comment" data-id="13446669" data-no="20" data-tags=""> <table class="layout-table change-head reporter" id="ch-20" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-20" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-20"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c20">Comment 20</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-07-03 11:40 PDT" data-time="1530643238">6 years ago</span> </div></td> </tr> <tr id="ctag-20"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-20" data-comment-id="13446669"> (In reply to Nick Cameron [:nrc] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c6">comment #6</a>) <span class="quote">> We're planning to stabilise Rustfmt (release a 1.0) this quarter, which > should include some mechanisms to work much better on CI, so it might be > worth holding out on implementing the linting/CI stuff until we have better > support in Rustfmt.</span> Seems like the consensus here is that we should do it, but wait for rustfmt to stabilize. Nick, how close are we? </div> <div class="activity"> <div class="change"> Flags: needinfo?(ncameron) </div> </div> </div> <div class="change-set" id="c21"> <div class="comment" data-id="13446672" data-no="21" data-tags=""> <table class="layout-table change-head reporter" id="ch-21" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-21" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-21"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c21">Comment 21</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-07-03 11:41 PDT" data-time="1530643283">6 years ago</span> </div></td> </tr> <tr id="ctag-21"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-21" data-comment-id="13446672"> (And I think Nathan's input is probably sufficient here unless Ehsan wants to jump in) </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(ehsan)</span> </div> </div> </div> <div class="change-set" id="c22"> <div class="comment" data-id="13446684" data-no="22" data-tags=""> <table class="layout-table change-head " id="ch-22" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-22" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-22"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c22">Comment 22</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-07-03 11:44 PDT" data-time="1530643475">6 years ago</span> </div></td> </tr> <tr id="ctag-22"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-22" data-comment-id="13446684"> We should probably use the "./mach lint" infra, it is helping a lot with development and maintenance. If we do that, we will also benefit almost at no cost from rustfmt at review phase (like the rest of the linters) </div> </div> <div class="change-set" id="c23"> <div class="comment" data-id="13446912" data-no="23" data-tags=""> <table class="layout-table change-head " id="ch-23" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_432813" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/7eb9a4f1258dbc22d0304a6aba6a1a9f?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_432813" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=432813&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nick Cameron [:nrc]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-23" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-23"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c23">Comment 23</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-07-03 13:23 PDT" data-time="1530649418">6 years ago</span> </div></td> </tr> <tr id="ctag-23"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-23" data-comment-id="13446912"> <span class="quote">> how close are we?</span> Ah such hubris on my part to predict a release so soon. We've made progress but are sill waiting on an RFC about formatting to conclude and to implement a few formatting fixes. I expect a release candidate to be on nightly by August 2nd, for 1.0 to be on nightly by early September, and 1.0 to be on stable for the edition release on October 25th. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(ncameron)</span> </div> </div> </div> <div class="change-set" id="c24"> <div class="comment" data-id="13447036" data-no="24" data-tags=""> <table class="layout-table change-head reporter" id="ch-24" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-24" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-24"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c24">Comment 24</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-07-03 14:10 PDT" data-time="1530652259">6 years ago</span> </div></td> </tr> <tr id="ctag-24"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-24" data-comment-id="13447036"> (In reply to Nick Cameron [:nrc] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c23">comment #23</a>) <span class="quote">> > how close are we? > > Ah such hubris on my part to predict a release so soon. We've made progress > but are sill waiting on an RFC about formatting to conclude and to implement > a few formatting fixes. I expect a release candidate to be on nightly by > August 2nd, for 1.0 to be on nightly by early September, and 1.0 to be on > stable for the edition release on October 25th.</span> Thanks. We probably don't want to roll anything out here until rustfmt is stable, though I may attempt a dry run with the release candidate in August if I have time. </div> </div> <div class="change-set" id="c25"> <div class="comment" data-id="13554667" data-no="25" data-tags=""> <table class="layout-table change-head reporter" id="ch-25" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-25" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-25"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c25">Comment 25</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-18 12:47 PDT" data-time="1534621658">6 years ago</span> </div></td> </tr> <tr id="ctag-25"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div id="att-9002232" class="attachment patch obsolete" data-id="9002232" itemscope itemtype="http://schema.org/MediaObject"> <meta itemprop="name" content="Bug-1454764---Format-mozilla-central-with-rustfmt-.patch"> <meta itemprop="contentSize" content="2440999"> <meta itemprop="encodingFormat" content="text/plain"> <div class="label"> Attached patch <meta itemprop="contentUrl" content="attachment.cgi?id=9002232"><a class="link" href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span id="att-9002232-description" itemprop="description">Format mozilla-central with rustfmt 1.0 RC. v1</span></a> (obsolete) — <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" itemprop="url" data-details="1">Details</a> — <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">Splinter Review</a> </div> </div> <div class="comment-text " id="ct-25" data-comment-id="13554667"> Here's the promised dry run with the release candidate for rustfmt. If anybody has any gripes about the formatting, now is the time to raise them. `cargo +nightly fmt --version` => rustfmt 0.99.2-nightly (5c9a2b6c 2018-08-07) Command: cargo +nightly fmt; pushd servo/components; for file in *; do pushd $file; cargo +nightly fmt; popd; done; popd; This currently excludes geckolib because of [1]. It also excludes webrender because that's effectively vendored code (we should format upstream at some point down the line). [1] <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/issues/2936">https://github.com/rust-lang-nursery/rustfmt/issues/2936</a> MozReview-Commit-ID: CRdDMOGExMQ </div> </div> <div class="change-set" id="c26"> <div class="comment" data-id="13554668" data-no="26" data-tags=""> <table class="layout-table change-head reporter" id="ch-26" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-26" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-26"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c26">Comment 26</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-18 12:50 PDT" data-time="1534621801">6 years ago</span> </div></td> </tr> <tr id="ctag-26"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-26" data-comment-id="13554668"> Comment on <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a> Format mozilla-central with rustfmt 1.0 RC. v1 Feedback from Nathan since I'll be requesting review of the final patch from him once rustfmt stabilizes (I don't plan to requesting review from individual module owners). Feedback from Emilio since he's had opinions about what rustfmt does, and this is probably the last chance to file bugs and get behavior changed. </div> <div class="activity"> <div class="change"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1" class="">Attachment #9002232</a> - Flags: feedback?(nfroyd) </div> <div class="change"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1" class="">Attachment #9002232</a> - Flags: feedback?(emilio) </div> </div> </div> <div class="change-set" id="c27"> <div class="comment" data-id="13554670" data-no="27" data-tags=""> <table class="layout-table change-head reporter" id="ch-27" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-27" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-27"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c27">Comment 27</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-18 12:51 PDT" data-time="1534621903">6 years ago</span> </div></td> </tr> <tr id="ctag-27"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-27" data-comment-id="13554670"> Also, per the command in <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c25">comment 25</a>, I had to manually traverse the servo directories to get cargo fmt to format them. It would be great if |cargo fmt| could provide an option to automatically traverse the dependency graph for all path dependencies. Is that feasible Nick? </div> <div class="activity"> <div class="change"> Flags: needinfo?(ncameron) </div> </div> </div> <div class="change-set" id="c28"> <div class="comment" data-id="13554700" data-no="28" data-tags=""> <table class="layout-table change-head " id="ch-28" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546716" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/1d91d22a4892f16f744b31fd4286144e?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546716" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546716&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Emilio Cobos Álvarez (:emilio)</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-28" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-28"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c28">Comment 28</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-18 13:41 PDT" data-time="1534624861">6 years ago</span> </div></td> </tr> <tr id="ctag-28"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-28" data-comment-id="13554700"> Comment on <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a> Format mozilla-central with rustfmt 1.0 RC. v1 Review of <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a>: ----------------------------------------------------------------- I'm generally ok with this, but we need to align with Servo, somehow, because I still sync code back and forth. Servo has the following in its rustfmt.toml: <span class="quote">> match_block_trailing_comma = true > binop_separator = "Back" > # Turn off import reordering, since the new algorithm clashes with tidy. > reorder_imports = false</span> I'm fine with import reordering, but we need to remove the equivalent tidy lint in Servo. Interestingly I posted <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/UqFr8rRNAAAJ">https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/UqFr8rRNAAAJ</a> this morning, so if people agree... :) I think I prefer binop_separator = "Back" as well, since that's what most of us use and is closer to the Gecko style, but I don't mind much either way. I couldn't care less about trailing comma vs. not. In any case, this diff is way more noisy than what it needs to because last time you reformatted Servo using a different style. ::: servo/components/style/values/generics/rect.rs @@ +20,5 @@ <span class="quote">> + MallocSizeOf, > + PartialEq, > + SpecifiedValueInfo, > + ToComputedValue, > +)]</span> This kind of thing is going to add so many lines to the style system... :) </div> <div class="activity"> <div class="change"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1" class="">Attachment #9002232</a> - Flags: <span class="activity-deleted">feedback?(emilio)</span> </div> </div> </div> <div class="change-set" id="c29"> <div class="comment" data-id="13555116" data-no="29" data-tags=""> <table class="layout-table change-head reporter" id="ch-29" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-29" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-29"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c29">Comment 29</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-19 11:13 PDT" data-time="1534702430">6 years ago</span> </div></td> </tr> <tr id="ctag-29"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-29" data-comment-id="13555116"> (In reply to Emilio Cobos Álvarez (:emilio) from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c28">comment #28</a>) <span class="quote">> Comment on <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a> > Format mozilla-central with rustfmt 1.0 RC. v1 > > Review of <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a>: > ----------------------------------------------------------------- > > I'm generally ok with this, but we need to align with Servo, somehow, > because I still sync code back and forth. > > Servo has the following in its rustfmt.toml: > > > match_block_trailing_comma = true > > binop_separator = "Back" > > # Turn off import reordering, since the new algorithm clashes with tidy. > > reorder_imports = false > > I'm fine with import reordering, but we need to remove the equivalent tidy > lint in Servo. Interestingly I posted > <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/UqFr8rRNAAAJ">https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/UqFr8rRNAAAJ</a> > this morning, so if people agree... :)</span> Yeah - commented on that thread. <span class="quote">> I think I prefer binop_separator = "Back" as well, since that's what most of > us use and is closer to the Gecko style, but I don't mind much either way.</span> I'll defer to Nathan - see below. <span class="quote">> I couldn't care less about trailing comma vs. not. In any case, this diff is > way more noisy than what it needs to because last time you reformatted Servo > using a different style.</span> Yeah - at the time I wasn't thinking as hard about the advantages of using standard Rust style, so I just used Servo's config. Given that people are unlikely to have a strong opinion about trailing commas, I think we probably should accept the default on that one. Nathan, thoughts on binop separator? Consistency with the rest of Rust is nice, though I think our existing code leans towards first line. The Mozilla style guide specifies first line for C++, and something more complicated for JS [1]. Google C++ leaves it unspecified [2]. Nick, is there a compelling reason for the default value of binop separator in rustfmt? [1] <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style%23Naming_and_Formatting_code">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#Naming_and_Formatting_code</a> [2] <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://google.github.io/styleguide/cppguide.html%23Boolean_Expressions">https://google.github.io/styleguide/cppguide.html#Boolean_Expressions</a> </div> </div> <div class="change-set" id="c30"> <div class="comment" data-id="13555309" data-no="30" data-tags=""> <table class="layout-table change-head reporter" id="ch-30" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-30" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-30"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c30">Comment 30</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-19 18:22 PDT" data-time="1534728170">6 years ago</span> </div></td> </tr> <tr id="ctag-30"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-30" data-comment-id="13555309"> Also NI jdm for thoughts, since we'd need to align servo and gecko's rustfmt configs if we want emilio's syncing job to stay easy. My gut sense is that the |match_block_trailing_comma| doesn't matter much and we should just use the rustfmt default, absent any concrete reasons to do otherwise. Less sure about binop_separator. </div> </div> <div class="change-set" id="c31"> <div class="comment" data-id="13557582" data-no="31" data-tags=""> <table class="layout-table change-head " id="ch-31" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546716" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/1d91d22a4892f16f744b31fd4286144e?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546716" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546716&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Emilio Cobos Álvarez (:emilio)</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-31" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-31"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c31">Comment 31</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-20 05:55 PDT" data-time="1534769737">6 years ago</span> </div></td> </tr> <tr id="ctag-31"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-31" data-comment-id="13557582"> (In reply to Bobby Holley (:bholley) from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c30">comment #30</a>) <span class="quote">> Also NI jdm for thoughts, since we'd need to align servo and gecko's rustfmt > configs if we want emilio's syncing job to stay easy. > > My gut sense is that the |match_block_trailing_comma| doesn't matter much > and we should just use the rustfmt default, absent any concrete reasons to > do otherwise. Less sure about binop_separator.</span> Forgot to ni? I assume :) </div> <div class="activity"> <div class="change"> Flags: needinfo?(josh) </div> </div> </div> <div class="change-set" id="c32"> <div class="comment" data-id="13557674" data-no="32" data-tags=""> <table class="layout-table change-head " id="ch-32" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_466787" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/2ac711ca4c5ab8c8545725bcd618c037?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_466787" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=466787&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Dan Minor [:dminor]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-32" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-32"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c32">Comment 32</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-20 06:30 PDT" data-time="1534771841">6 years ago</span> </div></td> </tr> <tr id="ctag-32"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-32" data-comment-id="13557674"> Please also exclude rsdparsa (media/webrtc/signaling/src/sdp/rsdparsa) for now. It is being developed on github [1] and we have upstream commits that have not been merged into mozilla-central yet. We'd be happy to run rustfmt on upstream before we merge again. Thanks! [1] <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/nils-ohlmeier/rsdparsa">https://github.com/nils-ohlmeier/rsdparsa</a> </div> </div> <div class="change-set" id="c33"> <div class="comment" data-id="13558215" data-no="33" data-tags=""> <table class="layout-table change-head " id="ch-33" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_241497" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/b13611d1950fd03a3e1d44195fa6f532?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_241497" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=241497&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Josh Matthews [:jdm]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-33" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-33"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c33">Comment 33</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-20 09:20 PDT" data-time="1534782058">6 years ago</span> </div></td> </tr> <tr id="ctag-33"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-33" data-comment-id="13558215"> We've started merging PRs to format Servo's components directory by directory as part of a plan to move to enforcing rustfmt style on all of our code. I'm fine with aligning with Gecko's style choices, but it would be nice if we could wait on reordering imports until Servo's ready to flip the switch (per <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/ia9CXcLSAAAJ">https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/ia9CXcLSAAAJ</a>). </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(josh)</span> </div> </div> </div> <div class="change-set" id="c34"> <div class="comment" data-id="13562186" data-no="34" data-tags=""> <table class="layout-table change-head " id="ch-34" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_432813" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/7eb9a4f1258dbc22d0304a6aba6a1a9f?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_432813" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=432813&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nick Cameron [:nrc]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-34" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-34"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c34">Comment 34</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-21 20:00 PDT" data-time="1534906842">6 years ago</span> </div></td> </tr> <tr id="ctag-34"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-34" data-comment-id="13562186"> <span class="quote">> It would be great if |cargo fmt| could provide an option to automatically traverse the dependency graph for all path dependencies. Is that feasible Nick?</span> Hmm, not sure. For normal Rust usage, you wouldn't want to format deps since these would be read-only downloaded from crates.io. It does make sense to format a whole workspace, so if Servo is a single Cargo workspace, we should implement that. Otherwise, it probably isn't too hard to write a small program to discover deps and format them. <span class="quote">> is there a compelling reason for the default value of binop separator in rustfmt?</span> It's discussed on this GH thread. tl;dr it makes code easier to scan because running the eye down the lhs is easier than running it down the rhs. Note that the Rustfmt 1.0 stability guarantees only apply to the default options. However, in practice I expect that small tweaks like the binop position are likely to be stable. If you end up with any major departures we should work to try and ensure stability of those options in rustfmt. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(ncameron)</span> </div> </div> </div> <div class="change-set" id="c35"> <div class="comment" data-id="13563731" data-no="35" data-tags=""> <table class="layout-table change-head reporter" id="ch-35" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-35" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-35"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c35">Comment 35</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-22 09:01 PDT" data-time="1534953662">6 years ago</span> </div></td> </tr> <tr id="ctag-35"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-35" data-comment-id="13563731"> (In reply to Josh Matthews [:jdm] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c33">comment #33</a>) <span class="quote">> We've started merging PRs to format Servo's components directory by > directory as part of a plan to move to enforcing rustfmt style on all of our > code. I'm fine with aligning with Gecko's style choices,</span> Ok great, sounds like it's up to Nathan then. Looks like I forgot to NI him in <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c29">comment 29</a>. <span class="quote">> but it would be > nice if we could wait on reordering imports until Servo's ready to flip the > switch (per > <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/ia9CXcLSAAAJ">https://groups.google.com/d/msg/mozilla.dev.servo/-06meTc7H1k/ia9CXcLSAAAJ</a>).</span> That seems fine. I think we wouldn't enable rustfmt until 1.0 hit stable Rust, which will be some months. So Servo will probably beat Gecko to that milestone, assuming efforts there don't stall. </div> <div class="activity"> <div class="change"> Flags: needinfo?(nfroyd) </div> </div> </div> <div class="change-set" id="c36"> <div class="comment" data-id="13569737" data-no="36" data-tags=""> <table class="layout-table change-head " id="ch-36" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288" id="a1460790_609492"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-36" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-36"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c36">Comment 36</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-24 13:06 PDT" data-time="1535141219">6 years ago</span> </div></td> </tr> <tr id="ctag-36"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-36" data-comment-id="13569737"> Comment on <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a> Format mozilla-central with rustfmt 1.0 RC. v1 Review of <span class="bz_obsolete"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" name="attach_9002232" title="Format mozilla-central with rustfmt 1.0 RC. v1">attachment 9002232</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[details]</a> <a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=diff&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1">[diff]</a></span> <a href="https://bugzilla-mozilla-org.translate.goog/page.cgi?id=splinter.html&ignore&bug=1454764&attachment=9002232&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">[review]</a>: ----------------------------------------------------------------- This was with the default rustfmt settings, correct? Maybe it's because I haven't used it enough, but just reading through the diff, I'm finding it kind of hard to formulate a mental heuristic of what rustfmt will do. I've noted a couple of examples below. I have not looked at the entire diff. I realize that the whole point of automatic formatting is that you don't really have to think about it, but it seems valuable to have people have a pretty good idea of what might get changed about their code. Maybe there are rules that people internalize and I'm just not there yet. The fairly-consistent expansion of things into multiple lines seems unfortunate. I have a slight preference for match_block_trailing_comma = true for consistency with...well, everything else that's a list of stuff, really. binop_separator should /probably/ be "Back" for consistency with Gecko? I don't think that's great for consistency with the wider ecosystem, but I think it eases some of the mental burden of switching between C++ and Rust. (Though JS uses "Front", apparently, so maybe consistency can just get thrown out the window.) ::: intl/encoding_glue/src/lib.rs @@ -529,5 @@ <span class="quote">> return (NS_OK, output_encoding); > } > CoderResult::OutputFull => { > - if let Some(needed) = > - checked_add(total_written,</span> This version seems slightly easier to scan than the rustfmt version. ::: js/rust/build.rs @@ +41,5 @@ <span class="quote">> > for entry in entries { > if let Ok(path) = entry { > + return path > + .canonicalize()</span> It seems bizarre to me that rustfmt expands this line, but contracts the bits above. What is going on here? This one is expanded because of the following .expect(), and chained function calls in a single expression call for expanding them? ::: servo/components/selectors/parser.rs @@ +2373,5 @@ <span class="quote">> + Ok(SelectorList::from_vec(vec![Selector::from_vec( > + vec![Component::LocalName(LocalName { > + name: DummyAtom::from("EeÉ"), > + lower_name: DummyAtom::from("eeÉ"), > + }),],</span> This trailing comma after the `vec!` element smooshed together with everything else seems suboptimal. This is much more compressed than I would expect from rustfmt. @@ +2388,5 @@ <span class="quote">> + lower_name: DummyAtom::from("e"), > + }), > + ], > + specificity(0, 0, 1), > + ),]))</span> This whole block right here is virtually identical to the above (one extra element in the innermost `vec!`, I guess), but the formatting is much more pleasing. Still with the smooshed comma at the end, though. It seems not good that relatively minor changes lead to large differences in formatting. @@ +2425,5 @@ <span class="quote">> ); > assert_eq!( > parse("*"), > + Ok(SelectorList::from_vec(vec![Selector::from_vec( > + vec![Component::ExplicitUniversalType],</span> Why doesn't this single-element vector have a trailing comma, as the example just a couple lines above, or the `Selector::from_vec()` element in the outermost `vec!` here? ::: servo/components/style/animation.rs @@ +507,5 @@ <span class="quote">> Some(previous_style), > Some(previous_style), > font_metrics_provider, > + CascadeMode::Unvisited { > + visited_rules: None,</span> I don't get why this needs to be expanded. ::: servo/components/style/counter_style/mod.rs @@ +496,5 @@ <span class="quote">> (opt_start, opt_end) > { > if start > end { > + return Err( > + input.new_custom_error(StyleParseErrorKind::UnspecifiedError)</span> What rule splits this up into multiple lines? ::: servo/components/style/gecko/media_features.rs @@ +41,5 @@ <span class="quote">> fn device_size(device: &Device) -> Size2D<Au> { > let mut width = 0; > let mut height = 0; > unsafe { > + bindings::Gecko_MediaFeatures_GetDeviceSize(device.document(), &mut width, &mut height);</span> I feel like there are multiple examples in this diff where rustfmt makes the exact opposite transformation of that being done here: it takes a call that fits just fine on a single line--shorter even than this one!--and expanding it to multiple lines. ::: servo/components/style/gecko/wrapper.rs @@ +1244,5 @@ <span class="quote">> } > > fn owner_doc_matches_for_testing(&self, device: &Device) -> bool { > + self.as_node().owner_doc().0 as *const structs::nsIDocument == device > + .pres_context()</span> This formatting of `device` on the previous line really hides what's going on here, IMHO. @@ +1869,5 @@ <span class="quote">> )); > } > > + let active = self > + .state()</span> I don't get the rules for chained function call expressions. Sometimes they start on different lines (here), sometimes the second call goes on a different line (below, with the debug_assert!), sometimes they all stay on the same line (above, owner_doc_matches_for_testing). @@ +2093,5 @@ <span class="quote">> > + debug_assert!( > + self.as_node() > + .parent_node() > + .map_or(false, |p| p.is_document())</span> This seems like reasonable chaining formatting...although the line was already short enough as-is. @@ +2100,5 @@ <span class="quote">> } > > fn is_empty(&self) -> bool { > + !self > + .as_node()</span> This change seems weird, given the debug_assert! formatting above. Why the difference? ::: servo/components/style/media_queries/media_list.rs @@ +29,5 @@ <span class="quote">> /// found, the media query list is only filled with the equivalent of > /// "not all", see: > /// > /// <<a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://drafts.csswg.org/mediaqueries/%23error-handling">https://drafts.csswg.org/mediaqueries/#error-handling</a>> > + pub fn parse(context: &ParserContext, input: &mut Parser) -> Self {</span> See the below comment in media_query.rs. Why does rustfmt compress this onto a single line, but expand the below into multiple lines? ::: servo/components/style/media_queries/media_query.rs @@ +143,5 @@ <span class="quote">> let media_type = explicit_media_type.unwrap_or(MediaQueryType::All); > + Ok(Self { > + qualifier, > + media_type, > + condition,</span> What is the bit that controls formatting this? `use_small_heuristics`? This change seems like unnecessary expansion of code. ::: testing/mozbase/rust/mozprofile/src/prefreader.rs @@ +28,5 @@ <span class="quote">> fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { > + write!( > + f, > + "{} at line {}, column {}", > + self.message, self.position.line, self.position.column</span> So this call gets split into multiple lines, but some arguments are on the same line, which seems kind of abnormal after looking at other similar calls. Is this because rustfmt knows something about the semantics of `write!`? @@ +180,5 @@ <span class="quote">> let data = match str::from_utf8(&buf[self.start_pos..end_pos]) { > Ok(x) => x, > Err(_) => { > + return Err(PrefReaderError::new( > + "Could not convert string to utf8",</span> This sort of splitting over multiple lines seems unnecessary. Is it because of the nested function calls, multiple arguments, or some combination of the two? ::: testing/webdriver/src/common.rs @@ +46,5 @@ <span class="quote">> } > > pub fn map<F, U: ToJson>(self, f: F) -> Nullable<U> > + where > + F: FnOnce(T) -> U,</span> It seems a little straight that having function declarations with multiple arguments can fit on a single line, but `where` predicates are forced to be multi-line like this. ::: testing/webdriver/src/httpapi.rs @@ +23,5 @@ <span class="quote">> + (Get, "/session/{sessionId}/window", Route::GetWindowHandle), > + ( > + Get, > + "/session/{sessionId}/window/handles", > + Route::GetWindowHandles,</span> Why do this? This line is already well within `max_width`. @@ +238,5 @@ <span class="quote">> CloseWindow, > + GetWindowSize, // deprecated > + SetWindowSize, // deprecated > + GetWindowPosition, // deprecated > + SetWindowPosition, // deprecated</span> This feature is nice, but it seems contrary to our ignorning alignment in other contexts. I know Emacs has bindings for automatically aligning trailing comments like this (though I think its defaults are much further right), but I don't know about our other editors. ::: testing/webdriver/src/response.rs @@ +56,5 @@ <span class="quote">> impl CloseWindowResponse { > pub fn new(handles: Vec<String>) -> CloseWindowResponse { > + CloseWindowResponse { > + window_handles: handles, > + }</span> OK, why does this get expanded... @@ +110,5 @@ <span class="quote">> } > > impl ValueResponse { > pub fn new(value: json::Json) -> ValueResponse { > + ValueResponse { value: value }</span> But this gets contracted? Aren't they basically examples of the exact same thing, number of tokens and everything? ::: xpcom/rust/xpcom/src/base.rs @@ +33,5 @@ <span class="quote">> unsafe { > + if (*(self as *const Self as *const nsISupports)) > + .QueryInterface(&T::IID, ga.void_ptr()) > + .succeeded() > + {</span> I don't get where this rule that the brace gets put on its own line comes from. AFAICT from `control_brace_style`, this shouldn't happen? </div> <div class="activity"> <div class="change"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1" class="">Attachment #9002232</a> - Flags: feedback?(nfroyd) → feedback+ </div> </div> </div> <div class="change-set" id="c37"> <div class="comment" data-id="13569740" data-no="37" data-tags=""> <table class="layout-table change-head " id="ch-37" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288" id="a1460790_609492"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-37" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-37"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c37">Comment 37</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-24 13:07 PDT" data-time="1535141236">6 years ago</span> </div></td> </tr> <tr id="ctag-37"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-37" data-comment-id="13569740"> Commented in <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c36">comment 36</a>. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(nfroyd)</span> </div> </div> </div> <div class="change-set" id="c38"> <div class="comment" data-id="13570028" data-no="38" data-tags=""> <table class="layout-table change-head reporter" id="ch-38" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-38" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-38"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c38">Comment 38</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-24 15:34 PDT" data-time="1535150062">6 years ago</span> </div></td> </tr> <tr id="ctag-38"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-38" data-comment-id="13570028"> This is great feedback. Passing it on to Nick. </div> <div class="activity"> <div class="change"> Flags: needinfo?(ncameron) </div> </div> </div> <div class="change-set" id="c39"> <div class="comment" data-id="13570030" data-no="39" data-tags=""> <table class="layout-table change-head reporter" id="ch-39" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_313730" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/ac57223c49d41049a4917e187b854377?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_313730" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div><span class="user-role">Reporter</span></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-39" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-39"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c39">Comment 39</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-24 15:35 PDT" data-time="1535150120">6 years ago</span> </div></td> </tr> <tr id="ctag-39"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-39" data-comment-id="13570030"> (In reply to Nathan Froyd [:froydnj] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c36">comment #36</a>) <span class="quote">> This was with the default rustfmt settings, correct?</span> Oh, and yes. </div> </div> <div class="change-set" id="c40"> <div class="comment" data-id="13570836" data-no="40" data-tags=""> <table class="layout-table change-head " id="ch-40" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_432813" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/7eb9a4f1258dbc22d0304a6aba6a1a9f?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_432813" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=432813&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nick Cameron [:nrc]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-40" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-40"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c40">Comment 40</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-26 00:41 PDT" data-time="1535269270">6 years ago</span> </div></td> </tr> <tr id="ctag-40"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-40" data-comment-id="13570836"> A couple of general points: commas can have semantic significance in macros, so we never add or remove them. If you want a trailing comma you need to add it manually and then Rustfmt will preserve it. A lot of the questions are about why an expression gets put on multiple lines when it doesn't need to - we found that some expression were *much* more readable when put on separate lines. We also found that people were doing this in practice without Rustfmt (usually based on complexity rather than absolute width, but that is hard to model). We use the number of characters as the heuristic for this - it's not perfect, but it comes out pretty well in practice. The same applies to chains of method calls/field accesses. One oddity with these is that if we go multi-line, then we usually put a newline before the first `.`. However, if that leaves the first line shorter than the indent of the second line it looks weird, so we put it all on one line. Hopefully I'll answer the rest below and not miss any <span class="quote">> It seems bizarre to me that rustfmt expands this line, but contracts the bits above. What is going on here?</span> There are two different rules acting here: if we can't put a chain on one line, then we split it one item per line (we nearly always do something like this, we avoid multiple lines and multiple items per line). When we get a `let ... = ...` statement (or any other assignment), we try putting the rhs on its own line, if that doesn't work then we keep the start of the rhs on the same line as the lhs (and go multiline from there). In this case the first chain fits just below the width heuristic, so we can split the assignment and keep the chain on one line. For the second chain, we're just above the width heuristic or maybe exceeding the max width. <span class="quote">> This whole block right here is virtually identical to the above (one extra element in the innermost `vec!`, I guess)</span> We have a 'combining rule' where there is only a single argument in a function call/macro (and some other expressions) we'll combine the first lines. This prevents things like: ``` foo( foo_bar( baz_baz( Foo { x: String, } ) ) ) // Rustfmt gives: foo(foo_bar(baz_baz(Foo { x: String, }))) ``` This can go a bit overboard sometimes, but in most cases its an improvement. <span class="quote">> This formatting of `device` on the previous line really hides what's going on here, IMHO.</span> Agreed: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/rustfmt/issues/2967">https://github.com/rust-lang-nursery/rustfmt/issues/2967</a> Although I'm not sure if we'll be able to come up with a fix (binops are suprisingly complicated because the AST does not match the intuition when you have say `a + b + c + d`). <span class="quote">> Why does rustfmt compress this onto a single line, but expand the below into multiple lines?</span> Function declarations are formatted differently to expressions, the latter usually has a width heuristic as well as the max width. <span class="quote">> What is the bit that controls formatting this? `use_small_heuristics`? This change seems like unnecessary expansion of code.</span> Yes. It's not perfect, but things look a lot worse without it - edge cases still go wrong, but it means the edge cases aren't as extreme and thus things look better. <span class="quote">> Is this because rustfmt knows something about the semantics of `write!`?</span> Sort of - it knows that a format string followed by some expressions is a common pattern and formats that specially. <span class="quote">> but `where` predicates are forced to be multi-line like this.</span> `where` clauses were a pain to get right. In the end we went for always putting each component on it's own line because `where` clauses should only be used with complicated constraints (if they are simple you can put them on the declaration of generic args), and when they are complex, then it really aids reading to have one per line. We even do this with just one constraint, because otherwise the indentation is bad or things look inconsistent. <span class="quote">> this feature is nice, but it seems contrary to our ignorning alignment in other contexts.</span> it's kind of an odd rule, it doesn't fit very well with everything else, but always using n spaces looks horrible. In general I would recommend putting comments above each variant, rather than to the right. <span class="quote">> OK, why does this get expanded...</span> The heuristic width for struct literals is very low. This kind of case is sub-optimal, but in general spread out struct literals are a big win. <span class="quote">> I don't get where this rule that the brace gets put on its own line comes from</span> So, for some item `blah blah { ... }` (control flow, declarations, whatever) we nearly always leave the `{` trailing. However, if `blah blah` goes multiline, we put `{` unindented on it's own line. The reason is that the second and subsequent lines of `blah blah` will be indented and so is the body (`...`). If we trail `{` then there is no indentation difference so it is hard to scan since the header expression blends in with the body. <span class="quote">> This trailing comma after the `vec!` element smooshed together with everything else seems suboptimal.</span> This is fixed on master, I get: ``` Ok(SelectorList::from_vec(vec![Selector::from_vec( vec![Component::LocalName(LocalName { name: DummyAtom::from("e"), lower_name: DummyAtom::from("e"), })], specificity(0, 0, 1), )])) ``` I hope that covers everything. Most of the discussion on the formatting rules is here: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/fmt-rfcs/issues?utf8%3D%25E2%259C%2593%26q%3Dis%253Aissue">https://github.com/rust-lang-nursery/fmt-rfcs/issues?utf8=%E2%9C%93&q=is%3Aissue</a> </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(ncameron)</span> </div> </div> </div> <div class="change-set" id="c41"> <div class="comment" data-id="13574536" data-no="41" data-tags=""> <table class="layout-table change-head " id="ch-41" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_417288" id="a1460790_609492"> <img src="extensions/Gravatar/web/default.jpg" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_417288" id="a1460790_609492"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=417288&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nathan Froyd [:froydnj]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-41" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-41"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c41">Comment 41</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-27 13:45 PDT" data-time="1535402730">6 years ago</span> </div></td> </tr> <tr id="ctag-41"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-41" data-comment-id="13574536"> Thanks for the explanations! (In reply to Nick Cameron [:nrc] from <a class="bz_bug_link bz_status_REOPENED" title="REOPENED - [meta] rustfmt mozilla-central" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c40">comment #40</a>) <span class="quote">> A lot of the questions are about why an expression gets put on multiple > lines when it doesn't need to - we found that some expression were *much* > more readable when put on separate lines. We also found that people were > doing this in practice without Rustfmt (usually based on complexity rather > than absolute width, but that is hard to model). We use the number of > characters as the heuristic for this - it's not perfect, but it comes out > pretty well in practice.</span> Do you have a pointer to discussion for this? Number of characters seems right, but I'd expect some sort of "complexity" metric to kick in first. I expect there are good reasons for this. <span class="quote">> > This whole block right here is virtually identical to the above (one extra element in the innermost `vec!`, I guess) > > We have a 'combining rule' where there is only a single argument in a > function call/macro (and some other expressions) we'll combine the first > lines. This prevents things like: > > ``` > foo( > foo_bar( > baz_baz( > Foo { > x: String, > } > ) > ) > ) > > // Rustfmt gives: > foo(foo_bar(baz_baz(Foo { > x: String, > }))) > ``` > > This can go a bit overboard sometimes, but in most cases its an improvement.</span> OK, thanks for the explanation. This one seems a bit unfortunate that `x:` lines up with what could be `foo`'s arguments, but oh well. <span class="quote">> > OK, why does this get expanded... > > The heuristic width for struct literals is very low. This kind of case is > sub-optimal, but in general spread out struct literals are a big win.</span> This seems like an unfortunate heuristic for this case. Do you have a pointer to some discussion for this? I found: <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/fmt-rfcs/issues/64">https://github.com/rust-lang-nursery/fmt-rfcs/issues/64</a> (struct literals, which seems to suggest that things should be placed on one line if possible) <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/fmt-rfcs/issues/47">https://github.com/rust-lang-nursery/fmt-rfcs/issues/47</a> (which I think agrees with not expanding things so eagerly, but it's hard to tell) but either I skimmed the discussions too fast, or there is some other discussion I couldn't find, since rustfmt doesn't agree with my understanding of those issues. (Or I have completely misunderstood the discussions!) <span class="quote">> > This trailing comma after the `vec!` element smooshed together with everything else seems suboptimal. > > This is fixed on master, I get: > > ``` > Ok(SelectorList::from_vec(vec![Selector::from_vec( > vec![Component::LocalName(LocalName { > name: DummyAtom::from("e"), > lower_name: DummyAtom::from("e"), > })], > specificity(0, 0, 1), > )])) > ```</span> Cool, that looks about right. Did the single-element case for the inner vec! get fixed as well as a byproduct? Did you have a comment on this particular one? I find it a little hard to understand why rustfmt put all this on one line, while other relatively similar examples got expanded: ::: servo/components/style/gecko/media_features.rs @@ +41,5 @@ <span class="quote">> fn device_size(device: &Device) -> Size2D<Au> { > let mut width = 0; > let mut height = 0; > unsafe { > + bindings::Gecko_MediaFeatures_GetDeviceSize(device.document(), &mut width, &mut height);</span> </div> <div class="activity"> <div class="change"> Flags: needinfo?(ncameron) </div> </div> </div> <div class="change-set" id="c42"> <div class="comment" data-id="13574947" data-no="42" data-tags=""> <table class="layout-table change-head " id="ch-42" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_432813" id="a1460790_609492"> <img src="https://secure.gravatar.com/avatar/7eb9a4f1258dbc22d0304a6aba6a1a9f?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_432813" id="a1460790_609492"> <a class="email disabled" href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=432813&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Nick Cameron [:nrc]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-42" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-42"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c42">Comment 42</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-08-27 19:28 PDT" data-time="1535423282">6 years ago</span> </div></td> </tr> <tr id="ctag-42"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-42" data-comment-id="13574947"> <span class="quote">> Do you have a pointer to discussion for this? Number of characters seems right, but I'd expect some sort of "complexity" metric to kick in first. I expect there are good reasons for this.</span> Discussion was here - <a rel="nofollow" href="https://translate.google.com/website?sl=pl&tl=iw&hl=en-GB&u=https://github.com/rust-lang-nursery/fmt-rfcs/issues/47">https://github.com/rust-lang-nursery/fmt-rfcs/issues/47</a> - and a lot more happened in meetings. We did prefer a complexity metric rather than a character count. However, our attempts to specify an algorithm got complex really fast. In the end we left it as an implementation detail for the tool to decide. For Rustfmt, we've stuck with char count because it is simple. We would like to implement a complexity metric, but not for 1.0 (hopefully for 2.0) - implementation is probably not too hard, but from experience it takes a lot of tweaking to get satisfactory parameters. <span class="quote">> Do you have a pointer to some discussion for this?</span> I think #64 would be where it was discussed, but it may have happened in video meetings. We found that in general people used multiline structs, so that should be the default, and that putting it on line should be exceptional, so the heuristic for that is pretty strict. This has been a bit controversial, people who think of struct literals like function calls (i.e., single-line by default) tend to find it odd, whereas people who think of struct literals like struct declarations with values find it more natural. <span class="quote">> Did the single-element case for the inner vec! get fixed as well as a byproduct?</span> I think this is the single-element case? I might be reading it wrong. <span class="quote">> Did you have a comment on this particular one? I find it a little hard to understand why rustfmt put all this on one line, while other relatively similar examples got expanded:</span> The whole line is under the max width and the heuristic we check applies only to he arguments, `device.document(), &mut width, &mut height` is fairly short. </div> <div class="activity"> <div class="change"> Flags: <span class="activity-deleted">needinfo?(ncameron)</span> </div> </div> </div> <div class="change-set" id="a12264925_382067"> <div class="change" id="aa12264925_382067"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_382067" id="a12264925_382067"> <img src="https://secure.gravatar.com/avatar/e560e7861c9f93a235b26565ed5c4010?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_382067" id="a12264925_382067"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=382067&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Andrew Halberstadt [:ahal]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a12264925_382067" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a12264925_382067"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a12264925_382067">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-09-06 12:04 PDT" data-time="1536260656">6 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Blocks: <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Add a new rustfmt linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1369792&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1369792</a> </div> </div> </div> <div class="change-set" id="c43"> <div class="comment" data-id="13745816" data-no="43" data-tags=""> <table class="layout-table change-head " id="ch-43" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546716" id="a12264925_382067"> <img src="https://secure.gravatar.com/avatar/1d91d22a4892f16f744b31fd4286144e?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546716" id="a12264925_382067"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546716&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Emilio Cobos Álvarez (:emilio)</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-43" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-43"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c43">Comment 43</a></h3> • <div class="change-time"> <span class="rel-time" title="2018-11-08 15:41 PST" data-time="1541720467">6 years ago</span> </div></td> </tr> <tr id="ctag-43"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text " id="ct-43" data-comment-id="13745816"> FWIW the Servo rustfmt happened and is in mozilla-central now. In <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED DUPLICATE - Add a new rustfmt linter" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1369792&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">bug 1369792</a> I want to figure out a way to use use the same rustfmt version between Servo and Gecko so that I can keep keeping them in sync, and add a lint to the servo/ directory. Once that's done we can probably proceed here without much problem. </div> </div> <div class="change-set" id="c44"> <div class="comment" data-id="14662338" data-no="44" data-tags=""> <table class="layout-table change-head " id="ch-44" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a12264925_382067"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a12264925_382067"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-44" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-44"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c44">Comment 44</a></h3> • <div class="change-time"> <span class="rel-time" title="2020-02-26 01:36 PST" data-time="1582709792">5 years ago</span> </div></td> </tr> <tr id="ctag-44"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text markdown-body " id="ct-44" data-comment-id="14662338"> <p>Started some of the work in <a class="bz_bug_link bz_status_ASSIGNED" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1617369&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="ASSIGNED - [rust] Reformat the tree using rustfmt">bug 1617369</a></p> </div> </div> <div class="change-set" id="c45"> <div class="comment" data-id="14680773" data-no="45" data-tags=""> <table class="layout-table change-head " id="ch-45" role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a12264925_382067"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a12264925_382067"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="cs-45" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="cr-45"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#c45">Comment 45</a></h3> • <div class="change-time"> <span class="rel-time" title="2020-03-06 09:30 PST" data-time="1583515827">5 years ago</span> </div></td> </tr> <tr id="ctag-45"> <td colspan="2" class="comment-tags"></td> </tr> </tbody> </table> </div> <div class="comment-text markdown-body " id="ct-45" data-comment-id="14680773"> <p>Let's morph this bug into a meta bug</p> </div> <div class="activity"> <div class="change"> Depends on: <a class="bz_bug_link bz_status_ASSIGNED" title="ASSIGNED - [rust] Reformat the tree using rustfmt" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1617369&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1617369</a>, <a class="bz_bug_link bz_status_RESOLVED bz_closed" title="RESOLVED FIXED - Add support of rustfmt in mozlint" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1551078&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1551078</a> </div> <div class="change"> Keywords: <a href="https://bugzilla-mozilla-org.translate.goog/buglist.cgi?keywords=meta&resolution=---&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">meta</a> </div> <div class="change"> Summary: rustfmt mozilla-central → [meta] rustfmt mozilla-central </div> </div> </div> <div class="change-set" id="a59520180_495955"> <div class="change" id="aa59520180_495955"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a59520180_495955"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a59520180_495955"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a59520180_495955" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a59520180_495955"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a59520180_495955">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2020-03-06 09:31 PST" data-time="1583515911">5 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Alias: rustfmt </div> </div> </div> <div class="change-set" id="a63811752_495955"> <div class="change" id="aa63811752_495955"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a63811752_495955"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a63811752_495955"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a63811752_495955" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a63811752_495955"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a63811752_495955">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2020-04-25 02:38 PDT" data-time="1587807483">5 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"><a href="https://bugzilla-mozilla-org.translate.goog/attachment.cgi?id=9002232&action=edit&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB" title="Format mozilla-central with rustfmt 1.0 RC. v1" class="">Attachment #9002232</a> - Attachment is obsolete: true </div> </div> </div> <div class="change-set" id="a70826249_495955"> <div class="change" id="aa70826249_495955"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_495955" id="a70826249_495955"> <img src="https://secure.gravatar.com/avatar/5aa2e382197a147b42d020b6878f8623?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_495955" id="a70826249_495955"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=495955&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Sylvestre Ledru [:Sylvestre]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a70826249_495955" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a70826249_495955"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a70826249_495955">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2020-07-15 07:06 PDT" data-time="1594821980">4 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Depends on: <a class="bz_bug_link bz_status_NEW" title="NEW - rustfmt could be a bit more polite" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1652981&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1652981</a> </div> </div> </div> <div class="change-set" id="a86345019_293943"> <div class="change" id="aa86345019_293943"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_293943" id="a86345019_293943"> <img src="https://secure.gravatar.com/avatar/711e7c6f7c2c62a7bc2eb760585c3daa?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_293943" id="a86345019_293943"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=293943&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Markus Stange [:mstange]</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a86345019_293943" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a86345019_293943"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a86345019_293943">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2021-01-10 20:52 PST" data-time="1610340750">4 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Depends on: <a class="bz_bug_link bz_status_NEW" title="NEW - Stop treating gfx/wr as third-party code" href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1685948&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">1685948</a> </div> </div> </div> <div class="change-set" id="a136774322_546015"> <div class="change" id="aa136774322_546015"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546015" id="a136774322_546015"> <img src="https://secure.gravatar.com/avatar/d0465b8931b44330f542efbeeae855da?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546015" id="a136774322_546015"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546015&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">BMO Automation</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a136774322_546015" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a136774322_546015"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a136774322_546015">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2022-08-17 14:00 PDT" data-time="1660770053">2 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Product: Firefox Build System → Developer Infrastructure </div> </div> </div> <div class="change-set" id="a141528094_546015"> <div class="change" id="aa141528094_546015"> <table class="layout-table change-head " role="presentation"> <tbody> <tr> <td rowspan="2" class="change-gravatar"> <div class="vcard vcard_546015" id="a141528094_546015"> <img src="https://secure.gravatar.com/avatar/d0465b8931b44330f542efbeeae855da?d=mm&size=64" class="gravatar" width="32" height="32"> </div></td> <td class="change-author"> <div class="vcard vcard_546015" id="a141528094_546015"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=546015&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">BMO Automation</span></a> </div></td> <td rowspan="2" class="comment-actions"> <div role="group"><button type="button" class="change-spinner ghost iconic" id="as-a141528094_546015" aria-label="Collapse" aria-expanded="true" data-strings="{ "collapse_label": "Collapse", "expand_label": "Expanded" }"> <span class="icon" aria-hidden="true"></span> </button> </div></td> </tr> <tr id="ar-a141528094_546015"> <td><h3 class="change-name"><a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB#a141528094_546015">Updated</a></h3> • <div class="change-time"> <span class="rel-time" title="2022-10-11 14:30 PDT" data-time="1665523825">2 years ago</span> </div></td> </tr> </tbody> </table> </div> <div class="activity"> <div class="change"> Severity: normal → S3 </div> </div> </div> <div id="new-comment-notice"> You need to <a href="https://bugzilla-mozilla-org.translate.goog/show_bug.cgi?id=1454764&GoAheadAndLogIn=1&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB">log in</a> before you can comment on or make changes to this bug. </div> <div id="bottom-actions"> <div id="bottom-right-actions"><button type="button" id="top-btn" class="secondary" aria-label="Go to Page Top">Top ↑</button> </div> </div><dialog id="att-overlay" class="readonly" aria-labelledby="att-overlay-title" data-attachment-count="1"> <form method="dialog"> <div class="header"> <h2 id="att-overlay-title" class="title">Attachment</h2> <div class="spacer"></div><button type="button" class="secondary iconic" hidden disabled data-action="prev" title="Previous Attachment" aria-keyshortcuts="ArrowLeft"> <span class="icon" aria-hidden="true"></span> </button> <button type="button" class="secondary iconic" hidden disabled data-action="next" title="Next Attachment" aria-keyshortcuts="ArrowRight"> <span class="icon" aria-hidden="true"></span> </button> <button type="button" class="secondary" data-action="toggle-details">Hide Details</button> <button type="button" class="secondary iconic" data-action="close" aria-label="Close"> <span class="icon" aria-hidden="true"></span> </button> </div> <div class="body"> <div class="sub-column"> <div class="detail-pane"> <section class="cols"> <h3>General</h3> <div class="creator"> Creator: <div class="vcard vcard_313730"> <a class="email " href="https://bugzilla-mozilla-org.translate.goog/user_profile?user_id=313730&_x_tr_sl=pl&_x_tr_tl=iw&_x_tr_hl=en-GB"> <span class="fna">Bobby Holley (:bholley)</span></a> </div> </div> <div> Created: <span class="created-date rel-time"></span> </div> <div> Updated: <span class="updated-date rel-time"></span> </div> <div> Size: <span class="file-size"></span> </div> </section> <section class="cols"> <h3><label for="att-overlay-description">Description</label></h3> <div><input type="text" name="description" disabled id="att-overlay-description" class="flex"> </div> </section> <section class="cols"> <h3><label for="att-overlay-file-name">File Name</label></h3> <div><input type="text" name="filename" disabled id="att-overlay-file-name" class="flex"> </div> </section> <section class="cols"> <h3><label for="att-overlay-content-type">Content Type</label></h3> <div><input type="text" name="contenttypeentry" disabled id="att-overlay-content-type" class="flex"> </div> </section> </div> </div> <div class="main-column"> <div class="preview-pane"> <div class="preview "> </div> </div> </div> </div> <div class="footer"><button type="button" class="secondary" data-action="raw">Raw</button> <button type="button" class="secondary" disabled data-action="diff">Diff</button> <button type="button" class="secondary" disabled data-action="review" data-base="https://bugzilla.mozilla.org/page.cgi?id=splinter.html&ignore=">Splinter Review</button> <div class="spacer"></div> </div> </form> </dialog> <link rel="stylesheet" href="/static/v20241119.1/extensions/FlagTypeComment/web/styles/ftc.css"> <script nonce="ZJnq5mZCpVWc4MfYnlNI9v381uqy6YgAdV3j3bkpO622W3Jb" src="/static/v20241119.1/extensions/FlagTypeComment/web/js/ftc.js"></script> </div> </main> </div> <script>function gtElInit() {var lib = new google.translate.TranslateService();lib.translatePage('pl', 'iw', function () {});}</script> <script src="https://translate.google.com/translate_a/element.js?cb=gtElInit&hl=en-GB&client=wt" type="text/javascript"></script> </body> </html>

CINXE.COM

1454764 - (rustfmt) [meta] rustfmt mozilla-central