clean up prototype shorthand

For some reason a few years ago I thought it would be neat to use a shorthand version of prototype to save myself some keystrokes.  That was a cosmetic mistake.  It also breaks ctags.

Also, normalized some whitespace.

Author: brianc

binding.gyp

@@ -6,18 +6,6 @@
         'src/binding.cc'
       ],
       'conditions' : [
-        ['OS=="mac"', {
-          'include_dirs': ['<!@(pg_config --includedir)'],
-          'libraries' : ['-lpq -L<!@(pg_config --libdir)']
-        }],
-        ['OS=="linux"', {
-          'include_dirs': ['<!@(pg_config --includedir)'],
-          'libraries' : ['-lpq -L<!@(pg_config --libdir)']
-        }],
-        ['OS=="solaris"', {
-          'include_dirs': ['<!@(pg_config --includedir)'],
-          'libraries' : ['-lpq -L<!@(pg_config --libdir)']
-        }],
         ['OS=="win"', {
           'include_dirs': ['<!@(pg_config --includedir)'],
           'libraries' : ['libpq.lib'],
@@ -28,6 +16,9 @@
               ]
             },
           }
+        }, { # OS!="win"
+          'include_dirs': ['<!@(pg_config --includedir)'],
+          'libraries' : ['-lpq -L<!@(pg_config --libdir)']
         }]
       ]
     }

lib/client.js

@@ -36,9 +36,7 @@ var Client = function(config) {
 
 util.inherits(Client, EventEmitter);
 
-var p = Client.prototype;
-
-p.connect = function(callback) {
+Client.prototype.connect = function(callback) {
   var self = this;
   var con = this.connection;
   if(this.host && this.host.indexOf('/') === 0) {
@@ -170,7 +168,7 @@ p.connect = function(callback) {
 
 };
 
-p.cancel = function(client, query) {
+Client.prototype.cancel = function(client, query) {
   if (client.activeQuery == query) {
     var con = this.connection;
 
@@ -190,7 +188,7 @@ p.cancel = function(client, query) {
   }
 };
 
-p._pulseQueryQueue = function() {
+Client.prototype._pulseQueryQueue = function() {
   if(this.readyForQuery===true) {
     this.activeQuery = this.queryQueue.shift();
     if(this.activeQuery) {
@@ -204,7 +202,8 @@ p._pulseQueryQueue = function() {
     }
   }
 };
-p._copy = function (text, stream) {
+
+Client.prototype._copy = function (text, stream) {
   var config = {},
     query;
   config.text = text;
@@ -222,13 +221,16 @@ p._copy = function (text, stream) {
   return config.stream;
 
 };
-p.copyFrom = function (text) {
+
+Client.prototype.copyFrom = function (text) {
   return this._copy(text, new CopyFromStream());
 };
-p.copyTo = function (text) {
+
+Client.prototype.copyTo = function (text) {
   return this._copy(text, new CopyToStream());
 };
-p.query = function(config, values, callback) {
+
+Client.prototype.query = function(config, values, callback) {
   //can take in strings, config object or query object
   var query = (config instanceof Query) ? config :
      new Query(config, values, callback);
@@ -243,19 +245,19 @@ p.query = function(config, values, callback) {
 
 //prevents client from otherwise emitting 'drain' event until 'resumeDrain' is
 //called
-p.pauseDrain = function() {
+Client.prototype.pauseDrain = function() {
   this._drainPaused = 1;
 };
 
 //resume raising 'drain' event
-p.resumeDrain = function() {
+Client.prototype.resumeDrain = function() {
   if(this._drainPaused > 1) {
     this.emit('drain');
   }
   this._drainPaused = 0;
 };
 
-p.end = function() {
+Client.prototype.end = function() {
   this.connection.end();
 };