----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17064/#review32291 -----------------------------------------------------------
src/cli.js <https://reviews.apache.org/r/17064/#comment61030> Nit: s/fall/call/ src/cli.js <https://reviews.apache.org/r/17064/#comment61031> Trivial Nit: whitespace (leading space inside if condition, no space before previous line's brace) - Michal Mocny On Jan. 20, 2014, 9:23 p.m., Mark Koudritsky wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17064/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2014, 9:23 p.m.) > > > Review request for cordova. > > > Bugs: CB-5782 > https://issues.apache.org/jira/browse/CB-5782 > > > Repository: cordova-cli > > > Description > ------- > > The on('uncaughtException') handler was removed recently in cordova-cli > because > it was hiding the stack traces of some legit exceptions, but this results in > scary traces for simple errors like running outside a corodva project. > > This change reintroduces the on('uncaughtException') handler and adds a > special > CordovaError class for such simple errors. For exceptions of CordovaError > class > only the message will be printed, for all other errors - the full stack trace. > > Another pass over the code will be needed to find and convert the Errors to > CordovaErrors where appropriate. Will be done in a separate change. > > > Diffs > ----- > > src/CordovaError.js PRE-CREATION > src/cli.js d22ba56 > src/util.js 278172a > > Diff: https://reviews.apache.org/r/17064/diff/ > > > Testing > ------- > > npm test > cordova plugin ls (inside and outside a a proj) > cordova bad_command > > > Thanks, > > Mark Koudritsky > >
