sysctl(CTL_HW, HW_PHYSMEM) always returns unsigned long. Will work fi…

…ne for 32 and 64 bit systems. Closes #1233.
nodejs · Jul 7, 2011 · cc83455 · cc83455
1 parent 3dbb3cd
commit cc83455
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -1,3 +1,7 @@
+In development, Version 0.5.1 (unstable)
+
+* #1233 Fix os.totalmem on FreeBSD amd64 (Artem Zaytsev)
+
 2011.07.05, Version 0.5.0 (unstable)
 
 * New non-default libuv backend to support IOCP on Windows.

diff --git a/src/platform_freebsd.cc b/src/platform_freebsd.cc
@@ -181,13 +181,9 @@ double Platform::GetFreeMemory() {
 }
 
 double Platform::GetTotalMemory() {
-#if defined(HW_PHYSMEM64)
-  uint64_t info;
-  static int which[] = {CTL_HW, HW_PHYSMEM64};
-#else
-  unsigned int info;
+  unsigned long info;
   static int which[] = {CTL_HW, HW_PHYSMEM};
-#endif
+
   size_t size = sizeof(info);
 
   if (sysctl(which, 2, &info, &size, NULL, 0) < 0) {