Re: [PATCH] Make map_symbol_filenames_psymtab interruptible

[Tom Tromey <tromey at redhat dot com>]

>>>>> "Sterling" == Sterling Augustine <saugustine@google.com> writes:

Sterling> Enclosed is a patch to allow interrupting
Sterling> map_symbol_filenames_psymtab. This patch adds a call to QUIT in
Sterling> map_symbol_filenames_psymtab, and several functions to cleanup data
Sterling> that would have leaked.

Thanks.

Sterling> +/* Free any memory associated with a completion list.  */
Sterling> +static void

Newline after comment.

Sterling> +free_completion_list (char ***list_ptr)
Sterling> +{
Sterling> +  int i = 0;
Sterling> +  char **list = *list_ptr;
Sterling> +  while (list[i] != NULL)

Newline after declaration block.

Sterling> +
Sterling> +static void
Sterling> +do_free_completion_list (void *list)

Comment before this function.

Ok with those changes.

Tom