Bug 735877 - Update syntax hints in pref files to reflect the fact that you can't use computed values (e.g. 50 * 1024). r=mossop
authorJustin Lebar <justin.lebar@gmail.com>
Wed, 14 Mar 2012 18:49:17 -0400
changeset 89416 0eb8564567df73622a0b90d6d5468ba6a69a5810
parent 89415 f767da72d616cf991b66a8c0df3e7403574d5598
child 89417 bd09e7a515b720aaf567cb08b66dc1c8cfc9cc6d
push id1
push userroot
push dateMon, 20 Oct 2014 17:29:22 +0000
reviewersmossop
bugs735877
milestone14.0a1
Bug 735877 - Update syntax hints in pref files to reflect the fact that you can't use computed values (e.g. 50 * 1024). r=mossop
browser/app/profile/firefox.js
modules/libpref/src/init/all.js
--- a/browser/app/profile/firefox.js
+++ b/browser/app/profile/firefox.js
@@ -34,18 +34,23 @@
 # the terms of any one of the MPL, the GPL or the LGPL.
 #
 # ***** END LICENSE BLOCK *****
 
 // XXX Toolkit-specific preferences should be moved into toolkit.js
 
 #filter substitution
 
-# SYNTAX HINTS:  dashes are delimiters.  Use underscores instead.
-#  The first character after a period must be alphabetic.
+#
+# 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.
+#
 
 #ifdef XP_UNIX
 #ifndef XP_MACOSX
 #define UNIX_BUT_NOT_MAC
 #endif
 #endif
 
 pref("browser.chromeURL","chrome://browser/content/");
--- a/modules/libpref/src/init/all.js
+++ b/modules/libpref/src/init/all.js
@@ -39,18 +39,23 @@
 /* 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
  *
  * 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.
+/*
+ * 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.
+ */
 
 pref("keyword.URL", "http://www.google.com/search?ie=UTF-8&oe=utf-8&q=");
 pref("keyword.enabled", false);
 pref("general.useragent.locale", "chrome://global/locale/intl.properties");
 pref("general.useragent.compatMode.firefox", false);
 
 pref("general.config.obscure_value", 13); // for MCD .cfg files