[Merge] lp:~abreu-alexandre/webbrowser-app/devtools-support into lp:webbrowser-app

Olivier Tilloy olivier.tilloy at canonical.com
Mon Jul 28 21:06:29 UTC 2014


Review: Needs Fixing

142	+    out << "  --inspector[=PORT] run a remote inspector on a specified port or" << REMOTE_INSPECTOR_PORT << "as the default port" << endl;

155	+    out << "  --inspector[=PORT]                  run a remote inspector on a specified port or" << REMOTE_INSPECTOR_PORT << "as the default port" << endl;

You’re missing whitespaces after "or" and before "as". This is how it looks when invoking the app with --help:

    run a remote inspector on a specified port or9221as the default port
-- 
https://code.launchpad.net/~abreu-alexandre/webbrowser-app/devtools-support/+merge/225561
Your team Ubuntu Phablet Team is subscribed to branch lp:webbrowser-app.



More information about the Ubuntu-reviews mailing list