Fixes .c -> .cpp in comments. For doxygen.

Doxyfile

@@ -422,7 +422,7 @@ INPUT                  =
 # *.c *.cc *.cxx *.cpp *.c++ *.java *.ii *.ixx *.ipp *.i++ *.inl *.h *.hh *.hxx *.hpp
 # *.h++ *.idl *.odl *.cs *.php *.php3 *.inc *.m *.mm
 
-FILE_PATTERNS          =  *.h *.c
+FILE_PATTERNS          =  *.h *.cpp
 
 # The RECURSIVE tag can be used to turn specify whether or not subdirectories
 # should be searched for input files as well. Possible values are YES and NO.

builtin.cpp

@@ -1,4 +1,4 @@
-/** \file builtin.c
+/** \file builtin.cpp
 	Functions for executing builtin functions.
 
 	How to add a new builtin function:

builtin_commandline.cpp

@@ -1,4 +1,4 @@
-/** \file builtin_commandline.c Functions defining the commandline builtin
+/** \file builtin_commandline.cpp Functions defining the commandline builtin
 
 Functions used for implementing the commandline builtin.
 

builtin_complete.cpp

@@ -1,4 +1,4 @@
-/** \file builtin_complete.c Functions defining the complete builtin
+/** \file builtin_complete.cpp Functions defining the complete builtin
 
 Functions used for implementing the complete builtin.
 
@@ -279,8 +279,8 @@ const wchar_t *builtin_complete_get_temporary_buffer()
 
 /**
    The complete builtin. Used for specifying programmable
-   tab-completions. Calls the functions in complete.c for any heavy
-   lifting. Defined in builtin_complete.c
+   tab-completions. Calls the functions in complete.cpp for any heavy
+   lifting. Defined in builtin_complete.cpp
 */
 static int builtin_complete(parser_t &parser, wchar_t **argv)
 {

builtin_jobs.cpp

@@ -1,4 +1,4 @@
-/** \file builtin_jobs.c
+/** \file builtin_jobs.cpp
   Functions for executing the jobs builtin.
 */
 #include "config.h"

builtin_printf.cpp

@@ -1,3 +1,6 @@
+/** \file builtin_printf.cpp
+*/
+
 /* printf - format and print data
    Copyright (C) 1990-2007 Free Software Foundation, Inc.
 

builtin_set.cpp

@@ -1,4 +1,4 @@
-/** \file builtin_set.c Functions defining the set builtin
+/** \file builtin_set.cpp Functions defining the set builtin
 
 Functions used for implementing the set builtin.
 

builtin_test.cpp

@@ -646,7 +646,6 @@ expression *test_parser::parse_args(const wcstring_list_t &args, wcstring &err)
     expression *result = parser.parse_expression(0, (unsigned int)args.size());
 
     /* Handle errors */
-    bool errored = false;
     for (size_t i = 0; i < parser.errors.size(); i++)
     {
         err.append(L"test: ");

builtin_ulimit.cpp

@@ -1,4 +1,4 @@
-/** \file builtin_ulimit.c Functions defining the ulimit builtin
+/** \file builtin_ulimit.cpp Functions defining the ulimit builtin
 
 Functions used for implementing the ulimit builtin.
 

common.cpp

@@ -1,4 +1,4 @@
-/** \file common.c
+/** \file common.cpp
 
 Various functions, mostly string utilities, that are used by most
 parts of fish.

complete.cpp

@@ -1,4 +1,4 @@
-/** \file complete.c Functions related to tab-completion.
+/** \file complete.cpp Functions related to tab-completion.
 
   These functions are used for storing and retrieving tab-completion data, as well as for performing tab-completion.
 */

env.cpp

@@ -1,4 +1,4 @@
-/** \file env.c
+/** \file env.cpp
   Functions for setting and getting environment variables.
 */
 #include "config.h"

env_universal.cpp

@@ -1,3 +1,9 @@
+/** \file env_universal.cpp
+
+Various functions, mostly string utilities, that are used by most
+parts of fish.
+*/
+
 #include "config.h"
 
 

env_universal_common.cpp

@@ -1,5 +1,4 @@
-/**
-   \file env_universal_common.c
+/** \file env_universal_common.cpp
 
    The utility library for universal variables. Used both by the
    client library and by the daemon.

event.cpp

@@ -1,4 +1,4 @@
-/** \file event.c
+/** \file event.cpp
 
   Functions for handling event triggers
 

exec.cpp

@@ -1,4 +1,4 @@
-/** \file exec.c
+/** \file exec.cpp
   Functions for executing a program.
 
   Some of the code in this file is based on code from the Glibc

expand.cpp

@@ -1,4 +1,4 @@
-/**\file expand.c
+/**\file expand.cpp
 
 String expansion functions. These functions perform several kinds of
 parameter expansion.

fallback.cpp

@@ -1,4 +1,5 @@
-/**
+/** \file fallback.cpp
+
    This file only contains fallback implementations of functions which
    have been found to be missing or broken by the configuration
    scripts.

fish.cpp

@@ -1,3 +1,6 @@
+/** \file fish.cpp
+	The main loop of fish.
+*/
 /*
 Copyright (C) 2005-2008 Axel Liljencrantz
 
@@ -15,11 +18,6 @@ along with this program; if not, write to the Free Software
 Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 */
 
-
-/** \file fish.c
-	The main loop of <tt>fish</tt>.
-*/
-
 #include "config.h"
 
 

fish_indent.cpp

@@ -1,3 +1,6 @@
+/** \file fish_indent.cpp
+  The fish_indent proegram.
+*/
 /*
 Copyright (C) 2005-2008 Axel Liljencrantz
 
@@ -15,11 +18,6 @@ along with this program; if not, write to the Free Software
 Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA
 */
 
-
-/** \file fish_indent.cpp
-  The fish_indent proegram.
-*/
-
 #include "config.h"
 
 #include <stdlib.h>

fish_pager.cpp

@@ -1,3 +1,7 @@
+/** \file fish_pager.cpp
+*/
+
+
 #include "config.h"
 
 #include <stdlib.h>
@@ -439,9 +443,6 @@ static void completion_print_item(const wchar_t *prefix, comp_t *c, int width, b
             writech(L' ');
         }
         set_color(get_color(HIGHLIGHT_PAGER_DESCRIPTION), bg);
-        written += print_max(L"(", 1, 0);
-        written += print_max(c->desc.c_str(), desc_width, 0);
-        written += print_max(L")", 1, 0);
     }
     else
     {

fish_tests.cpp

@@ -1,4 +1,4 @@
-/** \file fish_tests.c
+/** \file fish_tests.cpp
   Various bug and feature tests. Compiled and run by make test.
 */
 

fishd.cpp

@@ -1,4 +1,4 @@
-/** \file fishd.c
+/** \file fishd.cpp
 
 The universal variable server. fishd is automatically started by fish
 if a fishd server isn't already running. fishd reads any saved

function.cpp

@@ -1,4 +1,4 @@
-/** \file function.c
+/** \file function.cpp
 
     Prototypes for functions for storing and retrieving function
   information. These functions also take care of autoloading

highlight.cpp

@@ -1,4 +1,4 @@
-/** \file highlight.c
+/** \file highlight.cpp
   Functions for syntax highlighting
 */
 #include "config.h"

history.cpp

@@ -1338,7 +1338,7 @@ bool history_t::save_internal_via_rewrite()
         for (size_t attempt = 0; attempt < 10 && out_fd == -1; attempt++)
         {
             char *narrow_str = wcs2str(tmp_name_template.c_str());
-            if (narrow_str && mkstemp(narrow_str))
+            if (narrow_str && mktemp(narrow_str))
             {
                 /* It was successfully templated; try opening it atomically */
                 tmp_name = str2wcstring(narrow_str);
@@ -1507,6 +1507,11 @@ void history_t::save_internal(bool vacuum)
         /* Try doing a fast append */
         ok = save_internal_via_appending();
     }
+    if (! ok)
+    {
+        /* We did not or could not append; rewrite the file ("vacuum" it) */
+        ok = this->save_internal_via_rewrite();
+    }
 }
 
 void history_t::save(void)

input.cpp

@@ -1,4 +1,4 @@
-/** \file input.c
+/** \file input.cpp
 
     Functions for reading a character of input from stdin.
 

input_common.cpp

@@ -1,4 +1,4 @@
-/** \file input_common.c
+/** \file input_common.cpp
 
 Implementation file for the low level input library
 

intern.cpp

@@ -1,4 +1,4 @@
-/** \file intern.c
+/** \file intern.cpp
 
     Library for pooling common strings
 

io.cpp

@@ -1,4 +1,4 @@
-/** \file io.c
+/** \file io.cpp
 
 Utilities for io redirection.
 

iothread.cpp

@@ -1,3 +1,6 @@
+/** \file iothread.cpp
+*/
+
 #include "config.h"
 #include "iothread.h"
 #include "common.h"

key_reader.cpp

@@ -1,4 +1,5 @@
-/*
+/** \file key_reader.cpp
+
     A small utility to print the resulting key codes from pressing a
   key. Servers the same function as hitting ^V in bash, but I prefer
   the way key_reader works.

kill.cpp

@@ -1,4 +1,4 @@
-/** \file kill.c
+/** \file kill.cpp
   The killring.
 
   Works like the killring in emacs and readline. The killring is cut

mimedb.cpp

@@ -1,4 +1,4 @@
-/** \file mimedb.c
+/** \file mimedb.cpp
 
 mimedb is a program for checking the mimetype, description and
 default action associated with a file or mimetype.  It uses the

output.cpp

@@ -1,4 +1,4 @@
-/** \file output.c
+/** \file output.cpp
  Generic output functions
  */
 

pager.cpp

@@ -1,3 +1,7 @@
+/** \file pager.cpp
+*/
+
+
 #include "config.h"
 
 #include "pager.h"
@@ -145,8 +149,6 @@ line_t pager_t::completion_print_item(const wcstring &prefix, const comp_t *c, s
         {
             written += print_max(L" ", packed_color, 1, false, &line_data);
         }
-        written += print_max(L"(", packed_color, 1, false, &line_data);
-        written += print_max(c->desc, packed_color, desc_width, false, &line_data);
     }
     else
     {
@@ -630,10 +632,6 @@ bool pager_t::completion_try_print(size_t cols, const wcstring &prefix, const co
             {
                 search_field_text.append(PAGER_SEARCH_FIELD_WIDTH - search_field_text.size(), L' ');
             }
-            line_t *search_field = &rendering->screen_data.insert_line_at_index(0);
-            
-            /* We limit the width to term_width - 1 */
-            int search_field_written = print_max(SEARCH_FIELD_PROMPT, highlight_spec_normal, term_width - 1, false, search_field);
         }
         
     }

parse_execution.cpp

@@ -506,7 +506,6 @@ parse_execution_result_t parse_execution_context_t::run_for_statement(const pars
 parse_execution_result_t parse_execution_context_t::run_switch_statement(const parse_node_t &statement)
 {
     assert(statement.type == symbol_switch_statement);
-    parse_execution_result_t ret = parse_execution_success;
     const parse_node_t *matching_case_item = NULL;
 
     parse_execution_result_t result = parse_execution_success;

parse_productions.cpp

@@ -1,3 +1,7 @@
+/** \file parse_productions.cpp
+*/
+
+
 #include "parse_productions.h"
 
 using namespace parse_productions;

parse_tree.cpp

@@ -1,3 +1,6 @@
+/** \file parse_tree.cpp
+*/
+
 #include "parse_productions.h"
 #include "tokenizer.h"
 #include "fallback.h"

parse_util.cpp

@@ -1,4 +1,4 @@
-/** \file parse_util.c
+/** \file parse_util.cpp
 
     Various mostly unrelated utility functions related to parsing,
     loading and evaluating fish code.

parser.cpp

@@ -1,4 +1,4 @@
-/** \file parser.c
+/** \file parser.cpp
 
 The fish parser. Contains functions for parsing and evaluating code.
 
@@ -2590,9 +2590,6 @@ int parser_t::eval_new_parser(const wcstring &cmd, const io_chain_t &io, enum bl
     parse_execution_context_t *ctx = new parse_execution_context_t(tree, cmd, this, exec_eval_level);
     execution_contexts.push_back(ctx);
 
-    /* Execute the first node */
-    int result = 1;
-
     /* Clean up the execution context stack */
     assert(! execution_contexts.empty() && execution_contexts.back() == ctx);
     execution_contexts.pop_back();

parser_keywords.cpp

@@ -1,4 +1,4 @@
-/** \file parser_keywords.c
+/** \file parser_keywords.cpp
 
 Functions having to do with parser keywords, like testing if a function is a block command.
 */

path.cpp

@@ -1,3 +1,7 @@
+/** \file path.cpp
+*/
+
+
 #include "config.h"
 
 #include <stdlib.h>

print_help.cpp

@@ -1,5 +1,4 @@
-
-/** \file print_help.c
+/** \file print_help.cpp
   Print help message for the specified command
 */
 

proc.cpp

@@ -1,4 +1,4 @@
-/** \file proc.c
+/** \file proc.cpp
 
 Utilities for keeping track of jobs, processes and subshells, as
 well as signal handling functions for tracking children. These
@@ -383,8 +383,6 @@ static void mark_process_status(const job_t *j, process_t *p, int status)
     }
     else
     {
-        ssize_t ignore;
-
         /* This should never be reached */
         p->completed = 1;
 

reader.cpp

@@ -1,4 +1,4 @@
-/** \file reader.c
+/** \file reader.cpp
 
 Functions for reading data from stdin and passing to the
 parser. If stdin is a keyboard, it supplies a killring, history,

sanity.cpp

@@ -1,4 +1,4 @@
-/** \file sanity.c
+/** \file sanity.cpp
   Functions for performing sanity checks on the program state
 */
 #include "config.h"

screen.cpp

@@ -1,4 +1,4 @@
-/** \file screen.c High level library for handling the terminal screen
+/** \file screen.cpp High level library for handling the terminal screen
 
 The screen library allows the interactive reader to write its
 output to screen efficiently by keeping an internal representation
@@ -298,7 +298,6 @@ size_t escape_code_length(const wchar_t *code)
         if (code[1] >= L'@' && code[1] <= L'_')
         {
             resulting_length = 2;
-            found = true;
         }
     }
 

signal.cpp

@@ -1,4 +1,4 @@
-/** \file signal.c
+/** \file signal.cpp
 
 The library for various signal related issues
 
@@ -477,7 +477,7 @@ static void handle_int(int sig, siginfo_t *info, void *context)
 }
 
 /**
-   sigchld handler. Does notification and calls the handler in proc.c
+   sigchld handler. Does notification and calls the handler in proc.cpp
 */
 static void handle_chld(int sig, siginfo_t *info, void *context)
 {

tokenizer.cpp

@@ -1,4 +1,4 @@
-/** \file tokenizer.c
+/** \file tokenizer.cpp
 
 A specialized tokenizer for tokenizing the fish language. In the
 future, the tokenizer should be extended to support marks,

util.cpp

@@ -1,4 +1,4 @@
-/** \file util.c
+/** \file util.cpp
   Generic utilities library.
 
   Contains datastructures such as automatically growing array lists, priority queues, etc.

wgetopt.cpp

@@ -1,4 +1,4 @@
-/** \file wgetopt.c
+/** \file wgetopt.cpp
   A version of the getopt library for use with wide character strings.
 
   This is simply the gnu getopt library, but converted for use with

wildcard.cpp

@@ -1,4 +1,4 @@
-/** \file wildcard.c
+/** \file wildcard.cpp
 
 Fish needs it's own globbing implementation to support
 tab-expansion of globbed parameters. Also provides recursive

wutil.cpp

@@ -1,4 +1,4 @@
-/** \file wutil.c
+/** \file wutil.cpp
   Wide character equivalents of various standard unix
   functions.
 */

xdgmime.cpp

@@ -1,3 +1,6 @@
+/** \file xdgmime.cpp
+*/
+
 /* -*- mode: C; c-file-style: "gnu" -*- */
 /* xdgmime.c: XDG Mime Spec mime resolver.  Based on version 0.11 of the spec.
  *

xdgmimealias.cpp

@@ -1,5 +1,7 @@
+/** \file xdgmimealias.cpp
+*/
 /* -*- mode: C; c-file-style: "gnu" -*- */
-/* xdgmimealias.c: Private file.  Datastructure for storing the aliases.
+/* xdgmimealias.cpp: Private file.  Datastructure for storing the aliases.
  *
  * More info can be found at http://www.freedesktop.org/standards/
  *

xdgmimeglob.cpp

@@ -1,5 +1,7 @@
+/** \file xdgmimeglob.cpp
+*/
 /* -*- mode: C; c-file-style: "gnu" -*- */
-/* xdgmimeglob.c: Private file.  Datastructure for storing the globs.
+/* xdgmimeglob.cpp: Private file.  Datastructure for storing the globs.
  *
  * More info can be found at http://www.freedesktop.org/standards/
  *

xdgmimeint.cpp

@@ -1,5 +1,8 @@
+/** \file xdgmimeint.cpp
+*/
+
 /* -*- mode: C; c-file-style: "gnu" -*- */
-/* xdgmimeint.c: Internal defines and functions.
+/* xdgmimeint.cpp: Internal defines and functions.
  *
  * More info can be found at http://www.freedesktop.org/standards/
  *

xdgmimemagic.cpp

@@ -1,3 +1,6 @@
+/** \file xdgmimemagic.cpp
+*/
+
 /* -*- mode: C; c-file-style: "gnu" -*- */
 /* xdgmimemagic.: Private file.  Datastructure for storing magic files.
  *

xdgmimeparent.cpp

@@ -1,3 +1,5 @@
+/** \file xdgmimeparent.cpp
+*/
 /* -*- mode: C; c-file-style: "gnu" -*- */
 /* xdgmimealias.c: Private file.  Datastructure for storing the hierarchy.
  *