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.

Sign up for GitHub

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

Jump to bottom

Validate results from Tools/c-analyzer/c-statics.py using other tools. #50

Open
ericsnowcurrently opened this issue Jul 5, 2019 · 0 comments
Open

Validate results from Tools/c-analyzer/c-statics.py using other tools. #50

ericsnowcurrently opened this issue Jul 5, 2019 · 0 comments
Labels
complexity: medium moderately hard to figure out P1 medium priority size: medium requires a moderate amount of changes status: blocked waiting for another issue type: code health URGENT must be dealt with ASAP X [isolation] improved interpreter isolation X [runtime] CPython runtime (lifecycle, state)

Comments

@ericsnowcurrently
Copy link
Owner

(blocked by #47)

It is important that we have confidence in the results from the Tools/c-analyzer/c-statics.py script. So we should add at least one mechanism to validate those results using an independent tool.

Here are possible ways to double-check the results from the checker:

  • use pycparser (requires preprocessor, so only local ifdefs)
  • use the nm command (not on Windows; symbols only; locals do not know file or func)
  • use clang (Python bindings) to walk AST
@ericsnowcurrently ericsnowcurrently added X [isolation] improved interpreter isolation X [runtime] CPython runtime (lifecycle, state) P0 high priority type: code health URGENT must be dealt with ASAP status: blocked waiting for another issue size: medium requires a moderate amount of changes complexity: medium moderately hard to figure out labels Jul 5, 2019
@ericsnowcurrently ericsnowcurrently added P1 medium priority and removed P0 high priority labels Jul 5, 2019
@ericsnowcurrently ericsnowcurrently mentioned this issue Jul 5, 2019
Open
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
complexity: medium moderately hard to figure out P1 medium priority size: medium requires a moderate amount of changes status: blocked waiting for another issue type: code health URGENT must be dealt with ASAP X [isolation] improved interpreter isolation X [runtime] CPython runtime (lifecycle, state)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ericsnowcurrently