Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Fixes #6309 and several related issues #6318

Closed
wants to merge 2 commits into from

Conversation

davepacheco
Copy link

This change makes several improvements to the ustack helper and MDB support:

  • ustack helper and MDB: add support for two-byte strings
    (necessary to print many filenames in stacktraces in 0.10 and later).
  • ustack helper: fix position numbers, which were off by a factor of two
  • ustack helper: fix frames with undefined Scripts (e.g., "RegExp")
  • ustack helper: add stub frames
  • MDB: add support for sliced strings
  • MDB: sync up with changes from the illumos version of the module

@Nodejs-Jenkins
Copy link

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

Commit davepacheco/node@0f10e07 has the following error(s):

  • Commit message must indicate the subsystem this commit changes
  • Commit message line too long: 2

The following commiters were not found in the CLA:

  • Dave Pacheco

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

Dave Pacheco added 2 commits October 7, 2013 13:17
This change makes several improvements to the ustack helper and MDB
support:

- ustack helper and MDB: add support for two-byte strings
  (necessary to print many filenames in stacktraces in 0.10 and later).
- ustack helper: fix position numbers, which were off by a factor of two
- ustack helper: fix frames with undefined Scripts (e.g., "RegExp")
- ustack helper: add stub frames
- MDB: add support for sliced strings
- MDB: sync up with changes from the illumos version of the module
@bnoordhuis
Copy link
Member

@tjfontaine I suppose you should review this PR and #6319?

@tjfontaine tjfontaine closed this in 5921158 Oct 8, 2013
@tjfontaine
Copy link

Thanks landed in 2b9e3fb with a slightly reworded commit message

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants