Fix formatting

Author: ridiculousfish

builtin.cpp

@@ -1617,24 +1617,24 @@ static int builtin_echo(parser_t &parser, wchar_t **argv)
                 {
                     case L'n':
                         print_newline = false;
-                    break;
+                        break;
                     case L'e':
                         interpret_special_chars = true;
-                    break;
+                        break;
                     case L's':
                         // fish-specific extension,
                         // which we should try to nix
                         print_spaces = false;
-                    break;
+                        break;
                     case L'E':
                         interpret_special_chars = false;
-                    break;
+                        break;
                 }
             }
         }
         else
         {
-            invalid_echo_option:
+invalid_echo_option:
             break;
         }
         argv++;

builtin_commandline.cpp

@@ -169,7 +169,7 @@ static void write_part(const wchar_t *begin,
                     out.push_back(L'\n');
                     break;
                 }
-                
+
                 default:
                 {
                     break;

builtin_printf.cpp

@@ -632,7 +632,7 @@ int builtin_printf_state_t::print_formatted(const wchar_t *format, int argc, wch
                     }
                     break;
                 }
-                
+
                 modify_allowed_format_specifiers(ok, "aAcdeEfFgGiosuxX", true);
 
                 for (;; f++, direc_length++)

complete.cpp

@@ -1627,7 +1627,7 @@ void completer_t::complete_param_expand(const wcstring &sstr, bool do_file)
 
     if (expand_string(comp_str,
                       this->completions,
-                      flags ) == EXPAND_ERROR)
+                      flags) == EXPAND_ERROR)
     {
         debug(3, L"Error while expanding string '%ls'", comp_str);
     }
@@ -1911,7 +1911,7 @@ void complete(const wcstring &cmd, std::vector<completion_t> &comps, completion_
                     end_loop=1;
                     break;
                 }
-                
+
                 default:
                 {
                     break;

complete.h

@@ -127,7 +127,7 @@ class completion_t
     completion_t(const wcstring &comp, const wcstring &desc = L"", string_fuzzy_match_t match = string_fuzzy_match_t(fuzzy_match_exact), int flags_val = 0);
     completion_t(const completion_t &);
     completion_t &operator=(const completion_t &);
-    
+
     /* Compare two completions. No operating overlaoding to make this always explicit (there's potentially multiple ways to compare completions). */
     static bool is_alphabetically_less_than(const completion_t &a, const completion_t &b);
     static bool is_alphabetically_equal_to(const completion_t &a, const completion_t &b);

exec.cpp

@@ -551,7 +551,7 @@ static bool can_use_posix_spawn_for_job(const job_t *job, const process_t *proce
             return false;
         }
     }
-    
+
     /* Now see if we have a redirection involving a file. The only one we allow is /dev/null, which we assume will not fail. */
     bool result = true;
     if (chain_contains_redirection_to_real_file(job->block_io_chain()) || chain_contains_redirection_to_real_file(process->io_chain()))
@@ -790,11 +790,11 @@ void exec_job(parser_t &parser, job_t *j)
                echo alpha | cat < beta.txt
 
             Should cat output alpha or beta? bash and ksh output 'beta', tcsh gets it right and complains about ambiguity, and zsh outputs both (!). No shells appear to output 'alpha', so we match bash here. That would mean putting the pipe first, so that it gets trumped by the file redirection.
-            
+
             However, eval does this:
-            
+
                echo "begin; $argv "\n" ;end eval2_inner <&3 3<&-" | source 3<&0
-               
+
             which depends on the redirection being evaluated before the pipe. So the write end of the pipe comes first, the read pipe of the pipe comes last. See issue #966.
         */
 
@@ -811,7 +811,7 @@ void exec_job(parser_t &parser, job_t *j)
 
         /* The explicit IO redirections associated with the process */
         process_net_io_chain.append(p->io_chain());
-        
+
         /* Read pipe goes last */
         if (p != j->first_process)
         {
@@ -820,7 +820,7 @@ void exec_job(parser_t &parser, job_t *j)
             pipe_read->pipe_fd[0] = pipe_current_read;
             process_net_io_chain.push_back(pipe_read);
         }
-        
+
 
         /*
            This call is used so the global environment variable array
@@ -1241,16 +1241,16 @@ void exec_job(parser_t &parser, job_t *j)
                   forking is expensive, fish tries to avoid it when
                   possible.
                 */
-                
+
                 bool fork_was_skipped = false;
-                
+
                 const shared_ptr<io_data_t> stdout_io = process_net_io_chain.get_io_for_fd(STDOUT_FILENO);
                 const shared_ptr<io_data_t> stderr_io = process_net_io_chain.get_io_for_fd(STDERR_FILENO);
-                
+
                 /* If we are outputting to a file, we have to actually do it, even if we have no output, so that we can truncate the file. Does not apply to /dev/null. */
                 bool must_fork = redirection_is_to_real_file(stdout_io.get()) || redirection_is_to_real_file(stderr_io.get());
                 if (! must_fork)
-                {   
+                {
                     if (p->next == NULL)
                     {
                         const bool stdout_is_to_buffer = stdout_io && stdout_io->io_mode == IO_BUFFER;
@@ -1299,7 +1299,7 @@ void exec_job(parser_t &parser, job_t *j)
                         }
                     }
                 }
-                
+
 
                 if (fork_was_skipped)
                 {

expand.cpp

@@ -1752,15 +1752,15 @@ int expand_string(const wcstring &input, std::vector<completion_t> &output, expa
 
         remove_internal_separator(next_str, (EXPAND_SKIP_WILDCARDS & flags) ? true : false);
         const wchar_t *next = next_str.c_str();
-        
+
         const bool has_wildcard = wildcard_has(next, 1);
-        
+
         if (has_wildcard && (flags & EXECUTABLES_ONLY))
         {
             // Don't do wildcard expansion for executables. See #785. So do nothing here.
         }
         else if (((flags & ACCEPT_INCOMPLETE) && (!(flags & EXPAND_SKIP_WILDCARDS))) ||
-                has_wildcard)
+                 has_wildcard)
         {
             const wchar_t *start, *rest;
 

fallback.cpp

@@ -1506,7 +1506,7 @@ static int mk_wcswidth(const wchar_t *pwcs, size_t n)
     {
         if (pwcs[i] == L'\0')
             break;
-        
+
         int w = mk_wcwidth(pwcs[i]);
         if (w < 0)
         {

fish.cpp

@@ -366,9 +366,12 @@ static int fish_parse_opt(int argc, char **argv, std::vector<std::string> *out_c
     is_login |= (strcmp(argv[0], "-fish") == 0);
 
     /* We are an interactive session if we are either forced, or have not been given an explicit command to execute and stdin is a tty. */
-    if (force_interactive) {
+    if (force_interactive)
+    {
         is_interactive_session = true;
-    } else if (is_interactive_session) {
+    }
+    else if (is_interactive_session)
+    {
         is_interactive_session = ! has_cmd && (my_optind == argc) && isatty(STDIN_FILENO);
     }
 
@@ -402,7 +405,7 @@ int main(int argc, char **argv)
         debug(1, _(L"Can not use the no-execute mode when running an interactive session"));
         no_exec = 0;
     }
-    
+
     /* Only save (and therefore restore) the fg process group if we are interactive. See #197, #1002 */
     if (is_interactive_session)
     {
@@ -509,7 +512,7 @@ int main(int argc, char **argv)
     }
 
     proc_fire_event(L"PROCESS_EXIT", EVENT_EXIT, getpid(), res);
-    
+
     restore_term_mode();
     restore_term_foreground_process_group();
     history_destroy();

fish_tests.cpp

@@ -532,7 +532,7 @@ static void test_utils()
 {
     say(L"Testing utils");
     const wchar_t *a = L"echo (echo (echo hi";
-    
+
     const wchar_t *begin = NULL, *end = NULL;
     parse_util_cmdsubst_extent(a, 0, &begin, &end);
     if (begin != a || end != begin + wcslen(begin)) err(L"parse_util_cmdsubst_extent failed on line %ld", (long)__LINE__);
@@ -542,7 +542,7 @@ static void test_utils()
     if (begin != a || end != begin + wcslen(begin)) err(L"parse_util_cmdsubst_extent failed on line %ld", (long)__LINE__);
     parse_util_cmdsubst_extent(a, 3, &begin, &end);
     if (begin != a || end != begin + wcslen(begin)) err(L"parse_util_cmdsubst_extent failed on line %ld", (long)__LINE__);
-    
+
     parse_util_cmdsubst_extent(a, 8, &begin, &end);
     if (begin != a + wcslen(L"echo (")) err(L"parse_util_cmdsubst_extent failed on line %ld", (long)__LINE__);
 
@@ -788,7 +788,7 @@ static void test_path()
     {
         err(L"Bug in canonical PATH code");
     }
-    
+
     if (paths_are_equivalent(L"/foo/bar/baz", L"foo/bar/baz")) err(L"Bug in canonical PATH code on line %ld", (long)__LINE__);
     if (! paths_are_equivalent(L"///foo///bar/baz", L"/foo/bar////baz//")) err(L"Bug in canonical PATH code on line %ld", (long)__LINE__);
     if (! paths_are_equivalent(L"/foo/bar/baz", L"/foo/bar/baz")) err(L"Bug in canonical PATH code on line %ld", (long)__LINE__);

history.cpp

@@ -629,7 +629,7 @@ void history_t::get_string_representation(wcstring &result, const wcstring &sepa
     scoped_lock locker(lock);
 
     bool first = true;
-    
+
     std::set<wcstring> seen;
 
     /* Append new items. Note that in principle we could use const_reverse_iterator, but we do not because reverse_iterator is not convertible to const_reverse_iterator ( http://github.com/fish-shell/fish-shell/issues/431 ) */
@@ -638,7 +638,7 @@ void history_t::get_string_representation(wcstring &result, const wcstring &sepa
         /* Skip duplicates */
         if (! seen.insert(iter->str()).second)
             continue;
-            
+
         if (! first)
             result.append(separator);
         result.append(iter->str());
@@ -651,11 +651,11 @@ void history_t::get_string_representation(wcstring &result, const wcstring &sepa
     {
         size_t offset = *iter;
         const history_item_t item = history_t::decode_item(mmap_start + offset, mmap_length - offset, mmap_type);
-        
+
         /* Skip duplicates */
         if (! seen.insert(item.str()).second)
             continue;
-        
+
         if (! first)
             result.append(separator);
         result.append(item.str());

parse_util.cpp

@@ -245,10 +245,10 @@ void parse_util_cmdsubst_extent(const wchar_t *buff, size_t cursor_pos, const wc
     const wchar_t * const cursor = buff + cursor_pos;
 
     CHECK(buff,);
-    
+
     const size_t bufflen = wcslen(buff);
     assert(cursor_pos <= bufflen);
-    
+
     /* ap and bp are the beginning and end of the tightest command substitition found so far */
     const wchar_t *ap = buff, *bp = buff + bufflen;
     const wchar_t *pos = buff;
@@ -265,7 +265,7 @@ void parse_util_cmdsubst_extent(const wchar_t *buff, size_t cursor_pos, const wc
         {
             end = const_cast<wchar_t *>(buff) + bufflen;
         }
-        
+
         if (begin < cursor && end >= cursor)
         {
             /* This command substitution surrounds the cursor, so it's a tighter fit */
@@ -290,7 +290,7 @@ void parse_util_cmdsubst_extent(const wchar_t *buff, size_t cursor_pos, const wc
             assert(pos <= buff + bufflen);
         }
     }
-    
+
     if (a != NULL) *a = ap;
     if (b != NULL) *b = bp;
 }
@@ -383,7 +383,7 @@ static void job_or_process_extent(const wchar_t *buff,
 
                 break;
             }
-            
+
             default:
             {
                 break;

parser.cpp

@@ -2013,7 +2013,7 @@ int parser_t::parse_job(process_t *p,
                 {
 
                     const wchar_t *cmd = args.at(0).completion.c_str();
-                    
+
                     /*
                      We couldn't find the specified command.
 
@@ -2036,20 +2036,20 @@ int parser_t::parse_job(process_t *p,
                     if (equals_ptr != NULL)
                     {
                         /* Try to figure out if this is a pure variable assignment (foo=bar), or if this appears to be running a command (foo=bar ruby...) */
-                        
+
                         const wcstring name_str = wcstring(cmd, equals_ptr - cmd); //variable name, up to the =
                         const wcstring val_str = wcstring(equals_ptr + 1); //variable value, past the =
-                        
+
                         wcstring next_str;
                         if (tok_peek_next(tok, &next_str) == TOK_STRING && ! next_str.empty())
                         {
                             wcstring ellipsis_str = wcstring(1, ellipsis_char);
                             if (ellipsis_str == L"$")
                                 ellipsis_str = L"...";
-                            
+
                             /* Looks like a command */
                             debug(0,
-                                  _( L"Unknown command '%ls'. Did you mean to run %ls with a modified environment? Try 'env %ls=%ls %ls%ls'. See the help section on the set command by typing 'help set'."),
+                                  _(L"Unknown command '%ls'. Did you mean to run %ls with a modified environment? Try 'env %ls=%ls %ls%ls'. See the help section on the set command by typing 'help set'."),
                                   cmd,
                                   next_str.c_str(),
                                   name_str.c_str(),

path.cpp

@@ -400,7 +400,7 @@ void path_make_canonical(wcstring &path)
             path.at(trailing++) = c;
         }
         prev_was_slash = is_slash;
-    }    
+    }
     assert(trailing <= len);
     if (trailing < len)
         path.resize(trailing);
@@ -410,32 +410,32 @@ bool paths_are_equivalent(const wcstring &p1, const wcstring &p2)
 {
     if (p1 == p2)
         return true;
-    
+
     size_t len1 = p1.size(), len2 = p2.size();
-    
+
     // Ignore trailing slashes after the first character
     while (len1 > 1 && p1.at(len1 - 1) == L'/') len1--;
     while (len2 > 1 && p2.at(len2 - 1) == L'/') len2--;
-    
+
     // Start walking
     size_t idx1 = 0, idx2 = 0;
     while (idx1 < len1 && idx2 < len2)
     {
         wchar_t c1 = p1.at(idx1), c2 = p2.at(idx2);
-        
+
         // If the characters are different, the strings are not equivalent
         if (c1 != c2)
             break;
-        
+
         idx1++;
         idx2++;
-        
+
         // If the character was a slash, walk forwards until we hit the end of the string, or a non-slash
         // Note the first condition is invariant within the loop
         while (c1 == L'/' && idx1 < len1 && p1.at(idx1) == L'/') idx1++;
         while (c2 == L'/' && idx2 < len2 && p2.at(idx2) == L'/') idx2++;
     }
-    
+
     // We matched if we consumed all of the characters in both strings
     return idx1 == len1 && idx2 == len2;
 }