remove useless code

Author: haozhihan

source/module_psi/psi.cpp

@@ -193,25 +193,6 @@ Psi<T, Device>::Psi(const Psi& psi_in, const int nk_in, const int nband_in)
     }
 }
 
-// template <typename T, typename Device>
-// Psi<T, Device>::Psi(T* psi_pointer, const Psi& psi_in, const int nk_in, int nband_in)
-// {
-//     this->k_first = psi_in.get_k_first();
-//     assert(nk_in <= psi_in.get_nk());
-//     if (nband_in == 0)
-//     {
-//         nband_in = psi_in.get_nbands();
-//     }
-//     this->ngk = psi_in.ngk;
-//     this->npol = psi_in.npol;
-//     this->nk = nk_in;
-//     this->nbands = nband_in;
-//     this->nbasis = psi_in.nbasis;
-//     this->psi_current = psi_pointer;
-//     this->allocate_inside = false;
-//     this->psi = psi_pointer;
-// }
-
 template <typename T, typename Device>
 Psi<T, Device>::Psi(const Psi& psi_in)
 {

source/module_psi/psi.h

@@ -36,45 +36,43 @@ template <typename T, typename Device = base_device::DEVICE_CPU>
 class Psi
 {
   public:
-    // Constructor 1: basic
+    // Constructor 0: basic
     Psi();
 
-    // Constructor 3: specify nk, nbands, nbasis, ngk, and do not need to call resize() later
+    // Constructor 1-1: specify nk, nbands, nbasis, ngk, and do not need to call resize() later
     Psi(const int nk_in, const int nbd_in, const int nbs_in, const int* ngk_in, const bool k_first_in = true);
 
+    // Constructor 1-2: 
     Psi(const int nk_in, const int nbd_in, const int nbs_in, const std::vector<int>& ngk_in, const bool k_first_in);
 
-    // Constructor 4: copy a new Psi which have several k-points and several bands from inputted psi_in
-    Psi(const Psi& psi_in, const int nk_in, const int nband_in);
-
-    // // Constructor 5: a wrapper of a data pointer, used for Operator::hPsi()
-    // // in this case, fix_k can not be used
-    // Psi(T* psi_pointer, const Psi& psi_in, const int nk_in, int nband_in);
-
-    // Constructor 6: initialize a new psi from the given psi_in
+    // Constructor 2-1: initialize a new psi from the given psi_in
     Psi(const Psi& psi_in);
 
-    // Constructor 7: initialize a new psi from the given psi_in with a different class template
+    // Constructor 2-2: initialize a new psi from the given psi_in with a different class template
     // in this case, psi_in may have a different device type.
     template <typename T_in, typename Device_in = Device>
     Psi(const Psi<T_in, Device_in>& psi_in);
 
-    // Constructor 8-1: a pointer version of constructor 3
-    // only used in hsolver-pw function pointer.
+    // Constructor 3-1: 2D Psi version
+    // used in hsolver-pw function pointer and somewhere.
     Psi(T* psi_pointer,
         const int nk_in,
         const int nbd_in,
         const int nbs_in,
         const int current_nbasis_in,
         const bool k_first_in = true);
 
-    // Constructor 8-3: 2D Psi version 3
+    // Constructor 3-2: 2D Psi version
     Psi(const int nk_in, 
         const int nbd_in, 
         const int nbs_in, 
         const int current_nbasis_in, 
         const bool k_first_in);
 
+
+    // Constructor 4: copy a new Psi which have several k-points and several bands from inputted psi_in
+    Psi(const Psi& psi_in, const int nk_in, const int nband_in);
+
 
     // Destructor for deleting the psi array manually
     ~Psi();