Code review comment for lp:~zorba-coders/zorba/1025564

Revision history for this message
Matthias Brantner (matthias-brantner) wrote :

Works great. Some minor issues:

- the --help documentation should mention that the argument is deprecated
- also, the --help documentation should mention the syntax to pass the file
- please respect coding guidelines. For example,
if(qBeforeF()){ => if (qBeforeF())
{
- remove commented out code

review: Needs Fixing

« Back to merge proposal