Bug 1419128 - Don't use mutable default argument value; r?build draft
authorGregory Szorc <gps@mozilla.com>
Mon, 20 Nov 2017 14:14:56 -0800
changeset 700820 183b17c0c55da8f2f9ba3fc22e1c886e34c2a6e3
parent 700819 2ca87964fef1e8ed4d8719142b6a74c45e4beda8
child 700821 9eb58b3f546a32978a66cf3db7a92fa23f68f521
push id89986
push userbmo:gps@mozilla.com
push dateMon, 20 Nov 2017 23:17:58 +0000
reviewersbuild
bugs1419128
milestone59.0a1
Bug 1419128 - Don't use mutable default argument value; r?build We always passed this argument in so this change doesn't fix any bugs. I just didn't like the code smell. MozReview-Commit-ID: IXzj1gRbkJ6
build/compare-mozconfig/compare-mozconfigs.py
--- a/build/compare-mozconfig/compare-mozconfigs.py
+++ b/build/compare-mozconfig/compare-mozconfigs.py
@@ -37,17 +37,17 @@ def make_hg_url(hgHost, repoPath, protoc
                          filename]])
 
 def readConfig(configfile):
     c = {}
     execfile(configfile, c)
     return c['whitelist']
 
 def verify_mozconfigs(mozconfig_pair, nightly_mozconfig_pair, platform,
-                      mozconfigWhitelist={}):
+                      mozconfigWhitelist):
     """Compares mozconfig to nightly_mozconfig and compare to an optional
     whitelist of known differences. mozconfig_pair and nightly_mozconfig_pair
     are pairs containing the mozconfig's identifier and the list of lines in
     the mozconfig."""
 
     # unpack the pairs to get the names, the names are just for
     # identifying the mozconfigs when logging the error messages
     mozconfig_name, mozconfig_lines = mozconfig_pair