Skip to content

#35 Added an optional call-site id as an arg to spy and brk - #36

Open
npetryk wants to merge 1 commit into
vvvvalvalval:masterfrom
npetryk:named-call-sites
Open

#35 Added an optional call-site id as an arg to spy and brk#36
npetryk wants to merge 1 commit into
vvvvalvalval:masterfrom
npetryk:named-call-sites

Conversation

@npetryk

@npetryk npetryk commented Mar 1, 2019

Copy link
Copy Markdown

No description provided.

@npetryk

npetryk commented Mar 1, 2019

Copy link
Copy Markdown
Author

For issue #35

@npetryk
npetryk force-pushed the named-call-sites branch from a794c26 to 832cb1c Compare March 3, 2019 05:02
@viebel

viebel commented Sep 11, 2019

Copy link
Copy Markdown

@vvvvalvalval @npetryk any progress on merging this (useful) PR?

@vvvvalvalval

Copy link
Copy Markdown
Owner

Sorry for the response delay. I have no problem with extending last-ep-id (feel free to propose a new power doing just that); however, it's not clear to me that user-supplied Code Site ids are desirable (I like the minimalism of them being just negative numbers; opening them to more than this could lead conflicts or validation issues). So not merging this right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants