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

AddChildren: Filter out false children #220

Merged
merged 2 commits into from
Jul 11, 2023

Conversation

ajor
Copy link
Contributor

@ajor ajor commented Jul 7, 2023

Use fully qualified names to determine if a class is really the child of our type. It may be that it is the child of another type with an identical name in another namespace.

@ajor ajor force-pushed the add-children-false-children branch from ae9327e to fe48500 Compare July 7, 2023 16:13
@codecov-commenter
Copy link

codecov-commenter commented Jul 7, 2023

Codecov Report

Merging #220 (fe48500) into main (56b5873) will increase coverage by 0.81%.
The diff coverage is 94.44%.

@@            Coverage Diff             @@
##             main     #220      +/-   ##
==========================================
+ Coverage   65.31%   66.12%   +0.81%     
==========================================
  Files          88       90       +2     
  Lines        9805     9849      +44     
  Branches     1609     1614       +5     
==========================================
+ Hits         6404     6513     +109     
+ Misses       2561     2488      -73     
- Partials      840      848       +8     
Impacted Files Coverage Δ
oi/type_graph/AddChildren.h 100.00% <ø> (+100.00%) ⬆️
test/test_flattener.cpp 94.75% <ø> (ø)
oi/type_graph/Flattener.cpp 91.35% <50.00%> (-0.41%) ⬇️
oi/type_graph/AddChildren.cpp 83.11% <77.77%> (+83.11%) ⬆️
oi/type_graph/Types.cpp 71.42% <83.33%> (+7.36%) ⬆️
oi/type_graph/Printer.cpp 98.70% <100.00%> (ø)
oi/type_graph/Types.h 100.00% <100.00%> (+1.44%) ⬆️
test/test_add_children.cpp 100.00% <100.00%> (ø)
test/test_drgn_parser.cpp 99.31% <100.00%> (+0.73%) ⬆️
test/test_drgn_parser.h 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

ajor added 2 commits July 10, 2023 08:42
Use fully qualified names to determine if a class is really the child of
our type. It may be that it is the child of another type with an
identical name in another namespace.
@ajor ajor force-pushed the add-children-false-children branch from fe48500 to 2b03d3d Compare July 10, 2023 15:42
@ajor ajor marked this pull request as ready for review July 10, 2023 15:44
@ajor ajor merged commit f676112 into facebookexperimental:main Jul 11, 2023
@ajor ajor deleted the add-children-false-children branch July 11, 2023 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants