svn commit: r346110 - head/usr.bin/sort

2019-09-03 Thread Conrad Meyer
Author: cem
Date: Thu Apr 11 00:39:06 2019
New Revision: 346110
URL: https://svnweb.freebsd.org/changeset/base/346110

Log:
  sort(1): Whitespace and style cleanup
  
  No functional change.
  
  Sponsored by: Dell EMC Isilon

Modified:
  head/usr.bin/sort/sort.c

Modified: head/usr.bin/sort/sort.c
==
--- head/usr.bin/sort/sort.cWed Apr 10 22:26:59 2019(r346109)
+++ head/usr.bin/sort/sort.cThu Apr 11 00:39:06 2019(r346110)
@@ -566,55 +566,57 @@ static bool
 set_sort_modifier(struct sort_mods *sm, int c)
 {
 
-   if (sm) {
-   switch (c){
-   case 'b':
-   sm->bflag = true;
-   break;
-   case 'd':
-   sm->dflag = true;
-   break;
-   case 'f':
-   sm->fflag = true;
-   break;
-   case 'g':
-   sm->gflag = true;
-   need_hint = true;
-   break;
-   case 'i':
-   sm->iflag = true;
-   break;
-   case 'R':
-   sm->Rflag = true;
-   need_random = true;
-   break;
-   case 'M':
-   initialise_months();
-   sm->Mflag = true;
-   need_hint = true;
-   break;
-   case 'n':
-   sm->nflag = true;
-   need_hint = true;
-   print_symbols_on_debug = true;
-   break;
-   case 'r':
-   sm->rflag = true;
-   break;
-   case 'V':
-   sm->Vflag = true;
-   break;
-   case 'h':
-   sm->hflag = true;
-   need_hint = true;
-   print_symbols_on_debug = true;
-   break;
-   default:
-   return false;
-   }
-   sort_opts_vals.complex_sort = true;
-   sm->func = get_sort_func(sm);
+   if (sm == NULL)
+   return (true);
+
+   switch (c){
+   case 'b':
+   sm->bflag = true;
+   break;
+   case 'd':
+   sm->dflag = true;
+   break;
+   case 'f':
+   sm->fflag = true;
+   break;
+   case 'g':
+   sm->gflag = true;
+   need_hint = true;
+   break;
+   case 'i':
+   sm->iflag = true;
+   break;
+   case 'R':
+   sm->Rflag = true;
+   need_random = true;
+   break;
+   case 'M':
+   initialise_months();
+   sm->Mflag = true;
+   need_hint = true;
+   break;
+   case 'n':
+   sm->nflag = true;
+   need_hint = true;
+   print_symbols_on_debug = true;
+   break;
+   case 'r':
+   sm->rflag = true;
+   break;
+   case 'V':
+   sm->Vflag = true;
+   break;
+   case 'h':
+   sm->hflag = true;
+   need_hint = true;
+   print_symbols_on_debug = true;
+   break;
+   default:
+   return (false);
}
+
+   sort_opts_vals.complex_sort = true;
+   sm->func = get_sort_func(sm);
return (true);
 }
 
@@ -910,54 +912,50 @@ fix_obsolete_keys(int *argc, char **argv)
 static void
 set_random_seed(void)
 {
-   if (need_random) {
+   if (strcmp(random_source, DEFAULT_RANDOM_SORT_SEED_FILE) == 0) {
+   FILE* fseed;
+   MD5_CTX ctx;
+   char rsd[MAX_DEFAULT_RANDOM_SEED_DATA_SIZE];
+   size_t sz = 0;
 
-   if (strcmp(random_source, DEFAULT_RANDOM_SORT_SEED_FILE) == 0) {
-   FILE* fseed;
-   MD5_CTX ctx;
-   char rsd[MAX_DEFAULT_RANDOM_SEED_DATA_SIZE];
-   size_t sz = 0;
+   fseed = openfile(random_source, "r");
+   while (!feof(fseed)) {
+   int cr;
 
-   fseed = openfile(random_source, "r");
-   while (!feof(fseed)) {
-   int cr;
+   cr = fgetc(fseed);
+   if (cr == EOF)
+   break;
 
-   cr = fgetc(fseed);
-   if (cr == EOF)
-   break;
+   rsd[sz++] = (char) cr;
 
-   rsd[sz++] = (char) cr;
+   if (sz >= 

svn commit: r346110 - head/usr.bin/sort

2019-04-10 Thread Conrad Meyer
Author: cem
Date: Thu Apr 11 00:39:06 2019
New Revision: 346110
URL: https://svnweb.freebsd.org/changeset/base/346110

Log:
  sort(1): Whitespace and style cleanup
  
  No functional change.
  
  Sponsored by: Dell EMC Isilon

Modified:
  head/usr.bin/sort/sort.c

Modified: head/usr.bin/sort/sort.c
==
--- head/usr.bin/sort/sort.cWed Apr 10 22:26:59 2019(r346109)
+++ head/usr.bin/sort/sort.cThu Apr 11 00:39:06 2019(r346110)
@@ -566,55 +566,57 @@ static bool
 set_sort_modifier(struct sort_mods *sm, int c)
 {
 
-   if (sm) {
-   switch (c){
-   case 'b':
-   sm->bflag = true;
-   break;
-   case 'd':
-   sm->dflag = true;
-   break;
-   case 'f':
-   sm->fflag = true;
-   break;
-   case 'g':
-   sm->gflag = true;
-   need_hint = true;
-   break;
-   case 'i':
-   sm->iflag = true;
-   break;
-   case 'R':
-   sm->Rflag = true;
-   need_random = true;
-   break;
-   case 'M':
-   initialise_months();
-   sm->Mflag = true;
-   need_hint = true;
-   break;
-   case 'n':
-   sm->nflag = true;
-   need_hint = true;
-   print_symbols_on_debug = true;
-   break;
-   case 'r':
-   sm->rflag = true;
-   break;
-   case 'V':
-   sm->Vflag = true;
-   break;
-   case 'h':
-   sm->hflag = true;
-   need_hint = true;
-   print_symbols_on_debug = true;
-   break;
-   default:
-   return false;
-   }
-   sort_opts_vals.complex_sort = true;
-   sm->func = get_sort_func(sm);
+   if (sm == NULL)
+   return (true);
+
+   switch (c){
+   case 'b':
+   sm->bflag = true;
+   break;
+   case 'd':
+   sm->dflag = true;
+   break;
+   case 'f':
+   sm->fflag = true;
+   break;
+   case 'g':
+   sm->gflag = true;
+   need_hint = true;
+   break;
+   case 'i':
+   sm->iflag = true;
+   break;
+   case 'R':
+   sm->Rflag = true;
+   need_random = true;
+   break;
+   case 'M':
+   initialise_months();
+   sm->Mflag = true;
+   need_hint = true;
+   break;
+   case 'n':
+   sm->nflag = true;
+   need_hint = true;
+   print_symbols_on_debug = true;
+   break;
+   case 'r':
+   sm->rflag = true;
+   break;
+   case 'V':
+   sm->Vflag = true;
+   break;
+   case 'h':
+   sm->hflag = true;
+   need_hint = true;
+   print_symbols_on_debug = true;
+   break;
+   default:
+   return (false);
}
+
+   sort_opts_vals.complex_sort = true;
+   sm->func = get_sort_func(sm);
return (true);
 }
 
@@ -910,54 +912,50 @@ fix_obsolete_keys(int *argc, char **argv)
 static void
 set_random_seed(void)
 {
-   if (need_random) {
+   if (strcmp(random_source, DEFAULT_RANDOM_SORT_SEED_FILE) == 0) {
+   FILE* fseed;
+   MD5_CTX ctx;
+   char rsd[MAX_DEFAULT_RANDOM_SEED_DATA_SIZE];
+   size_t sz = 0;
 
-   if (strcmp(random_source, DEFAULT_RANDOM_SORT_SEED_FILE) == 0) {
-   FILE* fseed;
-   MD5_CTX ctx;
-   char rsd[MAX_DEFAULT_RANDOM_SEED_DATA_SIZE];
-   size_t sz = 0;
+   fseed = openfile(random_source, "r");
+   while (!feof(fseed)) {
+   int cr;
 
-   fseed = openfile(random_source, "r");
-   while (!feof(fseed)) {
-   int cr;
+   cr = fgetc(fseed);
+   if (cr == EOF)
+   break;
 
-   cr = fgetc(fseed);
-   if (cr == EOF)
-   break;
+   rsd[sz++] = (char) cr;
 
-   rsd[sz++] = (char) cr;
+   if (sz >=