Refactor: Only use jshint validthis in functions that need it

changelog.md

@@ -19,3 +19,4 @@
 * Fix: Do not clone ZipFile or Entry if not required
 * Fix: Typos in error messages
 * Do not copy event emitter properties to `ZipFile` instances
+* Refactor: Only use jshint `validthis` in functions that need it

lib/promisify.js

@@ -4,7 +4,6 @@
  * ------------------*/
 
 'use strict';
-// jshint -W040
 
 // Exports
 module.exports = (yauzl, Promise) => {
@@ -175,6 +174,7 @@ function readEntryDone(zipFile, event, value, resolver) {
  * Events are passed to read or close handler to resolve/reject that promise.
  */
 function emit(event, value) {
+	// jshint validthis:true
 	// Get resolver and destroy
 	const resolver = this._resolver;
 	this._resolver = undefined;
@@ -206,6 +206,7 @@ function rejectWithStoredError(zipFile, reject) {
  * `options.max` can be set to `0` for no limit
  */
 function readEntries(numEntries) {
+	// jshint validthis:true
 	const entries = [];
 	return this.walkEntries(entry => {
 		entries.push(entry);
@@ -266,6 +267,7 @@ function promisifyOpenReadStream(ZipFile, Promise) {
  * Entry `openReadStream` method
  */
 function entryOpenReadStream(options) {
+	// jshint validthis:true
 	return this.zipFile.openReadStream(this, options);
 }