I'm a strong believer that the better way to teach something is to give example: the documentation doesn't have one.
The interface is not easy to use: if I have to provide a function reference and the documentation describes the interface of it (expected parameters and return values) that means that the distribution should have at least a implementation, even if very basic.
The distribution does not includes any tests at all, which contributes for poor documentation (no examples at all).
1 out of 1 found this review helpful. Was this review helpful to you? Yes No
If you want to show any activity at the terminal while working on something that takes a while to work, this distribution will work greatly!
The only issue that I had is regarding using the options rotate and time: if I disable the time with time => 0 (at least, I think this is a boolean value because the documentation does not explains it), the time count is disable but the output becames messy.
Lack of documentation is a big issue since I'm not sure which should be the expected behaviour, but disabling time count seems to cause a bug.
2 out of 2 found this review helpful. Was this review helpful to you? Yes No
I'm using FSA-Rules for one of my open source projects (code.google.com/p/siebel-monitoring-t... ) for parsing output and I would probably would be in trouble without this module.
I must say that this is the first time that I'm using a state machine for a parser, so this reviewed is based on a newbie point of view related to state machines.
The distribution works as expected. It looks simple to use, but the lack of good examples in the documentation make it hard to create some working code. Documentation really lacks an example to GraphViz when using the graph() method. It is really cool to have such functionality but again the lack of documentation makes life harder. Looks like it needs a review anyway since GraphViz is deprecated in favour of GraphViz2.
The API is OK, but could be improved in the sense that after adding new states, code becomes harder and harder to read since the object is created with a big data references (hash and array references mixed). It would be great if the new states could be added just as new created objects (could be even greater if I could add them Perl modules in a directory and then they could be loaded during execution time).
Was this review helpful to you? Yes No