Skip to content

Adding position of next script token to TaintLocation#347

Merged
leeN merged 1 commit into
SAP:mainfrom
tmbrbr:next_pos
Nov 10, 2025
Merged

Adding position of next script token to TaintLocation#347
leeN merged 1 commit into
SAP:mainfrom
tmbrbr:next_pos

Conversation

@tmbrbr

@tmbrbr tmbrbr commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Currently we only store the source code position of the current program pointer when creating TaintLocations.

This commit computes the position of the next-highest source code position, to enable highlighting of locations in source code.

@leeN leeN left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tmbrbr

tmbrbr commented Nov 10, 2025

Copy link
Copy Markdown
Contributor Author

It looks like the SpiderMonkey tests are failing, let me check what is going wrong...

@leeN leeN merged commit e22ce85 into SAP:main Nov 10, 2025
7 of 12 checks passed
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.

2 participants