-
Notifications
You must be signed in to change notification settings - Fork 76
CUDD Release 4.0 #10
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
CUDD Release 4.0 #10
Conversation
Co-authored-by: Nils Husung <[email protected]>
* Add `Cudd::makeZddNode()` * Add `ZDD::Complement()` and `ZDD::operator~`
* Add `Cudd:makeAddNode()` * Add `ADD::operator^()` and `ADD::operator^=()` * Add ADD methods for picking cubes and minterms --------- Co-authored-by: Nils Husung <[email protected]>
doganulus
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't go deep, but some return values may be checked.
| extern DdNode * Cudd_addRoundOff(DdManager *dd, DdNode *f, int N); | ||
| extern DdNode * Cudd_addWalsh(DdManager *dd, DdNode **x, DdNode **y, int n); | ||
| extern DdNode * Cudd_addResidue(DdManager *dd, int n, int m, int options, int top); | ||
| extern int Cudd_addPickOneCube(DdManager *ddm, DdNode *node, char *string); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Return type should be DdNode * as well or int is intentional?
| } // BDD::PickOneMinterm | ||
|
|
||
|
|
||
| void |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't it return result?
| char * string) const | ||
| { | ||
| DdManager *mgr = p->manager; | ||
| int result = Cudd_addPickOneCube(mgr, node, string); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps DdNode *
|
Superseded by #28. |
This will be a maintenance release with a major version bump, despite no (major) changes to the source code being planned. The primary focus is the modernization of the repository layout and build system.