WIP: Resolve "Rework Problem::{check,print} into a recursive approach"

Closes #296 (closed)

added 70 commits

• 23572ae6...2e497182 - 69 commits from branch devel
• 0e21160a - Added option to layout stablocks in a more readable fashion if _metric and...

Compare with previous version

added 1 commit

• 1e53a5d5 - Rework print() into recursive approach

Compare with previous version

added 1 commit

• 0c90bd21 - Make printHeader() more uniform

Compare with previous version

added 2 commits

• 4273bdbd - Add indentation parameter for print
• a10fe38f - Labels name fix @problem::check

Compare with previous version

OK, but the indentation param _tabs should be the last one, with a default equal to "" so that I can print specific nodes and their children, and they appear flush to the left.

What I was imagining was something like

Parent::printHeader(params, string _tabs = "") const
{
   stream << _tabs << bla bla << endl;
}

Parent::print(params, string _tabs = "") const
{
   print Header(params, _tabs);
   for (const auto& child : clildrenList())
       child.print(params, _tabs + "  "); // SEE THAT WE ADD INDENTATION HERE
}

Problem::print(params) const; // NO TABS HERE!

Now you can do problem.print(params), like always. But you can also do Capture.print(params without the _tabs specified) and then Captures will have no indent, Features will indent two spaces, and Factors will indent 4 spaces.

added 2 commits

• 5b62e2a1 - Set default value for _tabs @problem::print
• f8aa9e0d - Add dynamic state blocks check @CaptureBase::print

Compare with previous version

In Xxx::print(...) I still dont get the order of the operands. Why is the std::stream in second position?

It should also be possible to call Xxx::print(4,1,1,1) without the output element and with Xxx any WOLF node in the tree. This is not possible in CaptureBase at least.

A question: how do I have to use the std::stream paramenter if e.g. I want to output to be a std::string?

added 1 commit

• a4dc2bc7 - Add test for Capture::print()

Compare with previous version

@jsola The stream object should have a .c_str() function to transform its contents into a std::string.

added 1 commit

• 03b96575 - Rearrange print function arguments

Compare with previous version

added 1 commit

• a3f54c5e - Rework check() 1st iteration

Compare with previous version

closed