diff --git a/src/mlpack/core/io/cli.hpp b/src/mlpack/core/io/cli.hpp index b09d7b92e9..15ad605e31 100644 --- a/src/mlpack/core/io/cli.hpp +++ b/src/mlpack/core/io/cli.hpp @@ -395,7 +395,8 @@ namespace po = boost::program_options; -namespace mlpack { +namespace mlpack +{ // Externally defined in option.hpp, this class holds information about the // program being run. @@ -537,7 +538,8 @@ class ProgramDoc; * collisions are still possible, and they produce bizarre error messages. See * http://mlpack.org/ticket/74 for more information. */ -class CLI { +class CLI +{ public: /** * Adds a parameter to the hierarchy; use the PARAM_*() macros instead of this diff --git a/src/mlpack/core/io/cli_deleter.cpp b/src/mlpack/core/io/cli_deleter.cpp index 57387488c3..e583753490 100644 --- a/src/mlpack/core/io/cli_deleter.cpp +++ b/src/mlpack/core/io/cli_deleter.cpp @@ -17,14 +17,16 @@ using namespace mlpack::io; /*** * Empty constructor that does nothing. */ -CLIDeleter::CLIDeleter() { +CLIDeleter::CLIDeleter() +{ /* nothing to do */ } /*** * This destructor deletes the CLI singleton. */ -CLIDeleter::~CLIDeleter() { +CLIDeleter::~CLIDeleter() +{ // Delete the singleton! CLI::Destroy(); } diff --git a/src/mlpack/core/io/cli_deleter.hpp b/src/mlpack/core/io/cli_deleter.hpp index 4e79e69633..ab131e8cdd 100644 --- a/src/mlpack/core/io/cli_deleter.hpp +++ b/src/mlpack/core/io/cli_deleter.hpp @@ -7,8 +7,10 @@ #ifndef __MLPACK_CORE_IO_CLI_DELETER_HPP #define __MLPACK_CORE_IO_CLI_DELETER_HPP -namespace mlpack { -namespace io { +namespace mlpack +{ +namespace io +{ /** * Extremely simple class whose only job is to delete the existing CLI object at @@ -17,7 +19,8 @@ namespace io { * CLIDeleter class, which will be initialized at the beginning of the program * and deleted at the end. The destructor destroys the CLI singleton. */ -class CLIDeleter { +class CLIDeleter +{ public: CLIDeleter(); ~CLIDeleter(); diff --git a/src/mlpack/core/io/cli_impl.hpp b/src/mlpack/core/io/cli_impl.hpp index e1c35b3b6f..296231011f 100644 --- a/src/mlpack/core/io/cli_impl.hpp +++ b/src/mlpack/core/io/cli_impl.hpp @@ -14,7 +14,8 @@ //Include option.hpp here because it requires CLI but is also templated. #include "option.hpp" -namespace mlpack { +namespace mlpack +{ /** * @brief Adds a parameter to CLI, making it accessibile via GetParam & @@ -32,7 +33,8 @@ template void CLI::Add(const char* identifier, const char* description, const char* parent, - bool required) { + bool required) +{ po::options_description& desc = CLI::GetSingleton().desc; // Generate the full pathname and insert the node into the hierarchy. @@ -61,7 +63,8 @@ void CLI::Add(const char* identifier, * valid. */ template -T& CLI::GetParam(const char* identifier) { +T& CLI::GetParam(const char* identifier) +{ // Used to ensure we have a valid value. T tmp = T(); @@ -71,13 +74,15 @@ T& CLI::GetParam(const char* identifier) { po::variables_map& vmap = GetSingleton().vmap; // If we have the option, set its value. - if (vmap.count(key) && !gmap.count(key)) { + if (vmap.count(key) && !gmap.count(key)) + { gmap[key] = boost::any(vmap[identifier].as()); } // We may have whatever is on the commandline, but what if the programmer has // made modifications? - if (!gmap.count(key)) { // The programmer hasn't done anything; register it + if (!gmap.count(key)) + { // The programmer hasn't done anything; register it gmap[key] = boost::any(tmp); *boost::any_cast(&gmap[key]) = tmp; } diff --git a/src/mlpack/core/io/log.cpp b/src/mlpack/core/io/log.cpp index 10fc0ea1e3..ca4cbd97c4 100644 --- a/src/mlpack/core/io/log.cpp +++ b/src/mlpack/core/io/log.cpp @@ -37,12 +37,15 @@ std::ostream& Log::cout = std::cout; // Only do anything for Assert() if in debugging mode. #ifdef DEBUG -void Log::Assert(bool condition, const char* message) { - if(!condition) { +void Log::Assert(bool condition, const char* message) +{ + if(!condition) + { Log::Debug << message << std::endl; exit(1); } } #else -void Log::Assert(bool condition, const char* message) { } +void Log::Assert(bool condition, const char* message) +{ } #endif diff --git a/src/mlpack/core/io/log.hpp b/src/mlpack/core/io/log.hpp index 8f0683323b..2d16633913 100644 --- a/src/mlpack/core/io/log.hpp +++ b/src/mlpack/core/io/log.hpp @@ -10,7 +10,8 @@ #include "prefixedoutstream.hpp" #include "nulloutstream.hpp" -namespace mlpack { +namespace mlpack +{ /** * Provides a convenient way to give formatted output. @@ -44,7 +45,8 @@ namespace mlpack { * * @see PrefixedOutStream, NullOutStream, CLI */ -class Log { +class Log +{ public: /** * Checks if the specified condition is true. diff --git a/src/mlpack/core/io/nulloutstream.cpp b/src/mlpack/core/io/nulloutstream.cpp index ad65550509..a4d3765aa4 100644 --- a/src/mlpack/core/io/nulloutstream.cpp +++ b/src/mlpack/core/io/nulloutstream.cpp @@ -9,43 +9,61 @@ using namespace mlpack::io; -NullOutStream::NullOutStream() { /* Nothing to do */ } +NullOutStream::NullOutStream() +{ /* Nothing to do */ } -NullOutStream::NullOutStream(const NullOutStream& other) { /* Nothing to do */ } +NullOutStream::NullOutStream(const NullOutStream& other) +{ /* Nothing to do */ } -NullOutStream& NullOutStream::operator<< (bool val) { return *this; } +NullOutStream& NullOutStream::operator<< (bool val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (short val) { return *this; } +NullOutStream& NullOutStream::operator<< (short val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (unsigned short val) { return *this; } +NullOutStream& NullOutStream::operator<< (unsigned short val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (int val) { return *this; } +NullOutStream& NullOutStream::operator<< (int val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (unsigned int val) { return *this; } +NullOutStream& NullOutStream::operator<< (unsigned int val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (long val) { return *this; } +NullOutStream& NullOutStream::operator<< (long val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (unsigned long val) { return *this; } +NullOutStream& NullOutStream::operator<< (unsigned long val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (float val) { return *this; } +NullOutStream& NullOutStream::operator<< (float val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (double val) { return *this; } +NullOutStream& NullOutStream::operator<< (double val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (long double val) { return *this; } +NullOutStream& NullOutStream::operator<< (long double val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (void* val) { return *this; } +NullOutStream& NullOutStream::operator<< (void* val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (std::string& str) { return *this; } +NullOutStream& NullOutStream::operator<< (std::string& str) +{ return *this; } -NullOutStream& NullOutStream::operator<< (const char* str) { return *this; } +NullOutStream& NullOutStream::operator<< (const char* str) +{ return *this; } -NullOutStream& NullOutStream::operator<< (std::streambuf* val) { return *this; } +NullOutStream& NullOutStream::operator<< (std::streambuf* val) +{ return *this; } -NullOutStream& NullOutStream::operator<< (std::ostream& (*pf) (std::ostream&)) { +NullOutStream& NullOutStream::operator<< (std::ostream& (*pf) (std::ostream&)) +{ return *this; } -NullOutStream& NullOutStream::operator<< (std::ios& (*pf) (std::ios&)) { +NullOutStream& NullOutStream::operator<< (std::ios& (*pf) (std::ios&)) +{ return *this; } diff --git a/src/mlpack/core/io/nulloutstream.hpp b/src/mlpack/core/io/nulloutstream.hpp index 661f33882b..ec2822f0af 100644 --- a/src/mlpack/core/io/nulloutstream.hpp +++ b/src/mlpack/core/io/nulloutstream.hpp @@ -68,7 +68,8 @@ class NullOutStream { //! Does nothing. template - NullOutStream& operator<<(T s) { return *this; } + NullOutStream& operator<<(T s) + { return *this; } }; } // namespace io diff --git a/src/mlpack/core/io/option.cpp b/src/mlpack/core/io/option.cpp index 26bf0e7ca2..21c3bccbd0 100644 --- a/src/mlpack/core/io/option.cpp +++ b/src/mlpack/core/io/option.cpp @@ -29,7 +29,8 @@ ProgramDoc::ProgramDoc(const std::string programName, const std::string defaultModule) : programName(programName), documentation(documentation), - defaultModule(defaultModule) { + defaultModule(defaultModule) +{ // Register this with CLI. CLI::RegisterProgramDoc(this); } diff --git a/src/mlpack/core/io/option.hpp b/src/mlpack/core/io/option.hpp index 952abbabe1..812d3788ea 100644 --- a/src/mlpack/core/io/option.hpp +++ b/src/mlpack/core/io/option.hpp @@ -12,8 +12,10 @@ #include "cli.hpp" -namespace mlpack { -namespace io { +namespace mlpack +{ +namespace io +{ /** * A static object whose constructor registers a parameter with the CLI class. @@ -24,7 +26,8 @@ namespace io { * @see core/io/cli.hpp, mlpack::CLI */ template -class Option { +class Option +{ public: /** * Construct an Option object. When constructed, it will register @@ -71,7 +74,8 @@ class Option { * * @see core/io/cli.hpp, mlpack::CLI */ -class ProgramDoc { +class ProgramDoc +{ public: /** * Construct a ProgramDoc object. When constructed, it will register itself diff --git a/src/mlpack/core/io/option_impl.hpp b/src/mlpack/core/io/option_impl.hpp index 70b64e306d..98a4bcde2f 100644 --- a/src/mlpack/core/io/option_impl.hpp +++ b/src/mlpack/core/io/option_impl.hpp @@ -11,8 +11,10 @@ #ifndef __MLPACK_CORE_IO_OPTION_IMPL_HPP #define __MLPACK_CORE_IO_OPTION_IMPL_HPP -namespace mlpack { -namespace io { +namespace mlpack +{ +namespace io +{ /** * Registers a parameter with CLI. @@ -23,10 +25,12 @@ Option::Option(bool ignoreTemplate, const char* identifier, const char* description, const char* parent, - bool required) { + bool required) +{ if (ignoreTemplate) CLI::Add(identifier, description, parent, required); - else { + else + { CLI::Add(identifier, description, parent, required); // Create the full pathname to set the default value. @@ -42,7 +46,8 @@ Option::Option(bool ignoreTemplate, template Option::Option(const char* identifier, const char* description, - const char* parent) { + const char* parent) +{ CLI::AddFlag(identifier, description, parent); } diff --git a/src/mlpack/core/io/optionshierarchy.cpp b/src/mlpack/core/io/optionshierarchy.cpp index c27bc95839..b9bf1bad2a 100644 --- a/src/mlpack/core/io/optionshierarchy.cpp +++ b/src/mlpack/core/io/optionshierarchy.cpp @@ -17,7 +17,8 @@ using namespace mlpack::io; /* Ctors, Dtors, and R2D2 [actually, just copy-tors] */ /* Constructs an empty OptionsHierarchy node. */ -OptionsHierarchy::OptionsHierarchy() : children() { +OptionsHierarchy::OptionsHierarchy() : children() +{ nodeData.node = ""; nodeData.desc = ""; nodeData.tname = ""; @@ -29,7 +30,8 @@ OptionsHierarchy::OptionsHierarchy() : children() { * * @param name The name of the node to be created. */ -OptionsHierarchy::OptionsHierarchy(const char* name) : children() { +OptionsHierarchy::OptionsHierarchy(const char* name) : children() +{ nodeData.node = string(name); nodeData.desc = ""; nodeData.tname = ""; @@ -41,14 +43,16 @@ OptionsHierarchy::OptionsHierarchy(const char* name) : children() { * * @param other The node to be copied */ -OptionsHierarchy::OptionsHierarchy(const OptionsHierarchy& other) { +OptionsHierarchy::OptionsHierarchy(const OptionsHierarchy& other) +{ return; } /* * Destroys the node. */ -OptionsHierarchy::~OptionsHierarchy() { +OptionsHierarchy::~OptionsHierarchy() +{ return; } @@ -59,7 +63,8 @@ OptionsHierarchy::~OptionsHierarchy() { * @param pathname The full pathname of the given node, eg /foo/bar. * @param tname A string unique to the type of the node. */ -void OptionsHierarchy::AppendNode(string& pathname, string& tname) { +void OptionsHierarchy::AppendNode(string& pathname, string& tname) +{ string tmp = string(""); OptionsData d; d.node = pathname; @@ -78,7 +83,8 @@ void OptionsHierarchy::AppendNode(string& pathname, string& tname) { */ void OptionsHierarchy::AppendNode(string& pathname, string& tname, - string& description) { + string& description) +{ OptionsData d; d.node = pathname; d.desc = description; @@ -96,14 +102,16 @@ void OptionsHierarchy::AppendNode(string& pathname, * @param data Specifies all fields of the new node. */ void OptionsHierarchy::AppendNode(string& pathname, string& tname, - string& description, OptionsData& data) { + string& description, OptionsData& data) +{ string name = GetName(pathname); string path = GetPath(pathname); //Append the new name, if it isn't already there if (children.count(name) == 0) children[name] = OptionsHierarchy(name.c_str()); - if (pathname.find('/') == pathname.npos || path.length() < 1) { + if (pathname.find('/') == pathname.npos || path.length() < 1) + { children[name].nodeData = data; return; } @@ -120,12 +128,14 @@ void OptionsHierarchy::AppendNode(string& pathname, string& tname, * @return Pointer to the node with that pathname, * null if not found. */ -OptionsHierarchy* OptionsHierarchy::FindNode(string& pathname) { +OptionsHierarchy* OptionsHierarchy::FindNode(string& pathname) +{ return FindNodeHelper(pathname, pathname); } OptionsHierarchy* OptionsHierarchy::FindNodeHelper(string& pathname, - string& target) { + string& target) +{ string name = GetName(pathname); string path = GetPath(pathname); //If the node is there, recurse to it. @@ -145,7 +155,8 @@ OptionsHierarchy* OptionsHierarchy::FindNodeHelper(string& pathname, * @return The data associated with the node, * eg it's name, description, and value. */ -OptionsData OptionsHierarchy::GetNodeData() { +OptionsData OptionsHierarchy::GetNodeData() +{ return nodeData; } @@ -157,7 +168,8 @@ OptionsData OptionsHierarchy::GetNodeData() { * @return The identifiers of all nodes after the next node in the path, * eg fizz/bar in foo/fizz/bar. */ -string OptionsHierarchy::GetPath(string& pathname) { +string OptionsHierarchy::GetPath(string& pathname) +{ //Want to make sure we return a valid string if (pathname.find('/') == pathname.npos) return string(""); @@ -173,7 +185,8 @@ string OptionsHierarchy::GetPath(string& pathname) { * @return The name of the next node in the path * eg foo in foo/bar. */ -string OptionsHierarchy::GetName(string& pathname) { +string OptionsHierarchy::GetName(string& pathname) +{ //Want to makesure we return a valid string if (pathname.find('/') == pathname.npos) return pathname; @@ -191,7 +204,8 @@ string OptionsHierarchy::GetName(string& pathname) { * @return Vector containing relative pathnames of subordinant nodes. */ std::vector - OptionsHierarchy::GetRelativePaths(std::string& pathname) { + OptionsHierarchy::GetRelativePaths(std::string& pathname) +{ std::vector ret; //Obtain the starting node. @@ -204,7 +218,8 @@ std::vector } std::vector - OptionsHierarchy::GetRelativePathsHelper(OptionsHierarchy& node) { + OptionsHierarchy::GetRelativePathsHelper(OptionsHierarchy& node) +{ std::vector ret; std::vector tmp; @@ -213,7 +228,8 @@ std::vector for(iter = node.children.begin(); iter != node.children.end(); iter++) tmp = GetRelativePathsHelper((*iter).second); - while(tmp.size()) { + while(tmp.size()) + { ret.push_back(tmp.back()); tmp.pop_back(); } @@ -223,7 +239,8 @@ std::vector /* * Prints a node, followed by it's entries and submodules. */ -void OptionsHierarchy::Print() { +void OptionsHierarchy::Print() +{ //Print the node, append '/' if that node is not a leaf PrintNode(); @@ -238,10 +255,12 @@ void OptionsHierarchy::Print() { /* * Prints every node and it's value, if any. */ -void OptionsHierarchy::PrintAll() { +void OptionsHierarchy::PrintAll() +{ PrintNode(); map::iterator iter; - for (iter = children.begin(); iter != children.end(); iter++) { + for (iter = children.begin(); iter != children.end(); iter++) + { iter->second.PrintAll(); } } @@ -249,7 +268,8 @@ void OptionsHierarchy::PrintAll() { /* * Prints every node and it's description. */ -void OptionsHierarchy::PrintAllHelp() { +void OptionsHierarchy::PrintAllHelp() +{ // Special case for the top of the hierarchy. if (nodeData.node == "Allowed Options") cout << "Allowed Options:" << endl << endl; @@ -259,7 +279,8 @@ void OptionsHierarchy::PrintAllHelp() { // Now print all the children. map::iterator iter; // First print modules. - for (iter = children.begin(); iter != children.end(); iter++) { + for (iter = children.begin(); iter != children.end(); iter++) + { if (iter->second.children.size() > 0) iter->second.PrintAllHelp(); } @@ -270,7 +291,8 @@ void OptionsHierarchy::PrintAllHelp() { if (nodeData.node == "Allowed Options") cout << "Other options:" << endl << endl; - for (iter = children.begin(); iter != children.end(); iter++) { + for (iter = children.begin(); iter != children.end(); iter++) + { if (iter->second.children.size() == 0) iter->second.PrintAllHelp(); } @@ -280,41 +302,51 @@ void OptionsHierarchy::PrintAllHelp() { } /* Prints all children of this node which are parents */ -void OptionsHierarchy::PrintBranches() { +void OptionsHierarchy::PrintBranches() +{ map::iterator iter; // Iterate through all children for (iter = children.begin(); iter != children.end(); iter++) // Does this child have children? - if (iter->second.children.size()) { + if (iter->second.children.size()) + { iter->second.PrintNode(); } } /* Prints all children nodes that have no children themselves */ -void OptionsHierarchy::PrintLeaves() { +void OptionsHierarchy::PrintLeaves() +{ map::iterator iter; - for (iter = children.begin(); iter != children.end(); iter++) { - if (!iter->second.children.size()) { + for (iter = children.begin(); iter != children.end(); iter++) + { + if (!iter->second.children.size()) + { // Print the node's name, data, and description, but only if it is not a // timer. if (iter->second.nodeData.tname != TYPENAME(timeval)) iter->second.PrintNode(); - } else { + } else + { iter->second.PrintLeaves(); } } } /* Prints all children nodes that are timers */ -void OptionsHierarchy::PrintTimers() { +void OptionsHierarchy::PrintTimers() +{ map::iterator iter; - for (iter = children.begin(); iter != children.end(); iter++) { - if (!iter->second.children.size()) { + for (iter = children.begin(); iter != children.end(); iter++) + { + if (!iter->second.children.size()) + { if (iter->second.nodeData.tname == TYPENAME(timeval)) iter->second.PrintNode(); - } else { + } else + { iter->second.PrintTimers(); } } @@ -323,14 +355,16 @@ void OptionsHierarchy::PrintTimers() { /* * Prints a node and its value. */ -void OptionsHierarchy::PrintNode() { +void OptionsHierarchy::PrintNode() +{ Log::Info << " " << nodeData.node << " = " ; if (nodeData.tname == TYPENAME(bool)) Log::Info << boolalpha << CLI::GetParam(nodeData.node.c_str()); else if (nodeData.tname == TYPENAME(int)) Log::Info << CLI::GetParam(nodeData.node.c_str()); - else if (nodeData.tname == TYPENAME(std::string)) { + else if (nodeData.tname == TYPENAME(std::string)) + { std::string value = CLI::GetParam(nodeData.node.c_str()); if (value == "") value = "\"\""; // So that the user isn't presented with an empty space. @@ -339,7 +373,8 @@ void OptionsHierarchy::PrintNode() { Log::Info << CLI::GetParam(nodeData.node.c_str()); else if (nodeData.tname == TYPENAME(double)) Log::Info << CLI::GetParam(nodeData.node.c_str()); - else if (nodeData.tname == TYPENAME(timeval)) { + else if (nodeData.tname == TYPENAME(timeval)) + { timeval& t = CLI::GetParam(nodeData.node.c_str()); Log::Info << t.tv_sec << "." << std::setw(6) << std::setfill('0') << t.tv_usec << "s"; @@ -350,27 +385,32 @@ void OptionsHierarchy::PrintNode() { int seconds = (t.tv_sec % 60); // No output if it didn't even take a minute. - if (!(days == 0 && hours == 0 && minutes == 0)) { + if (!(days == 0 && hours == 0 && minutes == 0)) + { bool output = false; // Denotes if we have output anything yet. Log::Info << " ("; // Only output units if they have nonzero values (yes, a bit tedious). - if (days > 0) { + if (days > 0) + { Log::Info << days << " days"; output = true; } - if (hours > 0) { + if (hours > 0) + { if (output) Log::Info << ", "; Log::Info << hours << " hrs"; output = true; } - if (minutes > 0) { + if (minutes > 0) + { if (output) Log::Info << ", "; Log::Info << minutes << " mins"; output = true; } - if (seconds > 0) { + if (seconds > 0) + { if (output) Log::Info << ", "; Log::Info << seconds << "." << std::setw(1) << (t.tv_usec / 100000) << @@ -389,10 +429,12 @@ void OptionsHierarchy::PrintNode() { * Prints a node and its description. The format is similar to that help given * by the ImageMagick suite of programs. */ -void OptionsHierarchy::PrintNodeHelp() { +void OptionsHierarchy::PrintNodeHelp() +{ // We want to print differently if this is a module node (i.e. if it has any // children). - if (children.size() > 0) { + if (children.size() > 0) + { if (nodeData.node == "default") // Special case for default module. cout << "Default options:" << endl; else // Other standard module title output. @@ -431,13 +473,15 @@ void OptionsHierarchy::PrintNodeHelp() { // So, we only want to use a new line if we have used more than 30 characters // already. Descriptions start at character 30. - if (len < 30) { + if (len < 30) + { cout << std::string(30 - len, ' '); if (nodeData.desc.length() > 0) cout << HyphenateString(nodeData.desc, 30) << endl; else cout << "Undocumented option." << endl; - } else { + } else + { cout << endl << std::string(30, ' '); if (nodeData.desc.length() > 0) cout << HyphenateString(nodeData.desc, 30) << endl; @@ -453,7 +497,8 @@ void OptionsHierarchy::PrintNodeHelp() { * @param str String to hyphenate (splits are on ' '). * @param padding Amount of padding on the left for each new line. */ -string OptionsHierarchy::HyphenateString(string str, int padding) { +string OptionsHierarchy::HyphenateString(string str, int padding) +{ size_t margin = 80 - padding; if (str.length() < margin) return str; @@ -462,15 +507,19 @@ string OptionsHierarchy::HyphenateString(string str, int padding) { unsigned int pos = 0; // First try to look as far as possible. - while(pos < str.length() - 1) { + while(pos < str.length() - 1) + { size_t splitpos; // Check that we don't have a newline first. splitpos = str.find('\n', pos); - if (splitpos == string::npos || splitpos > (pos + margin)) { + if (splitpos == string::npos || splitpos > (pos + margin)) + { // We did not find a newline. - if (str.length() - pos < margin) { + if (str.length() - pos < margin) + { splitpos = str.length(); // The rest fits on one line. - } else { + } else + { splitpos = str.rfind(' ', margin + pos); // Find nearest space. if (splitpos <= pos || splitpos == string::npos) // Not found. splitpos = pos + margin; @@ -478,7 +527,8 @@ string OptionsHierarchy::HyphenateString(string str, int padding) { } out += str.substr(pos, (splitpos - pos)); - if (splitpos < str.length()) { + if (splitpos < str.length()) + { out += '\n'; out += string(padding, ' '); } diff --git a/src/mlpack/core/io/optionshierarchy.hpp b/src/mlpack/core/io/optionshierarchy.hpp index 44e3e0bf53..e10e0882ef 100644 --- a/src/mlpack/core/io/optionshierarchy.hpp +++ b/src/mlpack/core/io/optionshierarchy.hpp @@ -12,14 +12,17 @@ #include #include -namespace mlpack { -namespace io { +namespace mlpack +{ +namespace io +{ /** * Aids in the extensibility of OptionsHierarchy by focusing the potential * changes into one structure. */ -struct OptionsData { +struct OptionsData +{ //! Name of this node. std::string node; //! Description of this node, if any. @@ -33,7 +36,8 @@ struct OptionsData { * information about itself and can have any number of children, each with * unique names. */ -class OptionsHierarchy { +class OptionsHierarchy +{ private: //! Holds all node specific data. OptionsData nodeData; diff --git a/src/mlpack/core/io/prefixedoutstream.cpp b/src/mlpack/core/io/prefixedoutstream.cpp index 3b3981ebe6..14e63d3be5 100644 --- a/src/mlpack/core/io/prefixedoutstream.cpp +++ b/src/mlpack/core/io/prefixedoutstream.cpp @@ -21,90 +21,107 @@ using namespace mlpack::io; * may not be the actual case, but it works when these is here. */ -PrefixedOutStream& PrefixedOutStream::operator<<(bool val) { +PrefixedOutStream& PrefixedOutStream::operator<<(bool val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(short val) { +PrefixedOutStream& PrefixedOutStream::operator<<(short val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(unsigned short val) { +PrefixedOutStream& PrefixedOutStream::operator<<(unsigned short val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(int val) { +PrefixedOutStream& PrefixedOutStream::operator<<(int val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(unsigned int val) { +PrefixedOutStream& PrefixedOutStream::operator<<(unsigned int val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(long val) { +PrefixedOutStream& PrefixedOutStream::operator<<(long val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(unsigned long val) { +PrefixedOutStream& PrefixedOutStream::operator<<(unsigned long val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(float val) { +PrefixedOutStream& PrefixedOutStream::operator<<(float val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(double val) { +PrefixedOutStream& PrefixedOutStream::operator<<(double val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(long double val) { +PrefixedOutStream& PrefixedOutStream::operator<<(long double val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(void* val) { +PrefixedOutStream& PrefixedOutStream::operator<<(void* val) +{ BaseLogic(val); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(const char* str) { +PrefixedOutStream& PrefixedOutStream::operator<<(const char* str) +{ BaseLogic(str); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(std::string& str) { +PrefixedOutStream& PrefixedOutStream::operator<<(std::string& str) +{ BaseLogic(str); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(std::streambuf* sb) { +PrefixedOutStream& PrefixedOutStream::operator<<(std::streambuf* sb) +{ BaseLogic(sb); return *this; } PrefixedOutStream& PrefixedOutStream::operator<<( - std::ostream& (*pf)(std::ostream&)) { + std::ostream& (*pf)(std::ostream&)) +{ BaseLogic(pf); return *this; } -PrefixedOutStream& PrefixedOutStream::operator<<(std::ios& (*pf)(std::ios&)) { +PrefixedOutStream& PrefixedOutStream::operator<<(std::ios& (*pf)(std::ios&)) +{ BaseLogic(pf); return *this; } PrefixedOutStream& PrefixedOutStream::operator<<( - std::ios_base& (*pf) (std::ios_base&)) { + std::ios_base& (*pf) (std::ios_base&)) +{ BaseLogic(pf); return *this; } diff --git a/src/mlpack/core/io/prefixedoutstream.hpp b/src/mlpack/core/io/prefixedoutstream.hpp index b4f2f818ba..736276f5a7 100644 --- a/src/mlpack/core/io/prefixedoutstream.hpp +++ b/src/mlpack/core/io/prefixedoutstream.hpp @@ -15,8 +15,10 @@ #include -namespace mlpack { -namespace io { +namespace mlpack +{ +namespace io +{ /** * Allows us to output to an ostream with a prefix at the beginning of each @@ -43,7 +45,8 @@ namespace io { * These objects are used for the mlpack::Log levels (DEBUG, INFO, WARN, and * FATAL). */ -class PrefixedOutStream { +class PrefixedOutStream +{ public: /** * Set up the PrefixedOutStream.