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

adds tracing for checks #34

Conversation

christophkloeffel
Copy link
Contributor

adds tracing for chapter 10

Copy link
Collaborator

@florianschanda florianschanda left a comment

Choose a reason for hiding this comment

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

LRM.Check_Block

Needs link to:

  • ast.Check_Block
  • ast.Check

LRM.Check_Evaluation_Order

Needs link to:

  • ast.Record_Object.perform_checks
  • ast.Composite_Type.add_check
  • ast.Composite_Type.iter_checks
  • ast.Record_Type.iter_checks
  • ast.Tuple_Type.perform_type_checks

LRM.Check_Evaluation_Order_For_Extensions

I do not think the link to parser.Parser.parse_check_block makes a
lot of sense.

Needs link to:

  • ast.Record_Type.iter_checks

language-reference-manual/lrm.trlc Show resolved Hide resolved
tests-system/checks-10/tracing Outdated Show resolved Hide resolved
@christophkloeffel christophkloeffel force-pushed the tracing/chapter-10 branch 5 times, most recently from 6c16dd3 to c5c2677 Compare October 15, 2023 16:15
@christophkloeffel
Copy link
Contributor Author

christophkloeffel commented Oct 15, 2023

LRM.Check_Block

Needs link to:

  • ast.Check_Block
  • ast.Check

After tracing a class it is not possible to trace the methods within anymore. They get ignored in the tracing report. Is this intended?

as of now LRM.Check_Messages and LRM.Check_Severity are not traced to ast.Check.perform because of this

@florianschanda florianschanda merged commit 7107b77 into bmw-software-engineering:main Oct 20, 2023
5 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