From: nds <nds@opencascade.com>
Date: Thu, 15 Aug 2019 08:47:36 +0000 (+0300)
Subject: Revert "0030901: Visualization - OSD_MemInfo moving memory computation out of the... 
X-Git-Url: http://git.dev.opencascade.org/gitweb/?a=commitdiff_plain;h=cbf87d0753da47836628c9d25df846a3b6858a80;p=occt-copy.git

Revert "0030901: Visualization - OSD_MemInfo moving memory computation out of the constructor"
---

diff --git a/src/Draw/Draw_BasicCommands.cxx b/src/Draw/Draw_BasicCommands.cxx
index a5d1f332dd..638fcfdd9d 100644
--- a/src/Draw/Draw_BasicCommands.cxx
+++ b/src/Draw/Draw_BasicCommands.cxx
@@ -763,7 +763,6 @@ static int dmeminfo (Draw_Interpretor& theDI,
                      const char**      theArgVec)
 {
   OSD_MemInfo aMemInfo;
-  aMemInfo.Update();
   if (theArgNb <= 1)
   {
     theDI << aMemInfo.ToString();
diff --git a/src/OSD/OSD_MemInfo.cxx b/src/OSD/OSD_MemInfo.cxx
index 04c883a2c7..d5eda9ee26 100644
--- a/src/OSD/OSD_MemInfo.cxx
+++ b/src/OSD/OSD_MemInfo.cxx
@@ -43,11 +43,7 @@
 // =======================================================================
 OSD_MemInfo::OSD_MemInfo()
 {
-  // reset values
-  for (Standard_Integer anIter = 0; anIter < MemCounter_NB; ++anIter)
-  {
-    myCounters[anIter] = Standard_Size(-1);
-  }
+  Update();
 }
 
 // =======================================================================
@@ -231,14 +227,14 @@ Standard_Size OSD_MemInfo::Value (const OSD_MemInfo::Counter theCounter) const
 // function : ValueMiB
 // purpose  :
 // =======================================================================
-Standard_Real OSD_MemInfo::ValueMiB (const OSD_MemInfo::Counter theCounter) const
+Standard_Size OSD_MemInfo::ValueMiB (const OSD_MemInfo::Counter theCounter) const
 {
   if (theCounter < 0 || theCounter >= MemCounter_NB)
   {
-    return Standard_Real(-1.);
+    return Standard_Size(-1);
   }
   return (myCounters[theCounter] == Standard_Size(-1))
-       ? Standard_Real(-1.) : ((Standard_Real)myCounters[theCounter] / (1024 * 1024));
+       ? Standard_Size(-1) : (myCounters[theCounter] / (1024 * 1024));
 }
 
 // =======================================================================
diff --git a/src/OSD/OSD_MemInfo.hxx b/src/OSD/OSD_MemInfo.hxx
index 40633e00f6..e1def2dc3d 100644
--- a/src/OSD/OSD_MemInfo.hxx
+++ b/src/OSD/OSD_MemInfo.hxx
@@ -82,7 +82,7 @@ public:
   //! Return value or specified counter in MiB.
   //! Notice that NOT all counters are available on various systems.
   //! Standard_Size(-1) means invalid (unavailable) value.
-  Standard_EXPORT Standard_Real ValueMiB (const OSD_MemInfo::Counter theCounter) const;
+  Standard_EXPORT Standard_Size ValueMiB (const OSD_MemInfo::Counter theCounter) const;
 
 public:
 
diff --git a/src/QABugs/QABugs_20.cxx b/src/QABugs/QABugs_20.cxx
index 93a04ae4fe..7b676e5e84 100644
--- a/src/QABugs/QABugs_20.cxx
+++ b/src/QABugs/QABugs_20.cxx
@@ -2786,7 +2786,6 @@ template<typename T> void AllocDummyArr (Draw_Interpretor& theDI, int theN1, int
   NCollection_Array1<T> aMapArr1(0, theN1), aMapArr2(0, theN2);
 
   OSD_MemInfo aMemTool;
-  aMemTool.Update();
   Standard_Size aMem0 = aMemTool.Value (OSD_MemInfo::MemHeapUsage);
 
   for (int i = 1; i < theN1; i++)