]> OCCT Git - occt.git/commitdiff
CR33200 added limits include CR33200
authorimochoa <nachomartinochoa@gmail.com>
Tue, 15 Nov 2022 11:01:51 +0000 (12:01 +0100)
committerimochoa <nachomartinochoa@gmail.com>
Tue, 15 Nov 2022 11:01:51 +0000 (12:01 +0100)
src/ViewerTest/ViewerTest_CmdParser.cxx

index 04cb7d4b1cf49fe0f8c4dd2e6dc4cd47ff21be49..ca728c39f34e4628cbdaaf362784ab88b2792303 100644 (file)
@@ -1,3 +1,4 @@
+#include <limits>
 // Created on: 2015-03-15
 // Created by: Danila ULYANOV
 // Copyright (c) 2014 OPEN CASCADE SAS
@@ -7,8 +8,9 @@
 // This library is free software; you can redistribute it and/or modify it under
 // the terms of the GNU Lesser General Public License version 2.1 as published
 // by the Free Software Foundation, with special exception defined in the file
-// OCCT_LGPL_EXCEPTION.txt. Consult the file LICENSE_LGPL_21.txt included in OCCT
-// distribution for complete text of the license and disclaimer of any warranty.
+// OCCT_LGPL_EXCEPTION.txt. Consult the file LICENSE_LGPL_21.txt included in
+// OCCT distribution for complete text of the license and disclaimer of any
+// warranty.
 //
 // Alternatively, this file may be used under the terms of Open CASCADE
 // commercial license or contractual agreement.
 
 #include <algorithm>
 #include <iostream>
