comments (memory leak)
Allan Cornet [Wed, 27 Aug 2008 06:19:22 +0000 (06:19 +0000)]
scilab/modules/gui/sci_gateway/c/sci_ClipBoard.c
scilab/modules/gui/sci_gateway/c/sci_uigetdir.c
scilab/modules/gui/sci_gateway/c/sci_uigetfont.c
scilab/modules/gui/sci_gateway/c/sci_x_choice.c
scilab/modules/gui/sci_gateway/c/sci_x_dialog.c
scilab/modules/gui/sci_gateway/c/sci_x_mdialog.c
scilab/modules/gui/src/cpp/ContextMenu.cpp
scilab/modules/gui/src/cpp/GetUicontrolValue.cpp

index 87c3f30..9005fed 100644 (file)
@@ -50,10 +50,13 @@ int C2F(sci_ClipBoard)(char *fname,unsigned long l)
                 {
                   /* Use the Java clipboard (CallScilabBridge.java returns "" if clipbaord could not be read) */
                   char *output = getClipboardContents();
+
+
                   m1 = (int)strlen(output);
                   n1 = 1;
                   
                   CreateVarFromPtr(Rhs+ 1,STRING_DATATYPE, &m1, &n1, &output);
+                   /* TO DO a delete [] and a FREE */
                   FREE(output);
                   LhsVar(1)=Rhs+1;
                   
index 1a332e1..cae4e48 100644 (file)
@@ -119,8 +119,9 @@ int sci_uigetdir(char *fname,unsigned long l)
       nbCol = 1;
 
       userSelection = getFileChooserSelection(fileChooserID);
-
       CreateVarFromPtr(Rhs+1, MATRIX_OF_STRING_DATATYPE, &nbRow, &nbCol, userSelection);
+      /* TO DO a delete [] userSelection */
+
     }
   else
     {
index ed3db1b..e864ab3 100644 (file)
@@ -149,6 +149,7 @@ int sci_uigetfont(char *fname,unsigned long fname_len)
   
   /* Read the user answer */
   selectedFontName = getFontChooserFontName(fontChooserID);
+  
 
   if (strcmp(selectedFontName,"")) /* The user selected a font */
     {
@@ -208,6 +209,8 @@ int sci_uigetfont(char *fname,unsigned long fname_len)
         }
     }
   
+  /* TO DO delete [] selectedFontName */
+  
   LhsVar(1) = Rhs+1;
   LhsVar(2) = Rhs+2;
   LhsVar(3) = Rhs+3;
index 44f9862..3c9c742 100644 (file)
@@ -116,6 +116,7 @@ int C2F(sci_x_choice)(char *fname,unsigned long fname_len)
     }
   else
     {
+         
       userValue = (int*)getMessageBoxUserSelectedButtons(messageBoxID);
   
       userValueDouble = (double *)MALLOC(nbRowDefaultValues*nbColDefaultValues*sizeof(double));
@@ -125,6 +126,7 @@ int C2F(sci_x_choice)(char *fname,unsigned long fname_len)
         }
 
       CreateVarFromPtr(Rhs+1, MATRIX_OF_DOUBLE_DATATYPE, &nbRowDefaultValues, &nbColDefaultValues, &userValueDouble);
+      /* TO DO : do a delete []  getMessageBoxUserSelectedButton */
     }
 
   FREE(defaultValuesInt);
index f09b649..34304f5 100644 (file)
@@ -83,9 +83,11 @@ int C2F(sci_x_dialog)(char *fname,unsigned long fname_len)
   else
     {
       userValue = getMessageBoxValue(messageBoxID);
+         
   
       nbCol = 1;
       CreateVarFromPtr(Rhs+1, MATRIX_OF_STRING_DATATYPE, &userValueSize, &nbCol, userValue);
+      /* TO DO : delete of userValue */
       
     }
 
index 7430436..3da9082 100644 (file)
@@ -140,9 +140,9 @@ int C2F(sci_x_mdialog)(char *fname,unsigned long fname_len)
   else
     {
       userValue = getMessageBoxValue(messageBoxID);
-  
       nbCol = 1;
       CreateVarFromPtr(Rhs+1, MATRIX_OF_STRING_DATATYPE, &nbRowDefaultValues, &nbColDefaultValues, userValue);
+      /* TO DO : delete of userValue */
     }
 
   LhsVar(1) = Rhs+1;
index 79dcc3f..b5e0a2d 100644 (file)
@@ -26,6 +26,7 @@ void createContextMenu(sciPointObj* sciObj)
 
 char* uiWaitContextMenu(sciPointObj* sciObj)
 {
+  /* TO DO : create a copy or move sci_uiwait to cpp to do a delete */
   return CallScilabBridge::displayAndWaitContextMenu(getScilabJavaVM(), pUICONTEXTMENU_FEATURE(sciObj)->hashMapIndex);
 }
 
index 5f34b0d..4aeb193 100644 (file)
@@ -28,6 +28,7 @@ int GetUicontrolValue(sciPointObj* sciObj)
       switch(pUICONTROL_FEATURE(sciObj)->style)
         {
         case SCI_LISTBOX:
+                     /* DO A DELETE @ end on value returned by getListBoxSelectedIndices */
           value = (int*) CallScilabBridge::getListBoxSelectedIndices(getScilabJavaVM(), 
                                                               pUICONTROL_FEATURE(sciObj)->hashMapIndex);
           valueSize = CallScilabBridge::getListBoxSelectionSize(getScilabJavaVM(),