Revised accoring to the coments by Nori

MathLib/LinAlg/PETSc/PETScLinearSolver.h

@@ -50,11 +50,19 @@ class PETScLinearSolver
         */
         PETScLinearSolver(PETScMatrix &A, const boost::property_tree::ptree &option);
 
+        /*!
+            Set Krylov subspace parameters with given option
+            \param  ksp_opt Given parameters 
+        */ 
         template <typename T_KSP_OPTION> void setKSP_Option(T_KSP_OPTION &ksp_opt)
         {
             ksp_opt.setOption(_solver);
         }
-
+        
+        /*!
+            Set preconditioner parameters with given option
+            \param  pc_opt Given parameters 
+        */ 
         template <typename T_PC_OPTION> void setPC_Option(T_PC_OPTION &pc_opt)
         {
             pc_opt.setOption(_pc);

MathLib/LinAlg/PETSc/PETScLinearSolverOption.h

@@ -41,7 +41,7 @@ class PETScLinearSolverOption
         /*!
             \param ksp_option ptree of basic configuration data for solver
         */
-        PETScLinearSolverOption(const boost::property_tree::ptree &ksp_option);
+        explicit PETScLinearSolverOption(const boost::property_tree::ptree &ksp_option);
 
         /*!
             \param ksp_option ptree of basic configuration data for solver

MathLib/LinAlg/PETSc/PETScTools.h

@@ -25,10 +25,8 @@
 namespace MathLib
 {
 /*!
-  \brief apply known solutions to a system of linear equations
-
-  This function introduces the constants into the system by the penalty method.
-
+   \brief apply known solutions to a system of linear equations
+  
    \param A                 Coefficient matrix
    \param b                 RHS vector
    \param vec_knownX_id    a vector of known solution entry IDs