Patch: src/NewField Changes

Jeffrey Oldham oldham at codesourcery.com
Thu Mar 29 18:19:12 UTC 2001


This patch to the r2/src/NewField Pooma directory prevents "g++ -Wall"
warnings.

2001-03-29  Jeffrey D. Oldham  <oldham at codesourcery.com>

	* PrintField.h (PrintField::PrintField): Reorder constructor
	initializers to reflect class member order.
	* Updater/UpdaterList.h (UpdaterListData::UpdaterListData):
	Likewise.
	(UpdaterListData::~UpdaterListData): Change loop iterator type to
	correspond to std::vector's specification.

Tested on	sequential Linux using gcc 3.1
Reviewed by	Scott Haney

Thanks,
Jeffrey D. Oldham
oldham at codesourcery.com
-------------- next part --------------
Index: PrintField.h
===================================================================
RCS file: /home/pooma/Repository/r2/src/NewField/PrintField.h,v
retrieving revision 1.2
diff -c -p -r1.2 PrintField.h
*** PrintField.h	2000/07/20 21:03:51	1.2
--- PrintField.h	2001/03/29 17:20:42
*************** public:
*** 188,194 ****
  	     bool scientific = false, int spacing = 1)
      : domainwidth_m(domainWidth), datawidth_m(dataWidth), 
        dataprecision_m(dataPrecision), carreturn_m(carReturn),
!       scientific_m(scientific), spacing_m(spacing)
      {
        PAssert(domainwidth_m > 0);
        PAssert(datawidth_m > 0);
--- 188,194 ----
  	     bool scientific = false, int spacing = 1)
      : domainwidth_m(domainWidth), datawidth_m(dataWidth), 
        dataprecision_m(dataPrecision), carreturn_m(carReturn),
!       spacing_m(spacing), scientific_m(scientific)
      {
        PAssert(domainwidth_m > 0);
        PAssert(datawidth_m > 0);
Index: Updater/UpdaterList.h
===================================================================
RCS file: /home/pooma/Repository/r2/src/NewField/Updater/UpdaterList.h,v
retrieving revision 1.7
diff -c -p -r1.7 UpdaterList.h
*** Updater/UpdaterList.h	2001/02/21 20:11:38	1.7
--- Updater/UpdaterList.h	2001/03/29 17:20:42
*************** public:
*** 66,73 ****
    // Copy constructor.
  
    UpdaterListData(const UpdaterListData &model)
!     : data_m(model.data_m),
!       updating_m(false)
    { }
    
    
--- 66,73 ----
    // Copy constructor.
  
    UpdaterListData(const UpdaterListData &model)
!     : updating_m(false), data_m(model.data_m)
!       
    { }
    
    
*************** public:
*** 88,94 ****
  
    ~UpdaterListData() 
      {
!       for (int i = 0; i < data_m.size(); i++)
          delete data_m[i]; 
      }
  
--- 88,95 ----
  
    ~UpdaterListData() 
      {
!       typedef std::vector<UpdaterListItem *>::size_type size_type;
!       for (size_type i = 0; i < data_m.size(); i++)
          delete data_m[i]; 
      }
  


More information about the pooma-dev mailing list