Enforces quotmark=single for jshint

This commit is contained in:
Yury Delendik 2013-02-02 17:00:13 -06:00
parent 5cf0d8fa80
commit d13aa38c9c
5 changed files with 31 additions and 13 deletions

View File

@ -7,6 +7,7 @@
// Enforcing // Enforcing
"maxlen": 80, "maxlen": 80,
"quotmark": "single",
// Relaxing // Relaxing
"boss": true, "boss": true,

View File

@ -734,7 +734,7 @@ var PartialEvaluator = (function PartialEvaluatorClosure() {
case 'Tj': case 'Tj':
chunk += fontCharsToUnicode(args[0], font); chunk += fontCharsToUnicode(args[0], font);
break; break;
case "'": case '\'':
// For search, adding a extra white space for line breaks would be // For search, adding a extra white space for line breaks would be
// better here, but that causes too much spaces in the // better here, but that causes too much spaces in the
// text-selection divs. // text-selection divs.

View File

@ -4312,7 +4312,7 @@ var Font = (function FontClosure() {
// Add the font-face rule to the document // Add the font-face rule to the document
var url = ('url(data:' + this.mimetype + ';base64,' + var url = ('url(data:' + this.mimetype + ';base64,' +
window.btoa(data) + ');'); window.btoa(data) + ');');
var rule = "@font-face { font-family:'" + fontName + "';src:" + url + '}'; var rule = '@font-face { font-family:"' + fontName + '";src:' + url + '}';
var styleElement = document.getElementById('PDFJS_FONT_STYLE_TAG'); var styleElement = document.getElementById('PDFJS_FONT_STYLE_TAG');
if (!styleElement) { if (!styleElement) {

View File

@ -481,13 +481,13 @@ var Lexer = (function LexerClosure() {
if (isFirstHex) { if (isFirstHex) {
firstDigit = toHexDigit(ch); firstDigit = toHexDigit(ch);
if (firstDigit === -1) { if (firstDigit === -1) {
warn("Ignoring invalid character '" + ch + "' in hex string"); warn('Ignoring invalid character "' + ch + '" in hex string');
continue; continue;
} }
} else { } else {
secondDigit = toHexDigit(ch); secondDigit = toHexDigit(ch);
if (secondDigit === -1) { if (secondDigit === -1) {
warn("Ignoring invalid character '" + ch + "' in hex string"); warn('Ignoring invalid character "' + ch + '" in hex string');
continue; continue;
} }
str += String.fromCharCode((firstDigit << 4) | secondDigit); str += String.fromCharCode((firstDigit << 4) | secondDigit);

View File

@ -1,28 +1,45 @@
"use strict"; /* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
/* vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */
/* Copyright 2012 Mozilla Foundation
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
'use strict';
module.exports = { module.exports = {
reporter: function (res) { reporter: function reporter(res) {
var len = 0; var len = 0;
var str = ""; var str = '';
res.forEach(function (r) { res.forEach(function(r) {
var file = r.file; var file = r.file;
var err = r.error; var err = r.error;
switch(err.code) { switch (err.code) {
case 'W004': // variable is already defined case 'W004': // variable is already defined
case 'W018': // confusing use of ! case 'W018': // confusing use of !
break; break;
default: default:
len++; len++;
str += file + ": line " + err.line + ", col " + str += file + ': line ' + err.line + ', col ' +
err.character + ", " + err.reason + "\n"; err.character + ', ' + err.reason + '\n';
} }
}); });
if (str) { if (str) {
process.stdout.write(str + "\n" + len + " error" + process.stdout.write(str + '\n' + len + ' error' +
((len === 1) ? "" : "s") + "\n"); ((len === 1) ? '' : 's') + '\n');
process.exit(2); process.exit(2);
} else { } else {
process.exit(0); process.exit(0);