fix global leaks and copy-paste errors
authorWade Simmons <wade@wades.im>
Tue, 1 Jul 2014 19:18:30 +0000 (12:18 -0700)
committerRoger Meier <roger@apache.org>
Fri, 4 Jul 2014 19:28:44 +0000 (21:28 +0200)
commit0dd35a6c4b6e211c0fec867d279cee5172280f4b
tree5b8c417db6e665cce8cb6c954c52b52a3e2e8ece
parentf16daad8d32cf6d0b04e2ce71d890fa080cb68a1
fix global leaks and copy-paste errors

- TCompactProtocol.prototype.writeBool not referencing `booleanField_`
  on `this` correctly. Also using `NULL` instead of `null`.

- TCompactProtocol.prototype.writeVarint64 had a typo for TProtocolException

- TCompactProtocol.prototype.readMapBegin had a typo between `kvtype` and
 `kvType`

- createMultiplexServer leaked a global var `processStatus`

- TFramedTransport had a line that was accidently copy pasted, leading
  to a global leak of `frameLeft`. (I created the patch the introduced
  this copy paste error in Thrift-1353, so I can confirm it was
  a mistake).

- createWebServer tried to check a non-existent var `route` and leaked
  a global `result`

Signed-off-by: Roger Meier <roger@apache.org>
lib/nodejs/lib/thrift/protocol.js
lib/nodejs/lib/thrift/server.js
lib/nodejs/lib/thrift/transport.js
lib/nodejs/lib/thrift/web_server.js