From 2a8a9b27d2eba4985b76d2f0218fd4ba83784ca9 Mon Sep 17 00:00:00 2001
From: jean-pierre roux <jean-pierre.roux@creatis.insa-lyon.fr>
Date: Wed, 27 Jul 2011 15:26:42 +0000
Subject: [PATCH] (some of the) unused variables

---
 src/creaImageIOGimmickView.cpp               |  1 +
 src/creaImageIOTreeHandlerImageAdder.cpp     |  9 +++++----
 src/creaImageIOWxAttributeSelectionPanel.cpp |  4 +++-
 src/creaImageIOWxCustomizeConfigPanel.cpp    |  8 ++++++--
 src/creaImageIOWxDescriptorPanel.cpp         | 14 ++++++++++----
 src/creaImageIOWxEditFieldsPanel.cpp         |  3 +++
 src/creaImageIOWxExportDlg.cpp               |  8 +++++---
 src/creaImageIOWxGimmickView.cpp             |  3 ++-
 src/creaImageIOWxListenerPanel.cpp           |  1 +
 src/creaImageIOWxOutputDlg.cpp               |  9 ++++++---
 src/creaImageIOWxPACSConnectionPanel.cpp     |  3 ++-
 11 files changed, 44 insertions(+), 19 deletions(-)

diff --git a/src/creaImageIOGimmickView.cpp b/src/creaImageIOGimmickView.cpp
index 2db53c5..5ad0525 100644
--- a/src/creaImageIOGimmickView.cpp
+++ b/src/creaImageIOGimmickView.cpp
@@ -705,6 +705,7 @@ void GimmickView::ReadImagesNotThreadedInVector(std::vector<vtkImageData*>& s, s
 	}
     	else if (im.size()>1) // Test inutile ? JPR
 	{
+		/// \TODO fix unused variable 'first'
 		vtkImageData* first = mReader.GetImage( im.front());
 		if (dimension == 2)
 		{
diff --git a/src/creaImageIOTreeHandlerImageAdder.cpp b/src/creaImageIOTreeHandlerImageAdder.cpp
index ce619c3..4f3d452 100644
--- a/src/creaImageIOTreeHandlerImageAdder.cpp
+++ b/src/creaImageIOTreeHandlerImageAdder.cpp
@@ -47,9 +47,9 @@ namespace creaImageIO
   {
     mProgress.Reset();
 	
-    unsigned int nbf = filenames.size(); 
+    //unsigned int nbf = filenames.size(); 
     std::vector<std::string>::const_iterator i;
-	mSynchronizer->GetList(mCurrentDB);
+    mSynchronizer->GetList(mCurrentDB);
     for (i=filenames.begin();i!=filenames.end();++i)
       {
 	
@@ -255,9 +255,10 @@ namespace creaImageIO
     mProgress.IncNumberScannedDirs();
 
     if ( !fs::exists( dirpath ) ) return;
-	time_t lastModif=fs::last_write_time(dirpath);
+    
+    /// \TODO fix warning: unused variable lastModif
+    time_t lastModif=fs::last_write_time(dirpath);
 
-	
     fs::directory_iterator end_itr; // default construction yields past-the-end
     for ( fs::directory_iterator itr( dirpath );
 	  itr != end_itr;
diff --git a/src/creaImageIOWxAttributeSelectionPanel.cpp b/src/creaImageIOWxAttributeSelectionPanel.cpp
index 29538b9..12d7043 100644
--- a/src/creaImageIOWxAttributeSelectionPanel.cpp
+++ b/src/creaImageIOWxAttributeSelectionPanel.cpp
@@ -32,7 +32,8 @@ namespace creaImageIO
   {
     GimmickDebugMessage(1,"WxCustomizeConfigPanel::WxCustomizeConfigPanel"
 			<<std::endl);
-	wxStaticText * aa=new wxStaticText(this,-1,_T(" Currently shown attributes for level: "), wxPoint(5,10));
+	/// \TODO fix warning: unused variable aa
+	wxStaticText * aa = new wxStaticText(this,-1,_T(" Currently shown attributes for level: "), wxPoint(5,10));
 	wxArrayString as;
 	std::stringstream out;
 	for(int i=1;i<=numLev;i++)
@@ -42,6 +43,7 @@ namespace creaImageIO
 		out.str("");
 	}
 	levels=new wxComboBox(this, ID_COMBO,_T("1"),wxPoint(190, 5),wxDefaultSize,as);
+	/// \TODO fix warning: unused variable na	
 	wxStaticText * na=new wxStaticText(this,-1,_T(" Currently hidden attributes: "), wxPoint(255,10));
 
     shownAtts=new wxListCtrl(this, wxID_ANY, wxPoint(5,30), wxSize(160,90), wxLC_REPORT | wxLC_NO_HEADER );
diff --git a/src/creaImageIOWxCustomizeConfigPanel.cpp b/src/creaImageIOWxCustomizeConfigPanel.cpp
index c28a37d..3a1bc9c 100644
--- a/src/creaImageIOWxCustomizeConfigPanel.cpp
+++ b/src/creaImageIOWxCustomizeConfigPanel.cpp
@@ -26,15 +26,19 @@ namespace creaImageIO
   {
     GimmickDebugMessage(1,"WxCustomizeConfigPanel::WxCustomizeConfigPanel"
 			<<std::endl);
-	wxStaticText * cp=new wxStaticText(this,-1,_T(" Copy Path: "), wxPoint(5,10));
-    copyPath=new wxTextCtrl(this, wxID_ANY, crea::std2wx(copyP), wxPoint(150,10), wxSize(250,20));
+	/// \TODO fix  warning: unused variable cp
+        wxStaticText * cp=new wxStaticText(this,-1,_T(" Copy Path: "), wxPoint(5,10));
+        copyPath=new wxTextCtrl(this, wxID_ANY, crea::std2wx(copyP), wxPoint(150,10), wxSize(250,20));
 
+	/// \TODO fix  warning: unused variable dp
 	wxStaticText * dp=new wxStaticText(this,-1,_T(" Database Path: "), wxPoint(5,40));
 	dbPath=new wxTextCtrl(this, wxID_ANY, crea::std2wx(databaseP), wxPoint(150,40), wxSize(250,20));
 
+	/// \TODO fix  warning: unused variable se
 	wxStaticText * se=new wxStaticText(this,-1,_T(" Synchronization Event: "), wxPoint(5,70));
 	syncEvent=new wxTextCtrl(this, wxID_ANY, crea::std2wx(syncEv), wxPoint(150,70), wxSize(250,20));
 
+	/// \TODO fix  warning: unused variable sf
 	wxStaticText * sf=new wxStaticText(this,-1,_T(" Synchronization Frequency: "), wxPoint(5,100));
 	syncFrequency=new wxTextCtrl(this, wxID_ANY, crea::std2wx(syncFr), wxPoint(150,100), wxSize(250,20));
 
diff --git a/src/creaImageIOWxDescriptorPanel.cpp b/src/creaImageIOWxDescriptorPanel.cpp
index d50de1f..d78d1e4 100644
--- a/src/creaImageIOWxDescriptorPanel.cpp
+++ b/src/creaImageIOWxDescriptorPanel.cpp
@@ -35,28 +35,32 @@ namespace creaImageIO
 
 	wxButton *LoadDescriptor = new wxButton(this, -1,_T("Load a descriptor"), wxPoint(150,7) );
 	Connect( LoadDescriptor->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxDescriptorPanel::OnLoad ); 
-	
+
+        /// \TODO fix warning: unused variable line1		
 	wxStaticLine *line1 = new wxStaticLine(this, -1, wxPoint(5,40), wxSize(540,2));
 
 	// LEVEL
+        /// \TODO fix warning: unused variable LevelText
 	wxStaticText * LevelText=new wxStaticText(this,-1,_T(" Level: "), wxPoint(5,50));
 	LevelCtrl=new wxTextCtrl(this, ID_GR_CTRL,_T("patient"), wxPoint(50,50), wxSize(50,25));
 	wxButton *addLevel = new wxButton(this, ID_LEVEL_ADD,_T("add a level"), wxPoint(150,50) );
 	Connect( addLevel->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxDescriptorPanel::OnAddLevel ); 
 
+        /// \TODO fix warning: unused variable line2
 	wxStaticLine *line2 = new wxStaticLine(this, -1, wxPoint(5,75), wxSize(540,2));
 
 	// ATTRIBUTES
 
+        /// \TODO fix warning: unused variable GR
 	wxStaticText * GR=new wxStaticText(this,-1,_T(" DICOM Group: "), wxPoint(5,110));
         GRCtrl=new wxTextCtrl(this, ID_GR_CTRL,_T("0x0010"), wxPoint(82,110), wxSize(50,25));
 	Connect( GRCtrl->GetId(), wxEVT_COMMAND_TEXT_UPDATED , (wxObjectEventFunction) &WxDescriptorPanel::OnDicomAttribute ); 
 	
+        /// \TODO fix warning: unused variable EL
 	wxStaticText * EL=new wxStaticText(this,-1,_T(" DICOM Element: "), wxPoint(140,110));
 	ELCtrl=new wxTextCtrl(this, ID_EL_CTRL,_T("0x0010"), wxPoint(230,110), wxSize(50,25));
 	Connect( ELCtrl->GetId(), wxEVT_COMMAND_TEXT_UPDATED , (wxObjectEventFunction) &WxDescriptorPanel::OnDicomAttribute ); 
 
-	
 	wxString choices[3];
 	choices[0] = _T("Unknow Attribute");
 	std::map<std::string, std::string>::iterator it_att =ownatt.begin();
@@ -72,7 +76,7 @@ namespace creaImageIO
 
 	wxButton *addAttribute = new wxButton(this, ID_ATTRIBUTE_ADD,_T("add an attribute"), wxPoint(440,110) );
 	Connect( addAttribute->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxDescriptorPanel::OnAddAttribute ); 
-
+        /// \TODO fix warning: unused variable line3
 	wxStaticLine *line3 = new wxStaticLine(this, -1, wxPoint(5,140), wxSize(540,2));
 
 	// RESULT
@@ -81,6 +85,7 @@ namespace creaImageIO
 	wxButton *RemoveCtrl = new wxButton(this, ID_REMOVE_ADD,_T("Remove an entry"), wxPoint(280,200) );
 	Connect( RemoveCtrl->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxDescriptorPanel::OnRemove ); 
 
+        /// \TODO fix warning: unused variable line4
 	wxStaticLine *line4 = new wxStaticLine(this, -1, wxPoint(5,470), wxSize(540,2));
 	// VALIDATION BUTTON
 	wxButton *Ok = new wxButton(this, -1,_T("OK"), wxPoint(10,480) );
@@ -89,6 +94,7 @@ namespace creaImageIO
 	wxButton *Apply = new wxButton(this, -1,_T("APPLY"), wxPoint(150,480) );
 	Connect( Apply->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxDescriptorPanel::OnApply ); 
 	
+        /// \TODO fix warning: unused variable Cancel
 	wxButton *Cancel = new wxButton(this, wxID_CANCEL,_T("CANCEL"), wxPoint(250,480) );
 //	Connect( Cancel->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxCloseEvent) &wxWindow::Close ); 
 
@@ -233,7 +239,7 @@ namespace creaImageIO
   //////////////////////////////////////////////////
   void WxDescriptorPanel::OnDicomAttribute(wxCommandEvent& event)
   {
-	  int i = 0;
+	  //int i = 0;
 	  if(!GRCtrl->GetValue().IsEmpty() && !ELCtrl->GetValue().IsEmpty() 
 		  && GRCtrl->GetValue().Len() == 6 && ELCtrl->GetValue().Len() == 6 && AttributeCombo->GetSelection() == 0)
 	  {
diff --git a/src/creaImageIOWxEditFieldsPanel.cpp b/src/creaImageIOWxEditFieldsPanel.cpp
index 6db827b..0fc5576 100644
--- a/src/creaImageIOWxEditFieldsPanel.cpp
+++ b/src/creaImageIOWxEditFieldsPanel.cpp
@@ -28,6 +28,7 @@ namespace creaImageIO
   {
     GimmickDebugMessage(1,"WxCustomizeConfigPanel::WxCustomizeConfigPanel"
                        <<std::endl);
+        /// \TODO fix warning: unused variable cp
 	wxStaticText * cp=new wxStaticText(this,-1,_T(" Attribute to change: "), wxPoint(5,10));
 	wxArrayString as;
 	std::vector<std::string>::const_iterator it;
@@ -39,9 +40,11 @@ namespace creaImageIO
 	std::string val=node->GetAttribute(keys[0]);
 	if(val.compare("")==0){val="?";}
 
+        /// \TODO fix warning: unused variable av
 	wxStaticText * av=new wxStaticText(this,-1,_T(" Current Value: "), wxPoint(5,40));
 	actualVal=new wxStaticText(this,-1,crea::std2wx(val), wxPoint(110,40));
 
+        /// \TODO fix warning: unused variable nv
 	wxStaticText * nv=new wxStaticText(this,-1,_T(" New Value: "), wxPoint(5,70));
 	newVal=new wxTextCtrl(this, wxID_ANY, crea::std2wx(val), wxPoint(110,70), wxSize(220,20));
 
diff --git a/src/creaImageIOWxExportDlg.cpp b/src/creaImageIOWxExportDlg.cpp
index 682cc89..873ebbd 100644
--- a/src/creaImageIOWxExportDlg.cpp
+++ b/src/creaImageIOWxExportDlg.cpp
@@ -6,8 +6,9 @@ namespace creaImageIO
    WxExportDlg::WxExportDlg(wxWindow *parent, const std::vector<std::string> storages)
     : wxDialog(parent, -1,_T("EXPORT FILES TO STORAGE"), wxDefaultPosition, wxSize(260,150))
    {
-	  int size = 16;
-	
+	  //int size = 16;
+
+          /// \TODO fix warning: unused variable ExportText	
 	  wxStaticText * ExportText=new wxStaticText(this,-1,_T(" Storage to export: "), wxPoint(5,10));
 	  wxArrayString names;
 	  std::vector<std::string>::const_iterator it = storages.begin();
@@ -22,7 +23,8 @@ namespace creaImageIO
 	  // VALIDATION BUTTON
 	  wxButton *Ok = new wxButton(this, -1,_T("OK"), wxPoint(5,50) );
 	  Connect( Ok->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxExportDlg::OnOk ); 
-	
+
+          /// \TODO fix warning: unused variable Cancel	
 	  wxButton *Cancel = new wxButton(this, wxID_CANCEL,_T("CANCEL"), wxPoint(100,50) );
 	  Layout(); 
 	  
diff --git a/src/creaImageIOWxGimmickView.cpp b/src/creaImageIOWxGimmickView.cpp
index e7dcfcf..9e79d43 100644
--- a/src/creaImageIOWxGimmickView.cpp
+++ b/src/creaImageIOWxGimmickView.cpp
@@ -171,6 +171,7 @@ namespace creaImageIO
     mBottomPanel->SetSizer(mbottom_sizer);
 
     // Splitting
+    /// \TODO fix warning: unused variable hsize
     int hsize = size.GetHeight();
 
     int top_minsize = 450;
@@ -606,7 +607,7 @@ namespace creaImageIO
   bool WxGimmickView::isNeedRecursive(std::string i_name)
   {
       boost::filesystem::directory_iterator iter(i_name), end_iter;
-	  bool bfindir = false;
+      //bool bfindir = false;
 		  for(; iter != end_iter; ++iter)
 		  {
 			  if(boost::filesystem::is_directory(*iter))
diff --git a/src/creaImageIOWxListenerPanel.cpp b/src/creaImageIOWxListenerPanel.cpp
index b921276..b9468cb 100644
--- a/src/creaImageIOWxListenerPanel.cpp
+++ b/src/creaImageIOWxListenerPanel.cpp
@@ -21,6 +21,7 @@ namespace creaImageIO
     GimmickDebugMessage(1,"WxListener::WxListener"
 			<<std::endl);
 	state=stat;
+	/// \TODO fix warning: unused variable cp
 	wxStaticText * cp=new wxStaticText(this,-1,_T(" Drive to monitor: "), wxPoint(5,15));
 	const wxString choices[] = { _T("D:"), 
 		_T("E:"), 
diff --git a/src/creaImageIOWxOutputDlg.cpp b/src/creaImageIOWxOutputDlg.cpp
index eee5600..3d2ac51 100644
--- a/src/creaImageIOWxOutputDlg.cpp
+++ b/src/creaImageIOWxOutputDlg.cpp
@@ -14,7 +14,7 @@ namespace creaImageIO
 		std::string sentence;
 		sentence = "You select";
 		std::string sentence2 = "You have the possibility to select output format :";
-		int nbsent= 0;
+		//int nbsent= 0;
 		std::vector<std::string> outsentences;
 		if (size == 1)
 		{
@@ -71,6 +71,7 @@ namespace creaImageIO
 
 
 	  int start_point = 45;
+	  /// \TODO fix unused variable ExportText
 	  wxStaticText * ExportText=new wxStaticText(this,-1,crea::std2wx(sentence), wxPoint(5,10));
 	  
 	  std::vector<std::string>::iterator it = outsentences.begin();
@@ -89,8 +90,10 @@ namespace creaImageIO
 
 	  // VALIDATION BUTTON
 	  wxButton *Ok = new wxButton(this, -1,_T("OK"), wxPoint(5,start_point+20) );
-	  Connect( Ok->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxOutputDlg::OnOk ); 
-	
+	  
+	  Connect( Ok->GetId(), wxEVT_COMMAND_BUTTON_CLICKED , (wxObjectEventFunction) &WxOutputDlg::OnOk );
+	  
+	  /// \TODO fix unused variable	Cancel
 	  wxButton *Cancel = new wxButton(this, wxID_CANCEL,_T("CANCEL"), wxPoint(100,start_point+20) );
 	  Layout(); 
 	  
diff --git a/src/creaImageIOWxPACSConnectionPanel.cpp b/src/creaImageIOWxPACSConnectionPanel.cpp
index 5fdb569..035fbc9 100644
--- a/src/creaImageIOWxPACSConnectionPanel.cpp
+++ b/src/creaImageIOWxPACSConnectionPanel.cpp
@@ -12,7 +12,7 @@ namespace creaImageIO
 		  -1, wxDefaultPosition, 
 		  wxDefaultSize,
 		  wxRESIZE_BORDER | 
-	      wxSYSTEM_MENU  |
+	          wxSYSTEM_MENU  |
 		  wxCLOSE_BOX |
 		  wxMAXIMIZE_BOX | 
 		  wxMINIMIZE_BOX | 
@@ -23,6 +23,7 @@ namespace creaImageIO
   {
     GimmickDebugMessage(1,"WxPACSConnectionPanel::WxPACSConnectionPanel"
 			<<std::endl);
+	/// \TODO fix  warning: unused variable dicId aet pn adv1 ad
 	wxStaticText * dicId=new wxStaticText(this,-1,_T(" DICOM Identification: "), wxPoint(5,5));
 	wxStaticText * aet=new wxStaticText(this,-1,_T(" AETitle: "), wxPoint(5,25));
 	aeTitle=new wxTextCtrl(this, wxID_ANY, _T("MyAeTitle"), wxPoint(75,25), wxSize(220,20));
-- 
2.45.0