(some of the) unused variables

diff --git a/src/creaImageIOGimmickView.cpp b/src/creaImageIOGimmickView.cpp
index 2db53c575eb0379b61f78660e297cf9d447adb30..5ad0525e5202c73c3960e59bb3f23a0042aa8bc3 100644 (file)
--- 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 ce619c3424fd943e928f40ad8c7920f1a09030df..4f3d452f793eb84b4fed86f11b87c941bd7f980f 100644 (file)
--- 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 29538b91040bcd48b3b75667d43ec169b5af9d53..12d704390161846871c74986c6967494302cf54e 100644 (file)
--- 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 c28a37da9f2b9b8d47028e3b7baf85f347888644..3a1bc9c79359f9d1a863f03086d4477099b3978e 100644 (file)
--- 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 d50de1fecfaae7803d68b00bec5fd8540b633312..d78d1e40b1236f65695b9a1276561dc49bfffe28 100644 (file)
--- 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 6db827b7eb9cf608ebc5481d36756cf03f304879..0fc55762a8871c1d5df3e918c421e75c8e4912c0 100644 (file)
--- 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 682cc894eb8910fd0159005d13f740616aeef698..873ebbdae9cf640f572ffb21ca138b3cf2f65bbc 100644 (file)
--- 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 e7dcfcf63918959ff5d1291ac5620a10b0477159..9e79d4341575493be2a7565b8ca708af9a9957dd 100644 (file)
--- 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 b9212766370a67ad563239f7634ba6493c54f875..b9468cb243784e1c8b75b293c507d0351747beee 100644 (file)
--- 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 eee5600914af868db09e818962694a5445756a18..3d2ac5117d0c96041fd2ca99f490188afd55d3f7 100644 (file)
--- 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 5fdb5697a5ff0ad33b235335a14341df2c5bf9bd..035fbc9bdbc3e78dfbc89d65e4026368c4712e21 100644 (file)
--- 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));