-
-namespace
-{
-
-  //! Converts the given string to lowercase
-  //! @param theString the string to be converted
-  //! @return a converted string (a string in lowercase)
-  static std::string toLowerCase (std::string theString)
-  {
-    std::transform (theString.begin(), theString.end(), theString.begin(), ::LowerCase);
-    return theString;
-  }
-
-  //! Converts the vector of std::strings to a vector of pointers to its data
-  //! @param theStringList the vector of strings to be converted
-  //! @return a vector of pointers to the data of given strings
-  static std::vector<const char*> convertToRawStringList (const std::vector<std::string>& theStringList)
-  {
-    const std::size_t        aListSize = theStringList.size();
-    std::vector<const char*> aRawStringList (aListSize);
-    for (std::size_t anIndex = 0; anIndex < aListSize; ++anIndex)
-    {
-      aRawStringList[anIndex] = theStringList[anIndex].c_str();
-    }
-    return aRawStringList;
+#include <limits>
+
+namespace {
+
+//! Converts the given string to lowercase
+//! @param theString the string to be converted
+//! @return a converted string (a string in lowercase)
+static std::string toLowerCase(std::string theString) {
+  std::transform(theString.begin(), theString.end(), theString.begin(),
+                 ::LowerCase);
+  return theString;
+}
+
+//! Converts the vector of std::strings to a vector of pointers to its data
+//! @param theStringList the vector of strings to be converted
+//! @return a vector of pointers to the data of given strings
+static std::vector<const char *>
+convertToRawStringList(const std::vector<std::string> &theStringList) {
+  const std::size_t aListSize = theStringList.size();
+  std::vector<const char *> aRawStringList(aListSize);
+  for (std::size_t anIndex = 0; anIndex < aListSize; ++anIndex) {
+    aRawStringList[anIndex] = theStringList[anIndex].c_str();
   }
+  return aRawStringList;
+}
 
 } // namespace
 
-const std::size_t ViewerTest_CmdParser::THE_UNNAMED_COMMAND_OPTION_KEY = (std::numeric_limits<std::size_t>::max)();
+const std::size_t ViewerTest_CmdParser::THE_UNNAMED_COMMAND_OPTION_KEY =
+    (std::numeric_limits<std::size_t>::max)();
 
 const std::size_t ViewerTest_CmdParser::THE_HELP_COMMAND_OPTION_KEY = 0;
 
@@ -58,50 +60,48 @@ const std::size_t ViewerTest_CmdParser::THE_HELP_COMMAND_OPTION_KEY = 0;
 // function : ViewerTest_CmdParser
 // purpose  :
 //===============================================================================================
-ViewerTest_CmdParser::ViewerTest_CmdParser (const std::string& theDescription) : myDescription (theDescription)
-{
-  AddOption ("help|h", "Prints a short description of the command and its options.");
+ViewerTest_CmdParser::ViewerTest_CmdParser(const std::string &theDescription)
+    : myDescription(theDescription) {
+  AddOption("help|h",
+            "Prints a short description of the command and its options.");
 }
 
 //===============================================================================================
 // function : AddOption
 // purpose  :
 //===============================================================================================
-ViewerTest_CommandOptionKey ViewerTest_CmdParser::AddOption (const std::string& theOptionNames,
-                                                             const std::string& theOptionDescription)
-{
+ViewerTest_CommandOptionKey
+ViewerTest_CmdParser::AddOption(const std::string &theOptionNames,
+                                const std::string &theOptionDescription) {
   CommandOption aNewOption;
 
   // extract option names
   std::vector<std::string> aNames;
-  std::stringstream        aStream (theOptionNames);
-  std::string              anItem;
-  while (std::getline (aStream, anItem, '|'))
-  {
-    if (!anItem.empty())
-    {
-      aNames.push_back (anItem);
+  std::stringstream aStream(theOptionNames);
+  std::string anItem;
+  while (std::getline(aStream, anItem, '|')) {
+    if (!anItem.empty()) {
+      aNames.push_back(anItem);
     }
   }
 
   aNewOption.Name = aNames.front();
-  if (aNames.size() > 1)
-  {
+  if (aNames.size() > 1) {
     const std::size_t aNumberOfAliases = aNames.size() - 1;
 
-    aNewOption.Aliases.reserve (aNumberOfAliases);
-    std::copy (aNames.begin() + 1, aNames.end(), std::back_inserter (aNewOption.Aliases));
+    aNewOption.Aliases.reserve(aNumberOfAliases);
+    std::copy(aNames.begin() + 1, aNames.end(),
+              std::back_inserter(aNewOption.Aliases));
   }
   aNewOption.Description = theOptionDescription;
 
   const ViewerTest_CommandOptionKey aNewOptionKey = myOptionStorage.size();
 
-  myOptionStorage.push_back (aNewOption);
+  myOptionStorage.push_back(aNewOption);
 
   std::vector<std::string>::const_iterator anIt = aNames.begin();
-  for (; anIt != aNames.end(); ++anIt)
-  {
-    const std::string aNameInLowerCase = toLowerCase (*anIt);
+  for (; anIt != aNames.end(); ++anIt) {
+    const std::string aNameInLowerCase = toLowerCase(*anIt);
 
     myOptionMap[aNameInLowerCase] = aNewOptionKey;
   }
@@ -113,21 +113,18 @@ ViewerTest_CommandOptionKey ViewerTest_CmdParser::AddOption (const std::string&
 // function : PrintHelp
 // purpose  :
 //===============================================================================================
-void ViewerTest_CmdParser::PrintHelp() const
-{
+void ViewerTest_CmdParser::PrintHelp() const {
   std::cout << myDescription << std::endl;
   std::vector<CommandOption>::const_iterator anIt = myOptionStorage.begin();
-  for (++anIt; anIt != myOptionStorage.end(); ++anIt)
-  {
-    const CommandOption& aCommandOption = *anIt;
+  for (++anIt; anIt != myOptionStorage.end(); ++anIt) {
+    const CommandOption &aCommandOption = *anIt;
     std::cout << "\n\t-" << aCommandOption.Name;
-    const OptionAliases& anAliases = aCommandOption.Aliases;
-    if (!anAliases.empty())
-    {
+    const OptionAliases &anAliases = aCommandOption.Aliases;
+    if (!anAliases.empty()) {
       std::cout << " (-" << anAliases.front();
-      for (OptionAliases::const_iterator anAliasIterator = anAliases.begin() + 1; anAliasIterator != anAliases.end();
-           ++anAliasIterator)
-      {
+      for (OptionAliases::const_iterator anAliasIterator =
+               anAliases.begin() + 1;
+           anAliasIterator != anAliases.end(); ++anAliasIterator) {
         std::cout << ", -" << *anAliasIterator;
       }
       std::cout << ")";
@@ -141,34 +138,28 @@ void ViewerTest_CmdParser::PrintHelp() const
 // function : Parse
 // purpose  :
 //===============================================================================================
-void ViewerTest_CmdParser::Parse (const Standard_Integer theArgsNb, const char* const* const theArgVec)
-{
+void ViewerTest_CmdParser::Parse(const Standard_Integer theArgsNb,
+                                 const char *const *const theArgVec) {
   std::size_t aCurrentUsedOptionIndex = 0;
-  for (Standard_Integer anIter = 1; anIter < theArgsNb; ++anIter)
-  {
-    const char* const anArgument = theArgVec[anIter];
-    if (anArgument[0] == '-' && !std::isdigit (anArgument[1]))
-    {
-      const std::string   anOptionName = toLowerCase (anArgument + 1);
-      OptionMap::iterator aMapIter     = myOptionMap.find (anOptionName);
-      if (aMapIter != myOptionMap.end())
-      {
-        const ViewerTest_CommandOptionKey aCurrentUsedOptionKey = aMapIter->second;
-        aCurrentUsedOptionIndex                                 = addUsedOption (aCurrentUsedOptionKey);
-      }
-      else
-      {
-        Message::SendFail() << "Error: unknown argument '" << anOptionName << "'";
+  for (Standard_Integer anIter = 1; anIter < theArgsNb; ++anIter) {
+    const char *const anArgument = theArgVec[anIter];
+    if (anArgument[0] == '-' && !std::isdigit(anArgument[1])) {
+      const std::string anOptionName = toLowerCase(anArgument + 1);
+      OptionMap::iterator aMapIter = myOptionMap.find(anOptionName);
+      if (aMapIter != myOptionMap.end()) {
+        const ViewerTest_CommandOptionKey aCurrentUsedOptionKey =
+            aMapIter->second;
+        aCurrentUsedOptionIndex = addUsedOption(aCurrentUsedOptionKey);
+      } else {
+        Message::SendFail()
+            << "Error: unknown argument '" << anOptionName << "'";
         return;
       }
-    }
-    else
-    {
-      if (anIter == 1)
-      {
-        aCurrentUsedOptionIndex = addUsedOption (THE_UNNAMED_COMMAND_OPTION_KEY);
+    } else {
+      if (anIter == 1) {
+        aCurrentUsedOptionIndex = addUsedOption(THE_UNNAMED_COMMAND_OPTION_KEY);
       }
-      myOptionArgumentStorage[aCurrentUsedOptionIndex].push_back (anArgument);
+      myOptionArgumentStorage[aCurrentUsedOptionIndex].push_back(anArgument);
     }
   }
 }
@@ -177,10 +168,9 @@ void ViewerTest_CmdParser::Parse (const Standard_Integer theArgsNb, const char*
 // function : GetOptionNameByKey
 // purpose  :
 //===============================================================================================
-std::string ViewerTest_CmdParser::GetOptionNameByKey (const ViewerTest_CommandOptionKey theOptionKey) const
-{
-  if (theOptionKey == THE_UNNAMED_COMMAND_OPTION_KEY)
-  {
+std::string ViewerTest_CmdParser::GetOptionNameByKey(
+    const ViewerTest_CommandOptionKey theOptionKey) const {
+  if (theOptionKey == THE_UNNAMED_COMMAND_OPTION_KEY) {
     return "Unnamed";
   }
   return myOptionStorage[theOptionKey].Name;
@@ -190,14 +180,13 @@ std::string ViewerTest_CmdParser::GetOptionNameByKey (const ViewerTest_CommandOp
 // function : GetUsedOptions
 // purpose  :
 //===============================================================================================
-ViewerTest_CommandOptionKeySet ViewerTest_CmdParser::GetUsedOptions() const
-{
+ViewerTest_CommandOptionKeySet ViewerTest_CmdParser::GetUsedOptions() const {
   ViewerTest_CommandOptionKeySet aUsedOptions;
-  for (UsedOptionMap::const_iterator aUsedOptionMapIterator = myUsedOptionMap.begin();
+  for (UsedOptionMap::const_iterator aUsedOptionMapIterator =
+           myUsedOptionMap.begin();
        aUsedOptionMapIterator != myUsedOptionMap.end();
-       ++aUsedOptionMapIterator)
-  {
-    aUsedOptions.insert (aUsedOptionMapIterator->first);
+       ++aUsedOptionMapIterator) {
+    aUsedOptions.insert(aUsedOptionMapIterator->first);
   }
   return aUsedOptions;
 }
@@ -206,8 +195,7 @@ ViewerTest_CommandOptionKeySet ViewerTest_CmdParser::GetUsedOptions() const
 // function : HasNoOption
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::HasNoOption() const
-{
+bool ViewerTest_CmdParser::HasNoOption() const {
   return myUsedOptionMap.empty();
 }
 
@@ -215,17 +203,16 @@ bool ViewerTest_CmdParser::HasNoOption() const
 // function : HasUnnamedOption
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::HasUnnamedOption() const
-{
-  return myUsedOptionMap.find (THE_UNNAMED_COMMAND_OPTION_KEY) != myUsedOptionMap.end();
+bool ViewerTest_CmdParser::HasUnnamedOption() const {
+  return myUsedOptionMap.find(THE_UNNAMED_COMMAND_OPTION_KEY) !=
+         myUsedOptionMap.end();
 }
 
 //===============================================================================================
 // function : HasNoUnnamedOption
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::HasOnlyUnnamedOption() const
-{
+bool ViewerTest_CmdParser::HasOnlyUnnamedOption() const {
   return HasUnnamedOption() && (myUsedOptionMap.size() == 1);
 }
 
@@ -233,37 +220,37 @@ bool ViewerTest_CmdParser::HasOnlyUnnamedOption() const
 // function : HasOption
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::HasOption (const std::string& theOptionName,
-                                      const std::size_t  theMandatoryArgsNb /* = 0 */,
-                                      const bool         isFatal /* = false */) const
-{
+bool ViewerTest_CmdParser::HasOption(
+    const std::string &theOptionName,
+    const std::size_t theMandatoryArgsNb /* = 0 */,
+    const bool isFatal /* = false */) const {
   ViewerTest_CommandOptionKey anOptionKey;
-  if (!findOptionKey (theOptionName, anOptionKey))
-  {
+  if (!findOptionKey(theOptionName, anOptionKey)) {
     return false;
   }
-  return HasOption (anOptionKey, theMandatoryArgsNb, isFatal);
+  return HasOption(anOptionKey, theMandatoryArgsNb, isFatal);
 }
 
 //===============================================================================================
 // function : HasOption
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::HasOption (const ViewerTest_CommandOptionKey theOptionKey,
-                                      const std::size_t                 theMandatoryArgsNb /* = 0 */,
-                                      const bool                        isFatal /* = false */) const
-{
+bool ViewerTest_CmdParser::HasOption(
+    const ViewerTest_CommandOptionKey theOptionKey,
+    const std::size_t theMandatoryArgsNb /* = 0 */,
+    const bool isFatal /* = false */) const {
   std::size_t aUsedOptionIndex = 0;
-  if (!findUsedOptionIndex (theOptionKey, aUsedOptionIndex))
-  {
+  if (!findUsedOptionIndex(theOptionKey, aUsedOptionIndex)) {
     return false;
   }
-  const OptionArguments& anOptionArguments = myOptionArgumentStorage[aUsedOptionIndex];
-  const bool             aResult           = (anOptionArguments.size() >= theMandatoryArgsNb);
-  if (isFatal && !aResult)
-  {
-    Message::SendFail() << "Error: wrong syntax at option '" << myOptionStorage[theOptionKey].Name << "'\n"
-                        << "At least " << theMandatoryArgsNb << "expected, but only " << anOptionArguments.size()
+  const OptionArguments &anOptionArguments =
+      myOptionArgumentStorage[aUsedOptionIndex];
+  const bool aResult = (anOptionArguments.size() >= theMandatoryArgsNb);
+  if (isFatal && !aResult) {
+    Message::SendFail() << "Error: wrong syntax at option '"
+                        << myOptionStorage[theOptionKey].Name << "'\n"
+                        << "At least " << theMandatoryArgsNb
+                        << "expected, but only " << anOptionArguments.size()
                         << "provided.";
   }
   return aResult;
@@ -273,68 +260,63 @@ bool ViewerTest_CmdParser::HasOption (const ViewerTest_CommandOptionKey theOptio
 // function : GetNumberOfOptionArguments
 // purpose  :
 //===============================================================================================
-Standard_Integer ViewerTest_CmdParser::GetNumberOfOptionArguments (const std::string& theOptionName) const
-{
+Standard_Integer ViewerTest_CmdParser::GetNumberOfOptionArguments(
+    const std::string &theOptionName) const {
   ViewerTest_CommandOptionKey anOptionKey = THE_UNNAMED_COMMAND_OPTION_KEY;
-  if (!findOptionKey (theOptionName, anOptionKey))
-  {
+  if (!findOptionKey(theOptionName, anOptionKey)) {
     return 0;
   }
-  return GetNumberOfOptionArguments (anOptionKey);
+  return GetNumberOfOptionArguments(anOptionKey);
 }
 
 //===============================================================================================
 // function : GetNumberOfOptionArguments
 // purpose  :
 //===============================================================================================
-Standard_Integer ViewerTest_CmdParser::GetNumberOfOptionArguments (const ViewerTest_CommandOptionKey theOptionKey) const
-{
+Standard_Integer ViewerTest_CmdParser::GetNumberOfOptionArguments(
+    const ViewerTest_CommandOptionKey theOptionKey) const {
   std::size_t aUsedOptionIndex = 0;
-  if (!findUsedOptionIndex (theOptionKey, aUsedOptionIndex))
-  {
+  if (!findUsedOptionIndex(theOptionKey, aUsedOptionIndex)) {
     return false;
   }
-  return static_cast<Standard_Integer> (myOptionArgumentStorage[aUsedOptionIndex].size());
+  return static_cast<Standard_Integer>(
+      myOptionArgumentStorage[aUsedOptionIndex].size());
 }
 
 //===============================================================================================
 // function : Arg
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::Arg (const std::string&     theOptionName,
-                                const Standard_Integer theArgumentIndex,
-                                std::string&           theOptionArgument) const
-{
-  Standard_ASSERT_RETURN (theArgumentIndex >= 0,
-                          "'theArgumentIndex' must be greater than or equal to zero.",
-                          false);
+bool ViewerTest_CmdParser::Arg(const std::string &theOptionName,
+                               const Standard_Integer theArgumentIndex,
+                               std::string &theOptionArgument) const {
+  Standard_ASSERT_RETURN(
+      theArgumentIndex >= 0,
+      "'theArgumentIndex' must be greater than or equal to zero.", false);
   ViewerTest_CommandOptionKey anOptionKey = THE_UNNAMED_COMMAND_OPTION_KEY;
-  if (!theOptionName.empty() && !findOptionKey (theOptionName, anOptionKey))
-  {
+  if (!theOptionName.empty() && !findOptionKey(theOptionName, anOptionKey)) {
     return false;
   }
-  return Arg (anOptionKey, theArgumentIndex, theOptionArgument);
+  return Arg(anOptionKey, theArgumentIndex, theOptionArgument);
 }
 
 //===============================================================================================
 // function : Arg
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::Arg (const ViewerTest_CommandOptionKey theOptionKey,
-                                const Standard_Integer            theArgumentIndex,
-                                std::string&                      theOptionArgument) const
-{
-  Standard_ASSERT_RETURN (theArgumentIndex >= 0,
-                          "'theArgumentIndex' must be greater than or equal to zero.",
-                          false);
+bool ViewerTest_CmdParser::Arg(const ViewerTest_CommandOptionKey theOptionKey,
+                               const Standard_Integer theArgumentIndex,
+                               std::string &theOptionArgument) const {
+  Standard_ASSERT_RETURN(
+      theArgumentIndex >= 0,
+      "'theArgumentIndex' must be greater than or equal to zero.", false);
   std::size_t aUsedOptionIndex = 0;
-  if (!findUsedOptionIndex (theOptionKey, aUsedOptionIndex))
-  {
+  if (!findUsedOptionIndex(theOptionKey, aUsedOptionIndex)) {
     return false;
   }
-  const OptionArguments& anOptionArguments = myOptionArgumentStorage[aUsedOptionIndex];
-  if (static_cast<std::size_t> (theArgumentIndex) >= anOptionArguments.size())
-  {
+  const OptionArguments &anOptionArguments =
+      myOptionArgumentStorage[aUsedOptionIndex];
+  if (static_cast<std::size_t>(theArgumentIndex) >= anOptionArguments.size()) {
     return false;
   }
   theOptionArgument = anOptionArguments[theArgumentIndex];
@@ -345,14 +327,15 @@ bool ViewerTest_CmdParser::Arg (const ViewerTest_CommandOptionKey theOptionKey,
 // function : Arg
 // purpose  :
 //===============================================================================================
-std::string ViewerTest_CmdParser::Arg (const std::string& theOptionName, const Standard_Integer theArgumentIndex) const
-{
-  Standard_ASSERT_RETURN (theArgumentIndex >= 0,
-                          "'theArgumentIndex' must be greater than or equal to zero.",
-                          std::string());
+std::string
+ViewerTest_CmdParser::Arg(const std::string &theOptionName,
+                          const Standard_Integer theArgumentIndex) const {
+  Standard_ASSERT_RETURN(
+      theArgumentIndex >= 0,
+      "'theArgumentIndex' must be greater than or equal to zero.",
+      std::string());
   std::string anOptionArgument;
-  if (!Arg (theOptionName, theArgumentIndex, anOptionArgument))
-  {
+  if (!Arg(theOptionName, theArgumentIndex, anOptionArgument)) {
     return std::string();
   }
   return anOptionArgument;
@@ -362,12 +345,11 @@ std::string ViewerTest_CmdParser::Arg (const std::string& theOptionName, const S
 // function : Arg
 // purpose  :
 //===============================================================================================
-std::string ViewerTest_CmdParser::Arg (const ViewerTest_CommandOptionKey theOptionKey,
-                                       const Standard_Integer            theArgumentIndex) const
-{
+std::string
+ViewerTest_CmdParser::Arg(const ViewerTest_CommandOptionKey theOptionKey,
+                          const Standard_Integer theArgumentIndex) const {
   std::string anOptionArgument;
-  if (!Arg (theOptionKey, theArgumentIndex, anOptionArgument))
-  {
+  if (!Arg(theOptionKey, theArgumentIndex, anOptionArgument)) {
     return std::string();
   }
   return anOptionArgument;
@@ -377,86 +359,92 @@ std::string ViewerTest_CmdParser::Arg (const ViewerTest_CommandOptionKey theOpti
 // function : ArgVec3f
 // purpose  :
 //===============================================================================================
-Graphic3d_Vec3 ViewerTest_CmdParser::ArgVec3f (const std::string& theOptionName,
-                                               Standard_Integer   theArgumentIndex) const
-{
-  return Graphic3d_Vec3 (
-    static_cast<Standard_ShortReal> (Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str())),
-    static_cast<Standard_ShortReal> (Draw::Atof (Arg (theOptionName, theArgumentIndex + 1).c_str())),
-    static_cast<Standard_ShortReal> (Draw::Atof (Arg (theOptionName, theArgumentIndex + 2).c_str())));
+Graphic3d_Vec3
+ViewerTest_CmdParser::ArgVec3f(const std::string &theOptionName,
+                               Standard_Integer theArgumentIndex) const {
+  return Graphic3d_Vec3(static_cast<Standard_ShortReal>(Draw::Atof(
+                            Arg(theOptionName, theArgumentIndex).c_str())),
+                        static_cast<Standard_ShortReal>(Draw::Atof(
+                            Arg(theOptionName, theArgumentIndex + 1).c_str())),
+                        static_cast<Standard_ShortReal>(Draw::Atof(
+                            Arg(theOptionName, theArgumentIndex + 2).c_str())));
 }
 
 //===============================================================================================
 // function : ArgVec3d
 // purpose  :
 //===============================================================================================
-Graphic3d_Vec3d ViewerTest_CmdParser::ArgVec3d (const std::string& theOptionName,
-                                                Standard_Integer   theArgumentIndex) const
-{
-  return Graphic3d_Vec3d (Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str()),
-                          Draw::Atof (Arg (theOptionName, theArgumentIndex + 1).c_str()),
-                          Draw::Atof (Arg (theOptionName, theArgumentIndex + 2).c_str()));
+Graphic3d_Vec3d
+ViewerTest_CmdParser::ArgVec3d(const std::string &theOptionName,
+                               Standard_Integer theArgumentIndex) const {
+  return Graphic3d_Vec3d(
+      Draw::Atof(Arg(theOptionName, theArgumentIndex).c_str()),
+      Draw::Atof(Arg(theOptionName, theArgumentIndex + 1).c_str()),
+      Draw::Atof(Arg(theOptionName, theArgumentIndex + 2).c_str()));
 }
 
 //===============================================================================================
 // function : ArgVec
 // purpose  :
 //===============================================================================================
-gp_Vec ViewerTest_CmdParser::ArgVec (const std::string& theOptionName, Standard_Integer theArgumentIndex) const
-{
-  return gp_Vec (Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str()),
-                 Draw::Atof (Arg (theOptionName, theArgumentIndex + 1).c_str()),
-                 Draw::Atof (Arg (theOptionName, theArgumentIndex + 2).c_str()));
+gp_Vec ViewerTest_CmdParser::ArgVec(const std::string &theOptionName,
+                                    Standard_Integer theArgumentIndex) const {
+  return gp_Vec(Draw::Atof(Arg(theOptionName, theArgumentIndex).c_str()),
+                Draw::Atof(Arg(theOptionName, theArgumentIndex + 1).c_str()),
+                Draw::Atof(Arg(theOptionName, theArgumentIndex + 2).c_str()));
 }
 
 //===============================================================================================
 // function : ArgPnt
 // purpose  :
 //===============================================================================================
-gp_Pnt ViewerTest_CmdParser::ArgPnt (const std::string& theOptionName, Standard_Integer theArgumentIndex) const
-{
-  return gp_Pnt (Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str()),
-                 Draw::Atof (Arg (theOptionName, theArgumentIndex + 1).c_str()),
-                 Draw::Atof (Arg (theOptionName, theArgumentIndex + 2).c_str()));
+gp_Pnt ViewerTest_CmdParser::ArgPnt(const std::string &theOptionName,
+                                    Standard_Integer theArgumentIndex) const {
+  return gp_Pnt(Draw::Atof(Arg(theOptionName, theArgumentIndex).c_str()),
+                Draw::Atof(Arg(theOptionName, theArgumentIndex + 1).c_str()),
+                Draw::Atof(Arg(theOptionName, theArgumentIndex + 2).c_str()));
 }
 
 //===============================================================================================
 // function : ArgDouble
 // purpose  :
 //===============================================================================================
-Standard_Real ViewerTest_CmdParser::ArgDouble (const std::string& theOptionName,
-                                               Standard_Integer   theArgumentIndex) const
-{
-  return Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str());
+Standard_Real
+ViewerTest_CmdParser::ArgDouble(const std::string &theOptionName,
+                                Standard_Integer theArgumentIndex) const {
+  return Draw::Atof(Arg(theOptionName, theArgumentIndex).c_str());
 }
 
 //===============================================================================================
 // function : ArgFloat
 // purpose  :
 //===============================================================================================
-Standard_ShortReal ViewerTest_CmdParser::ArgFloat (const std::string& theOptionName,
-                                                   Standard_Integer   theArgumentIndex) const
-{
-  return static_cast<Standard_ShortReal> (Draw::Atof (Arg (theOptionName, theArgumentIndex).c_str()));
+Standard_ShortReal
+ViewerTest_CmdParser::ArgFloat(const std::string &theOptionName,
+                               Standard_Integer theArgumentIndex) const {
+  return static_cast<Standard_ShortReal>(
+      Draw::Atof(Arg(theOptionName, theArgumentIndex).c_str()));
 }
 
 //===============================================================================================
 // function : ArgInt
 // purpose  :
 //===============================================================================================
-Standard_Integer ViewerTest_CmdParser::ArgInt (const std::string&     theOptionName,
-                                               const Standard_Integer theArgumentIndex) const
-{
-  return static_cast<Standard_Integer> (Draw::Atoi (Arg (theOptionName, theArgumentIndex).c_str()));
+Standard_Integer
+ViewerTest_CmdParser::ArgInt(const std::string &theOptionName,
+                             const Standard_Integer theArgumentIndex) const {
+  return static_cast<Standard_Integer>(
+      Draw::Atoi(Arg(theOptionName, theArgumentIndex).c_str()));
 }
 
 //===============================================================================================
 // function : ArgBool
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::ArgBool (const std::string& theOptionName, const Standard_Integer theArgumentIndex) const
-{
-  return Draw::Atoi (Arg (theOptionName, theArgumentIndex).c_str()) != 0;
+bool ViewerTest_CmdParser::ArgBool(
+    const std::string &theOptionName,
+    const Standard_Integer theArgumentIndex) const {
+  return Draw::Atoi(Arg(theOptionName, theArgumentIndex).c_str()) != 0;
 }
 
 //===============================================================================================
@@ -464,54 +452,55 @@ bool ViewerTest_CmdParser::ArgBool (const std::string& theOptionName, const Stan
 // purpose  :
 //===============================================================================================
 template <typename TheColor>
-bool ViewerTest_CmdParser::ArgColor (const std::string& theOptionName,
-                                     Standard_Integer&  theArgumentIndex,
-                                     TheColor&          theColor) const
-{
+bool ViewerTest_CmdParser::ArgColor(const std::string &theOptionName,
+                                    Standard_Integer &theArgumentIndex,
+                                    TheColor &theColor) const {
   ViewerTest_CommandOptionKey anOptionKey;
-  if (!findOptionKey (theOptionName, anOptionKey))
-  {
+  if (!findOptionKey(theOptionName, anOptionKey)) {
     return false;
   }
-  return ArgColor (anOptionKey, theArgumentIndex, theColor);
+  return ArgColor(anOptionKey, theArgumentIndex, theColor);
 }
 
 //! ViewerTest_CmdParser::ArgColor() explicit template instantiation definitions
-template bool ViewerTest_CmdParser::ArgColor (const std::string& theOptionName,
-                                              Standard_Integer&  theArgumentIndex,
-                                              Quantity_Color&    theColor) const;
+template bool ViewerTest_CmdParser::ArgColor(const std::string &theOptionName,
+                                             Standard_Integer &theArgumentIndex,
+                                             Quantity_Color &theColor) const;
 
-template bool ViewerTest_CmdParser::ArgColor (const std::string&  theOptionName,
-                                              Standard_Integer&   theArgumentIndex,
-                                              Quantity_ColorRGBA& theColor) const;
+template bool
+ViewerTest_CmdParser::ArgColor(const std::string &theOptionName,
+                               Standard_Integer &theArgumentIndex,
+                               Quantity_ColorRGBA &theColor) const;
 
 //===============================================================================================
 // function : ArgColor
 // purpose  :
 //===============================================================================================
 template <typename TheColor>
-bool ViewerTest_CmdParser::ArgColor (const ViewerTest_CommandOptionKey theOptionKey,
-                                     Standard_Integer&                 theArgumentIndex,
-                                     TheColor&                         theColor) const
-{
+bool ViewerTest_CmdParser::ArgColor(
+    const ViewerTest_CommandOptionKey theOptionKey,
+    Standard_Integer &theArgumentIndex, TheColor &theColor) const {
   std::size_t aUsedOptionIndex = 0;
-  if (!findUsedOptionIndex (theOptionKey, aUsedOptionIndex))
-  {
+  if (!findUsedOptionIndex(theOptionKey, aUsedOptionIndex)) {
     return false;
   }
-  const RawStringArguments aRawStringArguments = getRawStringArguments (aUsedOptionIndex);
-  const Standard_Integer   aNumberOfArguments  = static_cast<Standard_Integer> (aRawStringArguments.size());
-  Standard_ASSERT_RETURN (theArgumentIndex < aNumberOfArguments,
-                          "'theArgumentIndex' must be less than the number of command-line arguments "
-                          "passed with the option which access key is 'theOptionKey'.",
-                          false);
-  const Standard_Integer aNumberOfAvailableArguments = aNumberOfArguments - theArgumentIndex;
-  TheColor               aColor;
-  const Standard_Integer aNumberOfParsedArguments = Draw::ParseColor (aNumberOfAvailableArguments,
-                                                                      &aRawStringArguments[theArgumentIndex],
-                                                                      aColor);
-  if (aNumberOfParsedArguments == 0)
-  {
+  const RawStringArguments aRawStringArguments =
+      getRawStringArguments(aUsedOptionIndex);
+  const Standard_Integer aNumberOfArguments =
+      static_cast<Standard_Integer>(aRawStringArguments.size());
+  Standard_ASSERT_RETURN(
+      theArgumentIndex < aNumberOfArguments,
+      "'theArgumentIndex' must be less than the number of command-line "
+      "arguments "
+      "passed with the option which access key is 'theOptionKey'.",
+      false);
+  const Standard_Integer aNumberOfAvailableArguments =
+      aNumberOfArguments - theArgumentIndex;
+  TheColor aColor;
+  const Standard_Integer aNumberOfParsedArguments =
+      Draw::ParseColor(aNumberOfAvailableArguments,
+                       &aRawStringArguments[theArgumentIndex], aColor);
+  if (aNumberOfParsedArguments == 0) {
     return false;
   }
   theArgumentIndex += aNumberOfParsedArguments;
@@ -520,25 +509,27 @@ bool ViewerTest_CmdParser::ArgColor (const ViewerTest_CommandOptionKey theOption
 }
 
 //! ViewerTest_CmdParser::ArgColor() explicit template instantiation definitions
-template bool ViewerTest_CmdParser::ArgColor (ViewerTest_CommandOptionKey theOptionKey,
-                                              Standard_Integer&           theArgumentIndex,
-                                              Quantity_Color&             theColor) const;
+template bool
+ViewerTest_CmdParser::ArgColor(ViewerTest_CommandOptionKey theOptionKey,
+                               Standard_Integer &theArgumentIndex,
+                               Quantity_Color &theColor) const;
 
-template bool ViewerTest_CmdParser::ArgColor (ViewerTest_CommandOptionKey theOptionKey,
-                                              Standard_Integer&           theArgumentIndex,
-                                              Quantity_ColorRGBA&         theColor) const;
+template bool
+ViewerTest_CmdParser::ArgColor(ViewerTest_CommandOptionKey theOptionKey,
+                               Standard_Integer &theArgumentIndex,
+                               Quantity_ColorRGBA &theColor) const;
 
 //===============================================================================================
 // function : findUsedOptionKey
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::findOptionKey (const std::string&           theOptionName,
-                                          ViewerTest_CommandOptionKey& theOptionKey) const
-{
-  const std::string               anOptionNameInLowercase = toLowerCase (theOptionName);
-  const OptionMap::const_iterator aMapIter                = myOptionMap.find (anOptionNameInLowercase);
-  if (aMapIter == myOptionMap.end())
-  {
+bool ViewerTest_CmdParser::findOptionKey(
+    const std::string &theOptionName,
+    ViewerTest_CommandOptionKey &theOptionKey) const {
+  const std::string anOptionNameInLowercase = toLowerCase(theOptionName);
+  const OptionMap::const_iterator aMapIter =
+      myOptionMap.find(anOptionNameInLowercase);
+  if (aMapIter == myOptionMap.end()) {
     return false;
   }
   theOptionKey = aMapIter->second;
@@ -549,12 +540,12 @@ bool ViewerTest_CmdParser::findOptionKey (const std::string&           theOption
 // function : findUsedOptionKey
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::findUsedOptionIndex (const ViewerTest_CommandOptionKey theOptionKey,
-                                                std::size_t&                      theUsedOptionIndex) const
-{
-  const UsedOptionMap::const_iterator aUsedOptionIterator = myUsedOptionMap.find (theOptionKey);
-  if (aUsedOptionIterator == myUsedOptionMap.end())
-  {
+bool ViewerTest_CmdParser::findUsedOptionIndex(
+    const ViewerTest_CommandOptionKey theOptionKey,
+    std::size_t &theUsedOptionIndex) const {
+  const UsedOptionMap::const_iterator aUsedOptionIterator =
+      myUsedOptionMap.find(theOptionKey);
+  if (aUsedOptionIterator == myUsedOptionMap.end()) {
     return false;
   }
   theUsedOptionIndex = aUsedOptionIterator->second;
@@ -565,16 +556,14 @@ bool ViewerTest_CmdParser::findUsedOptionIndex (const ViewerTest_CommandOptionKe
 // function : findUsedOptionIndex
 // purpose  :
 //===============================================================================================
-bool ViewerTest_CmdParser::findUsedOptionIndex (const std::string& theOptionName, std::size_t& theUsedOptionIndex) const
-{
+bool ViewerTest_CmdParser::findUsedOptionIndex(
+    const std::string &theOptionName, std::size_t &theUsedOptionIndex) const {
   ViewerTest_CommandOptionKey anOptionKey = THE_UNNAMED_COMMAND_OPTION_KEY;
-  if (!findOptionKey (theOptionName, anOptionKey))
-  {
+  if (!findOptionKey(theOptionName, anOptionKey)) {
     return false;
   }
   std::size_t aUsedOptionIndex = 0;
-  if (!findUsedOptionIndex (anOptionKey, aUsedOptionIndex))
-  {
+  if (!findUsedOptionIndex(anOptionKey, aUsedOptionIndex)) {
     return false;
   }
   theUsedOptionIndex = aUsedOptionIndex;
@@ -585,10 +574,10 @@ bool ViewerTest_CmdParser::findUsedOptionIndex (const std::string& theOptionName
 // function : addUsedOption
 // purpose  :
 //===============================================================================================
-std::size_t ViewerTest_CmdParser::addUsedOption (const ViewerTest_CommandOptionKey theNewUsedOptionKey)
-{
+std::size_t ViewerTest_CmdParser::addUsedOption(
+    const ViewerTest_CommandOptionKey theNewUsedOptionKey) {
   const std::size_t aNewUsedOptionIndex = myOptionArgumentStorage.size();
-  myOptionArgumentStorage.push_back (OptionArguments());
+  myOptionArgumentStorage.push_back(OptionArguments());
   myUsedOptionMap[theNewUsedOptionKey] = aNewUsedOptionIndex;
   return aNewUsedOptionIndex;
 }
@@ -597,13 +586,14 @@ std::size_t ViewerTest_CmdParser::addUsedOption (const ViewerTest_CommandOptionK
 // function : getRawStringArguments
 // purpose  :
 //===============================================================================================
-ViewerTest_CmdParser::RawStringArguments ViewerTest_CmdParser::getRawStringArguments (
-  const std::size_t theUsedOptionIndex) const
-{
-  Standard_ASSERT_RETURN (
-    theUsedOptionIndex < myOptionArgumentStorage.size(),
-    "'theUsedOptionIndex' must be less than the size of 'myOptionArgumentStorage'.",
-    RawStringArguments());
-  const OptionArguments& anOptionArguments = myOptionArgumentStorage[theUsedOptionIndex];
-  return convertToRawStringList (anOptionArguments);
+ViewerTest_CmdParser::RawStringArguments
+ViewerTest_CmdParser::getRawStringArguments(
+    const std::size_t theUsedOptionIndex) const {
+  Standard_ASSERT_RETURN(theUsedOptionIndex < myOptionArgumentStorage.size(),
+                         "'theUsedOptionIndex' must be less than the size of "
+                         "'myOptionArgumentStorage'.",
+                         RawStringArguments());
+  const OptionArguments &anOptionArguments =
+      myOptionArgumentStorage[theUsedOptionIndex];
+  return convertToRawStringList(anOptionArguments);
 }