On December 9, 2015 2:07:05 AM GMT+01:00, David Malcolm <dmalc...@redhat.com>
wrote:
>I can't comment on Mikael's observations, but here's an updated version
>of Bernhard's patch which moves the duplicated code into a new
>"find_closest_string" function in gcc/spellcheck.c.
>With that, the lookup_*_fuzzy functions are all of the form:
>
>{
> auto_vec <const char *> candidates;
>
> /* call something to populate candidates e.g.: */
> lookup_function_fuzzy_find_candidates (fun, &candidates);
>
> return find_closest_string (fn, &candidates);
>}
>
>where, as before, the auto_vec is implicitly cleaned up via a
>C++ destructor as the function exits. Hopefully with this change it
>reduces the amount of proposed C++ in the fortran subdirectory to an
>palatable amount.
>
>That's all I did; I didn't address the other issues seen in this thread
>(e.g. Mikael's notes above).
>
>Not yet well-tested; it compiles and passes the new test cases; I'm
>posting it here in case someone more familiar with the Fortran FE wants
>to take this forward (Bernhard?)
I have rewritten the autovec to plain c, will send an updated patch including
current comments and maybe the parameter handling as suggested by Joost when
done.
Thanks,
>
>Hope this is constructive
>Dave
