On 08/10/2013 06:04 PM, Paolo Carlini wrote:
Also, in the new comment, you refer to a "stack", but in the same patch you removed a std::stack and added a std::vector as data member, something seems inconsistent.I had having a closer look to this: if one looks at the code now, without knowing that a std::stack was there probably the comment makes perfect sense, doesn't appear confusing (after all you are just using push_back and pop_back, and std::vector is fine for that).
Paolo.
