Replies: 4 comments 9 replies
-
Enhancements
|
Beta Was this translation helpful? Give feedback.
-
@klingaard is there a typo in that somewhere? Were you meaning to say "Also, the DAG includes nodes in the error message that are not really a part of the cyclic behavior."? Can you clarify that point in your OP? |
Beta Was this translation helpful? Give feedback.
-
Does this include the PARAMETER macros? Well, I guess you say callbacks so that would imply handlers here but it would be nice if we could use C++ to generate the PARAMETERs as well. |
Beta Was this translation helpful? Give feedback.
-
|
Per discussion in #547: we should probably fix undefined behavior of destruction order of TreeNodes contained in the |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Been chatting about making a v3 of MAP that addresses lots of annoyances/issues/missing features of the framework.
Enhancements
-- Related issues: sparta ReportHeader is clumsy with stdlib types #44, Disable pipeline collection on strings #13, Conditionally compile Collectable::collect on ostream operator availability #45, Remove pipeline collection functionalities from Resources #55
.dotfile with the cycle. This is useful, but ... not. Also, the DAG includes in the cyclic error, nodes that are really part of the cyclic behavior like the sparta phase groups.Cleanup
eraseshould return the next iterator, etcsparta::TreeNodewas cleaned up a bit, but can use more cleanup to reduce compilation timeclang-formatcoutorcerrshould be prepended with something likeSPARTA:so developers know where the message originatedBug Fixes
Reports
unsigned long, but if the notification is anunsigned int, the error message just says, "cannot find the notification"Beta Was this translation helpful? Give feedback.
All reactions