Skip to content

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
"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

3 participants