maint: sort: style adjustment to help clarify size determination
authorBernhard Voelker <mail@bernhard-voelker.de>
Wed, 20 Jun 2012 05:57:31 +0000 (07:57 +0200)
committerJim Meyering <meyering@redhat.com>
Wed, 20 Jun 2012 06:12:17 +0000 (08:12 +0200)
* src/sort.c (default_sort_size): Move physmem code "down" to first use.

src/sort.c

index 2593a2a..5a48ce6 100644 (file)
@@ -1400,22 +1400,15 @@ specify_nthreads (int oi, char c, char const *s)
   return nthreads;
 }
 
-
 /* Return the default sort size.  */
 static size_t
 default_sort_size (void)
 {
-  /* Let MEM be available memory or 1/8 of total memory, whichever
-     is greater.  */
-  double avail = physmem_available ();
-  double total = physmem_total ();
-  double mem = MAX (avail, total / 8);
-  struct rlimit rlimit;
-
   /* Let SIZE be MEM, but no more than the maximum object size or
      system resource limits.  Don't bother to check for values like
      RLIM_INFINITY since in practice they are not much less than SIZE_MAX.  */
   size_t size = SIZE_MAX;
+  struct rlimit rlimit;
   if (getrlimit (RLIMIT_DATA, &rlimit) == 0 && rlimit.rlim_cur < size)
     size = rlimit.rlim_cur;
 #ifdef RLIMIT_AS
@@ -1434,6 +1427,12 @@ default_sort_size (void)
     size = rlimit.rlim_cur / 16 * 15;
 #endif
 
+  /* Let MEM be available memory or 1/8 of total memory, whichever
+     is greater.  */
+  double avail = physmem_available ();
+  double total = physmem_total ();
+  double mem = MAX (avail, total / 8);
+
   /* Return the minimum of MEM and SIZE, but no less than
      MIN_SORT_SIZE.  Avoid the MIN macro here, as it is not quite
      right when only one argument is floating point.  */