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