Remove useless (and buggy) declaration of getGraphicObjectIntProperty 09/10209/1
Sylvestre Ledru [Mon, 14 Jan 2013 11:09:43 +0000 (12:09 +0100)]
Change-Id: Ia18a09f8100ee7a45a64572d5dad2095f1809dea

scilab/modules/graphic_objects/includes/DataController.hxx
scilab/modules/graphic_objects/includes/DataModel.hxx
scilab/modules/graphic_objects/src/cpp/dataModel.cpp

index 23c2714..30d8b8a 100644 (file)
@@ -30,11 +30,6 @@ public :
         return DataModel::get()->getGraphicObjectProperty(_pstID, _iName, _pvData);
     }
 
-    static void getGraphicObjectIntProperty(char const* _pstID, int _iName, void **_pvData)
-    {
-        return DataModel::get()->getGraphicObjectIntProperty(_pstID, _iName, _pvData);
-    }
-
     static char const* createDataObject(char const* _pstID, int _iType)
     {
         return DataModel::get()->createDataObject(_pstID, _iType);
index 6bb9874..6f35b28 100644 (file)
@@ -65,15 +65,6 @@ public :
     /** Returns a graphic object vector property */
     void getGraphicObjectProperty(char const* _pstID, int _iName, void **_pvData);
 
-    /**
-     * Returns a graphic object integer property
-     * Implemented in order to avoid returning the address of a
-     * static local variable across too many function calls (see the getProperty
-     * methods of the different Data classes when integer values are returned)
-     * Probably redundant with getGraphicObjectProperty
-     */
-    void getGraphicObjectIntProperty(char const* _pstID, int _iName, void **_pvData);
-
     /** Creates a data object */
     char const* createDataObject(char const* _pstID, int _iType);
 
index 8346d5c..69f4601 100644 (file)
@@ -56,16 +56,6 @@ void DataModel::getGraphicObjectProperty(char const* _pstID, int _iName, void **
     dataObject->getDataProperty(property, _pvData);
 }
 
-void DataModel::getGraphicObjectIntProperty(char const* _pstID, int _iName, void **_pvData)
-{
-    Data3D* dataObject = NULL;
-    int property = 0;
-
-    property = (*m_dataMap)[std::string(_pstID)]->getPropertyFromName(_iName);
-
-    dataObject->getDataProperty(property, _pvData);
-}
-
 char const* DataModel::createDataObject(char const* _pstID, int _iType)
 {
     Data3D* newObject = NULL;
@@ -75,27 +65,28 @@ char const* DataModel::createDataObject(char const* _pstID, int _iType)
      * (getTypeFromName) in order to avoid strcmp calls.
      */
 
-    switch (_iType) {
-    case __GO_GRAYPLOT__ :
-        newObject = new NgonGridData();
-        break;
-    case __GO_MATPLOT__ :
-        newObject = new NgonGridMatplotData();
-        break;
-    case __GO_FAC3D__ :
-        newObject = new NgonGeneralData();
-        break;
-    case __GO_PLOT3D__ :
-        newObject = new NgonGridData();
-        break;
-    case __GO_POLYLINE__ :
-        newObject = new NgonPolylineData();
-        break;
-    case __GO_FEC__ :
-        newObject = new TriangleMeshFecData();
-        break;
-    default :
-        return NULL;
+    switch (_iType)
+    {
+        case __GO_GRAYPLOT__ :
+            newObject = new NgonGridData();
+            break;
+        case __GO_MATPLOT__ :
+            newObject = new NgonGridMatplotData();
+            break;
+        case __GO_FAC3D__ :
+            newObject = new NgonGeneralData();
+            break;
+        case __GO_PLOT3D__ :
+            newObject = new NgonGridData();
+            break;
+        case __GO_POLYLINE__ :
+            newObject = new NgonPolylineData();
+            break;
+        case __GO_FEC__ :
+            newObject = new TriangleMeshFecData();
+            break;
+        default :
+            return NULL;
     }
 
     (*m_dataMap)[std::string(_pstID)] = newObject;