Bug 1436655 - Tweak the comment at the top of init/all.js. r=glandium draft
authorNicholas Nethercote <nnethercote@mozilla.com>
Thu, 15 Mar 2018 14:59:29 +1100
changeset 771488 b4e03ecf060c262d492ce73c7b7c3ff48447c251
parent 771487 c3392be67821a0597fa72ad689219a13109cce15
child 771489 73be150e8abf58cc8436dc30f3be08a7a6373c45
push id103687
push usernnethercote@mozilla.com
push dateFri, 23 Mar 2018 04:16:02 +0000
reviewersglandium
bugs1436655
milestone61.0a1
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
+ * for example browser/defaults/preferences/firefox.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