Correct the coding style according to gjslint in font.js
Also refactor the lines that are over 80 characters long.
This commit is contained in:
parent
ec9b3d67af
commit
b2037841e5
35
fonts.js
35
fonts.js
@ -536,7 +536,8 @@ var Font = (function Font() {
|
||||
// checksum
|
||||
var checksum = 0, n = data.length;
|
||||
for (var i = 0; i < n; i += 4)
|
||||
checksum = (checksum + int32([data[i], data[i + 1], data[i + 2], data[i + 3]])) | 0;
|
||||
checksum = (checksum + int32([data[i], data[i + 1], data[i + 2],
|
||||
data[i + 3]])) | 0;
|
||||
|
||||
var tableEntry = (tag + string32(checksum) +
|
||||
string32(offset) + string32(length));
|
||||
@ -803,7 +804,8 @@ var Font = (function Font() {
|
||||
file.pos = previousPosition;
|
||||
|
||||
if (tag == 'head')
|
||||
data[8] = data[9] = data[10] = data[11] = 0; // clearing checksum adjustment
|
||||
// clearing checksum adjustment
|
||||
data[8] = data[9] = data[10] = data[11] = 0;
|
||||
|
||||
return {
|
||||
tag: tag,
|
||||
@ -1267,10 +1269,9 @@ var Font = (function Font() {
|
||||
charsCache = this.charsCache = Object.create(null);
|
||||
|
||||
if (this.compositeFont) {
|
||||
// composite fonts have multi-byte strings
|
||||
// convert the string from single-byte to multi-byte
|
||||
// XXX assuming CIDFonts are two-byte - later need to extract the correct byte encoding
|
||||
// according to the PDF spec
|
||||
// composite fonts have multi-byte strings convert the string from
|
||||
// single-byte to multi-byte XXX assuming CIDFonts are two-byte - later
|
||||
// need to extract the correct byte encoding according to the PDF spec
|
||||
str = '';
|
||||
var multiByteStr = '';
|
||||
var length = chars.length;
|
||||
@ -1558,8 +1559,8 @@ var Type1Parser = function() {
|
||||
};
|
||||
|
||||
function readNumber(str, index) {
|
||||
while (str[index++] == ' ')
|
||||
;
|
||||
while (str[index++] == ' ');
|
||||
|
||||
var start = index;
|
||||
|
||||
var count = 0;
|
||||
@ -1628,18 +1629,21 @@ var Type1Parser = function() {
|
||||
case '/FamilyOtherBlues':
|
||||
case '/StemSnapH':
|
||||
case '/StemSnapV':
|
||||
program.properties.private[token.substring(1)] = readNumberArray(eexecStr, i + 2);
|
||||
program.properties.private[token.substring(1)] =
|
||||
readNumberArray(eexecStr, i + 2);
|
||||
break;
|
||||
case '/StdHW':
|
||||
case '/StdVW':
|
||||
program.properties.private[token.substring(1)] = readNumberArray(eexecStr, i + 2)[0];
|
||||
program.properties.private[token.substring(1)] =
|
||||
readNumberArray(eexecStr, i + 2)[0];
|
||||
break;
|
||||
case '/BlueShift':
|
||||
case '/BlueFuzz':
|
||||
case '/BlueScale':
|
||||
case '/LanguageGroup':
|
||||
case '/ExpansionFactor':
|
||||
program.properties.private[token.substring(1)] = readNumber(eexecStr, i + 1);
|
||||
program.properties.private[token.substring(1)] =
|
||||
readNumber(eexecStr, i + 1);
|
||||
break;
|
||||
}
|
||||
} else if (c == '/') {
|
||||
@ -1810,8 +1814,10 @@ CFF.prototype = {
|
||||
// Add another offset after this one because we need a new offset
|
||||
var relativeOffset = 1;
|
||||
for (var i = 0; i < count + 1; i++) {
|
||||
data += String.fromCharCode((relativeOffset >>> 24) & 0xFF, (relativeOffset >> 16) & 0xFF,
|
||||
(relativeOffset >> 8) & 0xFF, relativeOffset & 0xFF);
|
||||
data += String.fromCharCode((relativeOffset >>> 24) & 0xFF,
|
||||
(relativeOffset >> 16) & 0xFF,
|
||||
(relativeOffset >> 8) & 0xFF,
|
||||
relativeOffset & 0xFF);
|
||||
|
||||
if (objects[i])
|
||||
relativeOffset += objects[i].length;
|
||||
@ -1819,7 +1825,8 @@ CFF.prototype = {
|
||||
|
||||
for (var i = 0; i < count; i++) {
|
||||
for (var j = 0; j < objects[i].length; j++)
|
||||
data += isByte ? String.fromCharCode(objects[i][j] & 0xFF) : objects[i][j];
|
||||
data += isByte ? String.fromCharCode(objects[i][j] & 0xFF) :
|
||||
objects[i][j];
|
||||
}
|
||||
return data;
|
||||
},
|
||||
|
Loading…
Reference in New Issue
Block a user