Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: instrumentation bug against programs with inheritance #51

Merged
merged 15 commits into from
Dec 1, 2024
Merged

Conversation

masuhar
Copy link
Contributor

@masuhar masuhar commented Dec 1, 2024

  • added tests (using jest)
  • corrected example programs so as not to produce errors
  • fixed transformer to support inheritance
    • transformers used to insert code fragments at the beginning of every method, including constructors, and wrap every function call, including super calls, to record logs.
    • these transformations make super-constructor-calls incorrect because they should be placed at the beginning of a constructor, and super is not a variable
    • the bugs are fixed by not transforming super constructor calls, and inserting check pointing code just after the super constructor calls.

@masuhar masuhar merged commit 2d2cf9c into master Dec 1, 2024
@masuhar
Copy link
Contributor Author

masuhar commented Dec 1, 2024

resolves #6 and #45

@masuhar masuhar deleted the add-test branch December 8, 2024 05:18
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.

1 participant