Changeset 1353

Timestamp:

04/26/07 14:55:17 (17 years ago)

Author:

mar637

Message:

added explicit keyword for single arument constructors

Location:

trunk/src

Files:

• LineCatalog.h (modified) (1 diff)
• Logger.h (modified) (1 diff)
• RowAccumulator.h (modified) (1 diff)
• STFiller.h (modified) (2 diffs)
• STFillerWrapper.h (modified) (2 diffs)
• STFit.h (modified) (1 diff)
• STFocus.h (modified) (1 diff)
• STFrequencies.h (modified) (1 diff)
• STHistory.h (modified) (1 diff)
• STLineFinder.h (modified) (3 diffs)
• STMath.h (modified) (1 diff)
• STMathWrapper.h (modified) (1 diff)
• STMolecules.h (modified) (1 diff)
• STPolCircular.h (modified) (1 diff)
• STPolLinear.h (modified) (1 diff)
• STPolStokes.h (modified) (1 diff)
• STTcal.h (modified) (1 diff)
• STWeather.h (modified) (1 diff)
• STWriter.h (modified) (1 diff)
• STWriterWrapper.h (modified) (1 diff)

trunk/src/LineCatalog.h

@@ -35 +35 @@
     * @param name the name of the ASCII file or aips++ table
     */
-  LineCatalog(const std::string& name = "jpl");
+  explicit LineCatalog(const std::string& name = "jpl");
 
   virtual ~LineCatalog() {}

trunk/src/Logger.h

@@ -36 +36 @@
    * @param[in] enabled indicating the deafult state
    */  
-  Logger(bool enabled);
+  explicit Logger(bool enabled);
 
   /*

trunk/src/RowAccumulator.h

@@ -33 +33 @@
    * Constructor taking a weight type as defined in @ref STDefs
    */
-  RowAccumulator(WeightType wt = asap::NONE);
+  explicit RowAccumulator(WeightType wt = asap::NONE);
 
  ~RowAccumulator();

trunk/src/STFiller.h

@@ -51 +51 @@
    * @param stbl
    */
-  STFiller(casa::CountedPtr< Scantable > stbl);
+  explicit STFiller(casa::CountedPtr< Scantable > stbl);
 
 
@@ -60 +60 @@
     * @param whichBeam read a specific beam only (default -1 means all beams)
     */
