Add use strict to every file

brianc · brianc · commit 0f1f8626cf1e · 2017-06-18T16:48:14.000-05:00
diff --git a/lib/client.js b/lib/client.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/connection-parameters.js b/lib/connection-parameters.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/connection.js b/lib/connection.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/defaults.js b/lib/defaults.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/index.js b/lib/index.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/native/client.js b/lib/native/client.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/native/index.js b/lib/native/index.js
@@ -1 +1,2 @@
+"use strict";
 module.exports = require('./client')
diff --git a/lib/native/query.js b/lib/native/query.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/native/result.js b/lib/native/result.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/query.js b/lib/query.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/result.js b/lib/result.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/type-overrides.js b/lib/type-overrides.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/lib/utils.js b/lib/utils.js
@@ -1,3 +1,4 @@
+"use strict";
 /**
  * Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)
  * All rights reserved.
diff --git a/script/create-test-tables.js b/script/create-test-tables.js
@@ -1,3 +1,4 @@
+"use strict";
 var args = require(__dirname + '/../test/cli');
 var pg = require(__dirname + '/../lib');
 
diff --git a/script/dump-db-types.js b/script/dump-db-types.js
@@ -1,3 +1,4 @@
+"use strict";
 var pg = require(__dirname + '/../lib');
 var args = require(__dirname + '/../test/cli');
 
diff --git a/script/list-db-types.js b/script/list-db-types.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + "/../test/integration/test-helper");
 var pg = helper.pg;
 pg.connect(helper.config, assert.success(function(client) {
diff --git a/test/buffer-list.js b/test/buffer-list.js
@@ -1,4 +1,5 @@
-BufferList = function() {
+"use strict";
+global.BufferList = function() {
   this.buffers = [];
 };
 var p = BufferList.prototype;
diff --git a/test/cli.js b/test/cli.js
@@ -1,3 +1,4 @@
+"use strict";
 var ConnectionParameters = require(__dirname + '/../lib/connection-parameters');
 var config = new ConnectionParameters(process.argv[2]);
 
diff --git a/test/integration/client/api-tests.js b/test/integration/client/api-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + "/../test-helper");
 var pg = helper.pg;
 
diff --git a/test/integration/client/appname-tests.js b/test/integration/client/appname-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var Client = helper.Client;
 
diff --git a/test/integration/client/array-tests.js b/test/integration/client/array-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + "/test-helper");
 var pg = helper.pg;
 
diff --git a/test/integration/client/big-simple-query-tests.js b/test/integration/client/big-simple-query-tests.js
diff --git a/test/integration/client/configuration-tests.js b/test/integration/client/configuration-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var pg = helper.pg;
 
diff --git a/test/integration/client/custom-types-tests.js b/test/integration/client/custom-types-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 const helper = require('./test-helper');
 const Client = helper.pg.Client;
 const suite = new helper.Suite()
diff --git a/test/integration/client/empty-query-tests.js b/test/integration/client/empty-query-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 const suite = new helper.Suite()
 
diff --git a/test/integration/client/error-handling-tests.js b/test/integration/client/error-handling-tests.js
@@ -1,4 +1,5 @@
 "use strict";
+"use strict";
 
 var helper = require('./test-helper');
 var util = require('util');
diff --git a/test/integration/client/huge-numeric-tests.js b/test/integration/client/huge-numeric-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 const pool = new helper.pg.Pool()
 
diff --git a/test/integration/client/json-type-parsing-tests.js b/test/integration/client/json-type-parsing-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var assert = require('assert');
 
diff --git a/test/integration/client/network-partition-tests.js b/test/integration/client/network-partition-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var buffers = require('../../test-buffers')
 var helper = require('./test-helper')
 var suite = new helper.Suite()
diff --git a/test/integration/client/no-data-tests.js b/test/integration/client/no-data-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 const suite = new helper.Suite()
 
diff --git a/test/integration/client/no-row-result-tests.js b/test/integration/client/no-row-result-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./test-helper");
 var pg = helper.pg;
 const suite = new helper.Suite();
diff --git a/test/integration/client/notice-tests.js b/test/integration/client/notice-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 const suite = new helper.Suite()
 
diff --git a/test/integration/client/parse-int-8-tests.js b/test/integration/client/parse-int-8-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 
 var helper = require('../test-helper');
 var pg = helper.pg;
diff --git a/test/integration/client/prepared-statement-tests.js b/test/integration/client/prepared-statement-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var Query = helper.pg.Query;
 
diff --git a/test/integration/client/promise-api-tests.js b/test/integration/client/promise-api-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 'use strict';
 
 const helper = require('./test-helper')
diff --git a/test/integration/client/query-as-promise-tests.js b/test/integration/client/query-as-promise-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/../test-helper');
 var pg = helper.pg;
 
diff --git a/test/integration/client/query-column-names-tests.js b/test/integration/client/query-column-names-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/../test-helper');
 var pg = helper.pg;
 
diff --git a/test/integration/client/query-error-handling-prepared-statement-tests.js b/test/integration/client/query-error-handling-prepared-statement-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var Query = helper.pg.Query;
 var util = require('util');
diff --git a/test/integration/client/query-error-handling-tests.js b/test/integration/client/query-error-handling-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 var util = require('util');
 var Query = helper.pg.Query;
diff --git a/test/integration/client/quick-disconnect-tests.js b/test/integration/client/quick-disconnect-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 //test for issue #320
 //
 var helper = require('./test-helper');
diff --git a/test/integration/client/result-metadata-tests.js b/test/integration/client/result-metadata-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./test-helper");
 var pg = helper.pg;
 
diff --git a/test/integration/client/results-as-array-tests.js b/test/integration/client/results-as-array-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var util = require('util');
 var helper = require('./test-helper');
 
diff --git a/test/integration/client/row-description-on-results-tests.js b/test/integration/client/row-description-on-results-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 
 var Client = helper.Client;
diff --git a/test/integration/client/simple-query-tests.js b/test/integration/client/simple-query-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./test-helper");
 var Query = helper.pg.Query;
 
@@ -18,7 +19,7 @@ test("simple query interface", function() {
   query.once('row', function(row) {
     test('Can iterate through columns', function () {
       var columnCount = 0;
-      for (column in row) {
+      for (var column in row) {
         columnCount++;
       }
       if ('length' in row) {
diff --git a/test/integration/client/ssl-tests.js b/test/integration/client/ssl-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var pg = require(__dirname + '/../../../lib');
 var config = require(__dirname + '/test-helper').config;
 test('can connect with ssl', function() {
diff --git a/test/integration/client/test-helper.js b/test/integration/client/test-helper.js
@@ -1,3 +1,4 @@
-var helper = require(__dirname+'/../test-helper');
+"use strict";
+var helper = require('./../test-helper');
 
 module.exports = helper;
diff --git a/test/integration/client/timezone-tests.js b/test/integration/client/timezone-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./../test-helper');
 var exec = require('child_process').exec;
 
diff --git a/test/integration/client/transaction-tests.js b/test/integration/client/transaction-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 const suite = new helper.Suite()
 const pg = helper.pg
diff --git a/test/integration/client/type-coercion-tests.js b/test/integration/client/type-coercion-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/test-helper');
 var pg = helper.pg;
 var sink;
diff --git a/test/integration/client/type-parser-override-tests.js b/test/integration/client/type-parser-override-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 
 function testTypeParser(client, expectedResult, done) {
diff --git a/test/integration/connection-pool/connection-pool-size-tests.js b/test/integration/connection-pool/connection-pool-size-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./test-helper")
 
 helper.testPoolSize(1);
diff --git a/test/integration/connection-pool/error-tests.js b/test/integration/connection-pool/error-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./test-helper");
 const pg = helper.pg
 
diff --git a/test/integration/connection-pool/idle-timeout-tests.js b/test/integration/connection-pool/idle-timeout-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper');
 
 
diff --git a/test/integration/connection-pool/native-instance-tests.js b/test/integration/connection-pool/native-instance-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./../test-helper")
 var pg = helper.pg
 var native = helper.args.native
diff --git a/test/integration/connection-pool/test-helper.js b/test/integration/connection-pool/test-helper.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("./../test-helper");
 
 const suite = new helper.Suite()
diff --git a/test/integration/connection-pool/yield-support-tests.js b/test/integration/connection-pool/yield-support-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./test-helper')
 var co = require('co')
 
diff --git a/test/integration/connection/bound-command-tests.js b/test/integration/connection/bound-command-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/test-helper');
 //http://developer.postgresql.org/pgdocs/postgres/protocol-flow.html#PROTOCOL-FLOW-EXT-QUERY
 
diff --git a/test/integration/connection/copy-tests.js b/test/integration/connection/copy-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname+"/test-helper");
 var assert = require('assert');
 
diff --git a/test/integration/connection/notification-tests.js b/test/integration/connection/notification-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/test-helper');
 //http://www.postgresql.org/docs/8.3/static/libpq-notify.html
 test('recieves notification from same connection with no payload', function() {
diff --git a/test/integration/connection/query-tests.js b/test/integration/connection/query-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname+"/test-helper");
 var assert = require('assert');
 
diff --git a/test/integration/connection/test-helper.js b/test/integration/connection/test-helper.js
@@ -1,3 +1,4 @@
+"use strict";
 var net = require('net');
 var helper = require(__dirname+'/../test-helper');
 var Connection = require(__dirname + '/../../../lib/connection');
diff --git a/test/integration/domain-tests.js b/test/integration/domain-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var async = require('async')
 
 var helper = require('./test-helper')
diff --git a/test/integration/gh-issues/130-tests.js b/test/integration/gh-issues/130-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/../test-helper');
 var exec = require('child_process').exec;
 
diff --git a/test/integration/gh-issues/131-tests.js b/test/integration/gh-issues/131-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('../test-helper');
 var pg = helper.pg;
 
diff --git a/test/integration/gh-issues/199-tests.js b/test/integration/gh-issues/199-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('../test-helper');
 var client = helper.client();
 
diff --git a/test/integration/gh-issues/507-tests.js b/test/integration/gh-issues/507-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + "/../test-helper");
 var pg = helper.pg;
 
diff --git a/test/integration/gh-issues/600-tests.js b/test/integration/gh-issues/600-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var async = require('async');
 var helper = require('../test-helper');
 const suite = new helper.Suite()
diff --git a/test/integration/gh-issues/675-tests.js b/test/integration/gh-issues/675-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('../test-helper');
 var assert = require('assert');
 
diff --git a/test/integration/gh-issues/699-tests.js b/test/integration/gh-issues/699-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('../test-helper');
 var assert = require('assert');
 var copyFrom = require('pg-copy-streams').from;
diff --git a/test/integration/gh-issues/787-tests.js b/test/integration/gh-issues/787-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('../test-helper');
 const pool = new helper.pg.Pool()
 
diff --git a/test/integration/gh-issues/882-tests.js b/test/integration/gh-issues/882-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 //client should not hang on an empty query
 var helper = require('../test-helper');
 var client = helper.client();
diff --git a/test/integration/gh-issues/981-tests.js b/test/integration/gh-issues/981-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require('./../test-helper');
 
 //native bindings are only installed for native tests
diff --git a/test/integration/test-helper.js b/test/integration/test-helper.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + '/../test-helper');
 
 if(helper.args.native) {
diff --git a/test/native/callback-api-tests.js b/test/native/callback-api-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var domain = require('domain');
 var helper = require("./../test-helper");
 var Client = require("./../../lib/native");
diff --git a/test/native/error-tests.js b/test/native/error-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require(__dirname + "/../test-helper");
 var Client = require(__dirname + "/../../lib/native");
 
diff --git a/test/native/evented-api-tests.js b/test/native/evented-api-tests.js
@@ -1,3 +1,4 @@
+"use strict";
 var helper = require("../test-helper");
 var Client = require("../../lib/native");
 var Query = Client.Query;
diff --git a/test/native/missing-native.js b/test/native/missing-native.js
@@ -1,3 +1,4 @@
+"use strict";
 //this test assumes it has been run from the Makefile
 //and that node_modules/pg-native has been deleted
 
diff --git a/test/native/native-vs-js-error-tests.js b/test/native/native-vs-js-error-tests.js
diff --git a/test/native/stress-tests.js b/test/native/stress-tests.js
diff --git a/test/suite.js b/test/suite.js
diff --git a/test/test-buffers.js b/test/test-buffers.js
diff --git a/test/test-helper.js b/test/test-helper.js
diff --git a/test/unit/client/cleartext-password-tests.js b/test/unit/client/cleartext-password-tests.js
diff --git a/test/unit/client/configuration-tests.js b/test/unit/client/configuration-tests.js
diff --git a/test/unit/client/early-disconnect-tests.js b/test/unit/client/early-disconnect-tests.js
diff --git a/test/unit/client/escape-tests.js b/test/unit/client/escape-tests.js
diff --git a/test/unit/client/md5-password-tests.js b/test/unit/client/md5-password-tests.js
diff --git a/test/unit/client/notification-tests.js b/test/unit/client/notification-tests.js
diff --git a/test/unit/client/prepared-statement-tests.js b/test/unit/client/prepared-statement-tests.js
diff --git a/test/unit/client/query-queue-tests.js b/test/unit/client/query-queue-tests.js
diff --git a/test/unit/client/result-metadata-tests.js b/test/unit/client/result-metadata-tests.js
diff --git a/test/unit/client/simple-query-tests.js b/test/unit/client/simple-query-tests.js
diff --git a/test/unit/client/stream-and-query-error-interaction-tests.js b/test/unit/client/stream-and-query-error-interaction-tests.js
diff --git a/test/unit/client/test-helper.js b/test/unit/client/test-helper.js
diff --git a/test/unit/client/throw-in-type-parser-tests.js b/test/unit/client/throw-in-type-parser-tests.js
diff --git a/test/unit/connection-parameters/creation-tests.js b/test/unit/connection-parameters/creation-tests.js
diff --git a/test/unit/connection-parameters/environment-variable-tests.js b/test/unit/connection-parameters/environment-variable-tests.js
diff --git a/test/unit/connection/error-tests.js b/test/unit/connection/error-tests.js
diff --git a/test/unit/connection/inbound-parser-tests.js b/test/unit/connection/inbound-parser-tests.js
diff --git a/test/unit/connection/outbound-sending-tests.js b/test/unit/connection/outbound-sending-tests.js
diff --git a/test/unit/connection/startup-tests.js b/test/unit/connection/startup-tests.js
diff --git a/test/unit/connection/test-helper.js b/test/unit/connection/test-helper.js
diff --git a/test/unit/test-helper.js b/test/unit/test-helper.js
diff --git a/test/unit/utils-tests.js b/test/unit/utils-tests.js

Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,4 @@`
	`1`	`+"use strict";`
`1`	`2`	`/**`
`2`	`3`	`* Copyright (c) 2010-2017 Brian Carlson (brian.m.carlson@gmail.com)`
`3`	`4`	`* All rights reserved.`
Original file line number	Diff line number	Diff line change
`@@ -1 +1,2 @@`
	`1`	`+"use strict";`
`1`	`2`	`module.exports = require('./client')`