Bug 1436655 - Tweak the comment at the top of init/all.js. r=glandium
authorNicholas Nethercote <nnethercote@mozilla.com>
Thu, 15 Mar 2018 14:59:29 +1100
changeset 410122 8509ebf8fbb3af7e580c85aa73d5bb4a9b78c61a
parent 410121 c34cc8156a71b1feab6cf4351b80ac60cd12cc78
child 410123 08af6778dd6a669e896d61ef7599a75d7024d5fd
push id33718
push userbtara@mozilla.com
push dateTue, 27 Mar 2018 09:16:52 +0000
treeherdermozilla-central@97cdd8febc40 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersglandium
bugs1436655
milestone61.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1436655 - Tweak the comment at the top of init/all.js. r=glandium - The first two SYNTAX HINTS are wrong, and citing the syntax is more useful than specifying a single example of what isn't allowed. - The sentence about #ifdefs is wrong. (#ifdefs appear all throughout, and prefs are only specified once.) - I chose a better example file. MozReview-Commit-ID: JyYFyutqrFD
modules/libpref/init/all.js
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -1,27 +1,19 @@
 /* -*- indent-tabs-mode: nil; js-indent-level: 2 -*- */
 /* This Source Code Form is subject to the terms of the Mozilla Public
  * License, v. 2.0. If a copy of the MPL was not distributed with this
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 /* The prefs in this file are shipped with the GRE and should apply to all
  * embedding situations. Application-specific preferences belong somewhere else,
- * for example xpfe/bootstrap/browser-prefs.js
+ * such as browser/app/profile/firefox.js or mobile/android/app/mobile.js.
  *
- * Platform-specific #ifdefs at the end of this file override the generic
- * entries at the top.
- */
-
-/*
- * SYNTAX HINTS:
- *
- *  - Dashes are delimiters; use underscores instead.
- *  - The first character after a period must be alphabetic.
- *  - Computed values (e.g. 50 * 1024) don't work.
+ * For the syntax used by this file, consult the comments at the top of
+ * modules/libpref/parser/src/lib.rs.
  */
 
 pref("preferences.allow.omt-write", true);
 
 pref("keyword.enabled", false);
 pref("general.useragent.compatMode.firefox", false);
 
 // This pref exists only for testing purposes. In order to disable all