active-logic / activelogic-cs

Behavior Trees and Case Logic for C#7
GNU Affero General Public License v3.0
106 stars 7 forks source link

Plain status constants #50

Closed eelstork closed 3 years ago

eelstork commented 3 years ago

For logging purposes status constants (done, cont, fail) are postfixed with a () which may optionally include logging information:

return done(log && "All caught up");

Without disregarding #28, test-oriented workflows are less dependent on logging and, overall, an option to use plain status constants would be welcome.

Still unclear whether both plain and annotated status constants may coexist in the same class (it only looks impossible). That isn't really a problem though.

Also, marginal performance gains with raw constants.

eelstork commented 3 years ago

Reasonably, splitting the namespace should be the first option.

eelstork commented 3 years ago

813a48d5d161070b56b32303d3c0e838f5e955c6