New testfile: dwarf-getstring
Marek Polacek
mpolacek at redhat.com
Fri Mar 4 16:32:55 UTC 2011
On Wed, 2011-03-02 at 12:33 -0800, Roland McGrath wrote:
> Really the best thing to do is to join the "gitelfutils" group via
> the https://admin.fedoraproject.org/accounts system. Once you're
> approved there, you can push to git directly. The convention is
> that you are free to push any branches you like that are called
> <username>/* where <username> is your fedoraproject.org user name
> (and * is anything, of course). If you push a branch like that with
> a new change, then it is very easy for us to merge it in.
Done. Thanks for approving. I've created mpolacek/coverage branch.
The new tests altogether with changed ChangeLog and Makefile.am are
already there. Now when I think over, I probably should've made only
one commit for that, not three. But I hope it can be merged now.
> The dwarf_getstring function is not one that really ever gets used, so
> there isn't a lot of need for this. But anything improving the test
> coverage is a good thing, and that test is just fine as it's written.
> It's a fine place to start for a first contribution.
I was concerned about that. In libdw there are a few other functions
like that. I don't know if it would be better to write just one test
for all of them or write a set of tests each testing one particular
function. I don't want to pollute tests/ directory so I'd favor the
first option. Suggestions? Thank you,
Marek
More information about the elfutils-devel
mailing list