This repository has been archived by the owner. It is now read-only.

debugger: refactor, no more res.success checks #1779

wants to merge 2 commits into


None yet
2 participants

indutny commented Sep 27, 2011

I'm sick of stupid, repeating checks like res.success || res.message.
Time to make changes - refactored all code, so error handling is working automatically, like in any normal node.js script.

ry commented Sep 27, 2011

breaks test/simple/test-debugger-client.js


indutny commented Sep 28, 2011

@Ryah fixed, thanks

@ry ry closed this in 1b8b097 Sep 28, 2011

sam-github pushed a commit to sam-github/node-v0.x-archive that referenced this pull request Jul 18, 2015

2015-05-24 io.js v2.1.0 Release
PR-URL: nodejs/node#1777

Notable Changes:

* crypto: Diffie-Hellman key exchange (DHE) parameters ('dhparams') must now be
  1024 bits or longer or an error will be thrown. A warning will also be printed
  to the console if you supply less than 2048 bits. See for
  further context on this security concern. (Shigeki Ohtsu) #1739.
* node: A new --trace-sync-io command line flag will print a warning and a stack
  trace whenever a synchronous API is used. This can be used to track down
  synchronous calls that may be slowing down an application.
  (Trevor Norris) #1707.
* node: To allow for chaining of methods, the setTimeout(), setKeepAlive(),
  setNoDelay(), ref() and unref() methods used in 'net', 'dgram', 'http',
  'https' and 'tls' now return the current instance instead of undefined
  (Roman Reiss & Evan Lucas) #1699 #1768 #1779.
* npm: Upgraded to v2.10.1, release notes can be found in and
* util: A significant speed-up (in the order of 35%) for the common-case of a
  single string argument to util.format(), used by console.log()
  (Сковорода Никита Андреевич) #1749.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.