[mlpack-git] [mlpack/mlpack] fix default output problem and some styles (#680)

Ryan Curtin notifications at github.com
Tue Jun 14 19:29:51 EDT 2016


> @@ -72,6 +73,10 @@ int main(int argc, char** argv)
>    // Parse command line options.
>    CLI::ParseCommandLine(argc, argv);
>  
> +  if (CLI::HasParam("output_file"))
> +    Log::Warn << "--output_file (-o) is not specified;"
> +      << "no results will be saved!" << endl;

Could you reformat this a little bit please? (There are lots of messages like this so my comment applies to all of them, not just this one.)  More of the message might fit on one line, and when you wrap, be sure to have two extra indents before the `<< "rest of the message." << endl` part.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/pull/680/files/8ebabddf829267008c22393820e07dc753d289c7#r67076673
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160614/d7645284/attachment.html>


More information about the mlpack-git mailing list