Skip to content

Adding support for multiple variables of the same type to a plugin for vscode #585

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

Merged
merged 2 commits into from
Mar 15, 2023

Conversation

Nypiaka
Copy link
Collaborator

@Nypiaka Nypiaka commented Feb 15, 2023

No description provided.

@Nypiaka Nypiaka linked an issue Feb 21, 2023 that may be closed by this pull request
@Nypiaka Nypiaka force-pushed the fix_issue_423 branch 5 times, most recently from b86581a to cf080b8 Compare February 24, 2023 10:54
@Nypiaka Nypiaka requested a review from ladisgin February 26, 2023 20:32
@@ -133,6 +134,149 @@ namespace {
"buggy_function2");
}

static bool comparator(const std::string firstVar, const std::string secondVar, bool mustEqual, bool isInt){
Copy link
Member

Choose a reason for hiding this comment

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

Bad naming

Copy link
Member

Choose a reason for hiding this comment

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

Bad naming of parameter

@Nypiaka Nypiaka requested a review from ladisgin March 7, 2023 20:18
return std::stoi(first.view->getEntryValue(nullptr)) == std::stoi(second.view->getEntryValue(nullptr));
}

void checkDifferentVariablesTrue_C(BaseTestGen &testGen){
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary extract to function

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

checkDifferentVariables_C(testGen, true);
}

void checkDifferentVariablesFalse_C(BaseTestGen &testGen){
Copy link
Member

Choose a reason for hiding this comment

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

unnecessary extract to function

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@ladisgin ladisgin merged commit 10c065d into main Mar 15, 2023
@ladisgin ladisgin deleted the fix_issue_423 branch March 15, 2023 15:14
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.

Force KLEE to generate unique values for test parameters
2 participants