Hallo Ralf, On Jul 5, 2010, at 1:26 AM, Ralf Wildenhues <ralf.wildenh...@gmx.de> wrote: > Hi Gary, Peter, > > * Gary V. Vaughan wrote on Sat, Jul 03, 2010 at 10:23:12AM CEST: >> On 3 Jul 2010, at 15:15, Peter Rosin wrote: >>> Fix typo in "Add func_append_quoted and do..." >>> >>> * libltdl/config/ltmain.m4sh (func_mode_compile): Use >>> func_append_quoted instead of func_append. > [...] >> In an ideal world, we'd have test coverage of this part of the >> code, > > at least after the bug was found and fixed ;-) > >> and I would never have made the bogus commit in the >> first place :( > > Oh well. So, is anyone of you working on a testsuite addition, so I'd > be doing double work?
I plan to look at it eventually, and have it on my todo list to try and make a teat that covers that part of the code... But I haven't even looked at the bogus lines to analyse what is required to reach them with an appropriate test. Likely I won't get to it for at least a week, so if you see what is needed already, or want to work on the test case and fix, that is perfectly fine by me :-) Cheers, -- Gary V. Vaughan (g...@gnu.org)
