[Pkg-mozext-commits] [requestpolicy] 171/257: Merge branch 'introduce-jshint' into dev-1.0

David Prévot taffit at moszumanska.debian.org
Thu Jan 28 03:20:09 UTC 2016


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository requestpolicy.

commit c8f1da1f28fe4e9c96cbfa9c049f06e1cc349033
Merge: 52d7665 d998841
Author: Martin Kimmerle <dev at 256k.de>
Date:   Tue Dec 1 15:26:23 2015 +0100

    Merge branch 'introduce-jshint' into dev-1.0
    
    Start to use JSHint. Many errors and warnings have been fixed!
    
    Some warnings haven't been fixed though:
    
    * The body of a for in should be wrapped in an if statement to
      filter unwanted properties from the prototype. (W089)
    * This function's cyclomatic complexity is too high. (W074)
    * Blocks are nested too deeply. (W073)
    
    Also certain warnings in `subscription.jsm` haven't been fixed.

 .gitignore                                         |   1 +
 .jshintrc                                          |  15 +
 Makefile                                           |   9 +
 src/bootstrap.js                                   |  34 +-
 src/content/lib/default-preferences.js             |   1 +
 src/content/lib/environment.jsm                    | 881 +++++++++++----------
 src/content/lib/environment.process.js             |  27 +-
 .../lib/framescript-to-overlay-communication.jsm   |  46 +-
 src/content/lib/gui-location.jsm                   |  72 +-
 src/content/lib/http-response.jsm                  | 299 ++++---
 src/content/lib/logger.jsm                         |  98 ++-
 src/content/lib/manager-for-event-listeners.jsm    |  24 +-
 src/content/lib/manager-for-message-listeners.jsm  |  26 +-
 src/content/lib/observer-manager.jsm               |  35 +-
 src/content/lib/old-rules.jsm                      |  30 +-
 src/content/lib/policy-manager.alias-functions.js  |  18 +-
 src/content/lib/policy-manager.jsm                 | 194 +++--
 src/content/lib/prefs.jsm                          |  50 +-
 src/content/lib/request-processor.compat.js        |  34 +-
 src/content/lib/request-processor.jsm              | 266 ++++---
 src/content/lib/request-processor.redirects.js     | 158 ++--
 src/content/lib/request-result.jsm                 |  59 +-
 src/content/lib/request-set.jsm                    |  45 +-
 src/content/lib/request.jsm                        |  74 +-
 src/content/lib/ruleset-storage.jsm                |  49 +-
 src/content/lib/ruleset.jsm                        | 348 ++++----
 src/content/lib/script-loader.jsm                  |  59 +-
 src/content/lib/subscription.jsm                   | 170 ++--
 src/content/lib/utils.jsm                          |  35 +-
 src/content/lib/utils/constants.jsm                |  10 +-
 src/content/lib/utils/dom.jsm                      |  47 +-
 src/content/lib/utils/domains.jsm                  | 150 ++--
 src/content/lib/utils/files.jsm                    |  37 +-
 src/content/lib/utils/info.jsm                     |  39 +-
 src/content/lib/utils/observers.jsm                |  19 +-
 src/content/lib/utils/strings.jsm                  |  25 +-
 src/content/lib/utils/windows.jsm                  |  36 +-
 src/content/lib/utils/xul.jsm                      |  40 +-
 src/content/main/about-uri.jsm                     |  84 +-
 src/content/main/content-policy.jsm                |  66 +-
 src/content/main/default-pref-handler.js           |  31 +-
 src/content/main/pref-manager.jsm                  |  41 +-
 src/content/main/requestpolicy-service.jsm         | 131 +--
 src/content/main/window-manager-toolbarbutton.js   |  28 +-
 src/content/main/window-manager.jsm                |  58 +-
 src/content/main/window-manager.listener.js        |  47 +-
 src/content/settings/advancedprefs.html            |   2 +-
 src/content/settings/advancedprefs.js              | 266 ++++---
 src/content/settings/basicprefs.html               |   2 +-
 src/content/settings/basicprefs.js                 | 168 ++--
 src/content/settings/common.js                     | 282 +++----
 src/content/settings/defaultpolicy.html            |   2 +-
 src/content/settings/defaultpolicy.js              | 168 ++--
 src/content/settings/oldrules.html                 |   2 +-
 src/content/settings/oldrules.js                   | 202 ++---
 src/content/settings/setup.html                    |   2 +-
 src/content/settings/setup.js                      | 331 ++++----
 src/content/settings/subscriptions.html            |   2 +-
 src/content/settings/subscriptions.js              | 281 ++++---
 src/content/settings/yourpolicy.html               |   2 +-
 src/content/settings/yourpolicy.js                 | 416 +++++-----
 src/content/ui/classicmenu.js                      |  30 +-
 src/content/ui/frame.blocked-content.js            |  57 +-
 src/content/ui/frame.dom-content-loaded.js         |  26 +-
 src/content/ui/frame.js                            |  80 +-
 src/content/ui/menu.js                             | 456 ++++++-----
 src/content/ui/overlay.js                          |  97 +--
 src/content/ui/request-log.filtering.js            |  19 +-
 src/content/ui/request-log.interface.js            |  29 +-
 src/content/ui/request-log.js                      |  23 +-
 src/content/ui/request-log.tree-view.js            |  29 +-
 src/content/ui/xul-trees.js                        |   4 +-
 src/install.rdf                                    |   4 +-
 .../dev-helper/content/console-observer.jsm        |  26 +-
 .../content/restart-detection-helper.jsm           |   2 +-
 tests/helper-addons/dev-helper/content/rpc-uri.jsm | 101 +--
 76 files changed, 3797 insertions(+), 3360 deletions(-)

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/requestpolicy.git



More information about the Pkg-mozext-commits mailing list