-  STFiller( const std::string& filename, int whichIF=-1,
+  explicit STFiller( const std::string& filename, int whichIF=-1,
                   int whichBeam=-1 );
 

trunk/src/STFillerWrapper.h

@@ -27 +27 @@
 //#                        AUSTRALIA
 //#
-//# $Id:$
+//# $Id$
 //#---------------------------------------------------------------------------
 #ifndef STFILLERWRAPPER_H
@@ -42 +42 @@
 public:
   STFillerWrapper() {;}
-  STFillerWrapper(const std::string& filename, int ifSel=-1, int beamSel=-1) :
+  explicit STFillerWrapper(const std::string& filename, int ifSel=-1, int beamSel=-1) :
      STFiller(filename, ifSel, beamSel) {;}
-  STFillerWrapper(ScantableWrapper tbl) :
+  explicit STFillerWrapper(ScantableWrapper tbl) :
     STFiller(tbl.getCP()){;}
   ScantableWrapper getScantable() const {

trunk/src/STFit.h

@@ -31 +31 @@
 public:
   STFit() {;}
-  STFit(casa::Table tab);
-  STFit( const Scantable& parent);
+  explicit STFit(casa::Table tab);
+  explicit STFit( const Scantable& parent);
 
   virtual ~STFit();

trunk/src/STFocus.h

@@ -30 +30 @@
 public:
   STFocus() {;}
-  STFocus(casa::Table tab);
-  STFocus( const Scantable& parent );
+  explicit STFocus(casa::Table tab);
+  explicit STFocus( const Scantable& parent );
 
   virtual ~STFocus();

trunk/src/STFrequencies.h

@@ -32 +32 @@
 public:
   STFrequencies() {;}
-  STFrequencies(casa::Table tab);
-  STFrequencies(const Scantable& parent);
+  explicit STFrequencies(casa::Table tab);
+  explicit STFrequencies(const Scantable& parent);
 
   virtual ~STFrequencies();

trunk/src/STHistory.h

@@ -30 +30 @@
 public:
   STHistory() {;}
-  STHistory(casa::Table tab);
-  STHistory( const Scantable& parent );
+  explicit STHistory(casa::Table tab);
+  explicit STHistory( const Scantable& parent );
 
   virtual ~STHistory();

trunk/src/STLineFinder.h

@@ -83 +83 @@
                                            // start channel and stop+1
    public:
-        IntersectsWith(const std::pair<int,int> &in_line1);
+       explicit IntersectsWith(const std::pair<int,int> &in_line1);
         // return true if line2 intersects with line1 with at least one
         // common channel, and false otherwise
@@ -95 +95 @@
                                            // start channel and stop+1
    public:
-        BuildUnion(const std::pair<int,int> &line1);
+        explicit BuildUnion(const std::pair<int,int> &line1);
         // update temp_line with a union of temp_line and new_line
         // provided there is no gap between the lines
@@ -109 +109 @@
                                            // start channel and stop+1
    public:
-        LaterThan(const std::pair<int,int> &in_line1);
+        explicit LaterThan(const std::pair<int,int> &in_line1);
 
         // return true if line2 should be placed later than line1

trunk/src/STMath.h

@@ -46 +46 @@
   * @param insitu the toggle for this behaviour
   */
-  STMath(bool insitu=true);
+  explicit STMath(bool insitu=true);
 
   virtual ~STMath();

trunk/src/STMathWrapper.h

@@ -32 +32 @@
 public:
   STMathWrapper() {;}
-  STMathWrapper(bool insitu) : STMath(insitu) {;}
+  explicit STMathWrapper(bool insitu) : STMath(insitu) {;}
 
   virtual ~STMathWrapper() {;}

trunk/src/STMolecules.h

@@ -30 +30 @@
 public:
   STMolecules() {;}
-  STMolecules(casa::Table tab);
-  STMolecules( const Scantable& parent);
+  explicit STMolecules(casa::Table tab);
+  explicit STMolecules( const Scantable& parent);
 
   virtual ~STMolecules();

trunk/src/STPolCircular.h

@@ -40 +40 @@
   STPolCircular() {}
 
-  STPolCircular(const casa::Matrix<casa::Float>& specs)
+  explicit STPolCircular(const casa::Matrix<casa::Float>& specs)
     { setSpectra(specs); }
 

trunk/src/STPolLinear.h

@@ -39 +39 @@
   STPolLinear() {}
 
-  STPolLinear(const casa::Matrix<casa::Float>& specs)
+  explicit STPolLinear(const casa::Matrix<casa::Float>& specs)
     { setSpectra(specs); }
 

trunk/src/STPolStokes.h

@@ -28 +28 @@
   STPolStokes() {}
 
-  STPolStokes(const casa::Matrix<casa::Float>& specs)
+  explicit STPolStokes(const casa::Matrix<casa::Float>& specs)
     { setSpectra(specs); }
 

trunk/src/STTcal.h

@@ -31 +31 @@
 public:
   STTcal() {;}
-  STTcal(casa::Table tab);
-  STTcal( const Scantable& parent);
+  explicit STTcal(casa::Table tab);
+  explicit STTcal( const Scantable& parent);
 
   virtual ~STTcal();

trunk/src/STWeather.h

@@ -30 +30 @@
 public:
   STWeather() {;}
-  STWeather(casa::Table tab);
-  STWeather( const Scantable& parent);
+  explicit STWeather(casa::Table tab);
+  explicit STWeather(const Scantable& parent);
 
   virtual ~STWeather();

trunk/src/STWriter.h

@@ -55 +55 @@
 class STWriter : public Logger {
 public:
-  STWriter(const string &format = "SDFITS");
+  explicit STWriter(const string &format = "SDFITS");
   virtual ~STWriter();
 

trunk/src/STWriterWrapper.h

@@ -45 +45 @@
   //STWriterWrapper() {;}
   virtual ~STWriterWrapper() {;}
-  STWriterWrapper(const string& format = "SDFITS") : STWriter(format) {;}
+  explicit STWriterWrapper(const string& format = "SDFITS") : STWriter(format) {;}
 
   casa::Int write(const ScantableWrapper& table, const string &filename) {