Add explicit to the Search constructor; fix the comments.

1 files changed, 13 insertions, 11 deletions

diff --git a/moses/Search.h b/moses/Search.h
index d961d17cd..ed1775fe4 100644
--- a/moses/Search.h
+++ b/moses/Search.h
@@ -3,7 +3,6 @@
 
 #include <vector>
 #include "TypeDef.h"
-#include "Phrase.h"
 
 namespace Moses
 {
@@ -13,31 +12,34 @@ class Hypothesis;
 class InputType;
 class TranslationOptionCollection;
 class Manager;
+class Phrase;
 
-/** Abstract class used in the phrase-based decoder. 
- *  Cube pruning and normal searches are the classes that inherits from this class
+/** Base search class used in the phrase-based decoder.
+ *
+ * Actual search class that implement the cube pruning algorithm (SearchCubePruning)
+ * or standard beam search (SearchNormal) should inherits from this class, and
+ * override pure virtual functions.
  */
 class Search
 {
 public:
-  virtual const std::vector < HypothesisStack* >& GetHypothesisStacks() const = 0;
+  virtual const std::vector<HypothesisStack*>& GetHypothesisStacks() const = 0;
   virtual const Hypothesis *GetBestHypothesis() const = 0;
+
+  //! Decode the sentence according to the specified search algorithm.
   virtual void ProcessSentence() = 0;
-  Search(Manager& manager) : m_manager(manager) {}
-  virtual ~Search()
-  {}
 
-  // Factory
+  explicit Search(Manager& manager) : m_manager(manager) {}
+  virtual ~Search() {}
+
+  // Factory method
   static Search *CreateSearch(Manager& manager, const InputType &source, SearchAlgorithm searchAlgorithm,
                               const TranslationOptionCollection &transOptColl);
 
 protected:
-
   const Phrase *m_constraint;
   Manager& m_manager;
-
 };
 
-
 }
 #endif