[patch/rebase] Add a rebase database to keep track of DLL addresses

Corinna Vinschen corinna-cygwin@cygwin.com
Mon Jul 25 08:53:00 GMT 2011


On Jul 24 15:12, Charles Wilson wrote:
> Ok, here's round 2. I've omitted rebase-dump-related changes for now,
> other than moving the db typedefs to rebase-db.[h,c] (*).
> 
> (*) There's a .c because we probably need to have IMG_INFO_MAGIC and
> IMG_INFO_VERSION tightly coupled with rebase-db.h.  So, since I already
> had the .c, I also moved the *_cmp() functions to rebase-db.[h,c].  It's
> possible load_image_info, save_image_info, and print_image_info could
> also be moved, but I think the needs of rebase.exe and rebase-dump.exe
> are different enough that I didn't bother right now.

Your patch looks good to me.  I don't think we need a round 3 since your
patch is basically unaffected by my change to load_image_info.

When you applied your patch, I'll move the entire db stuff into
rebase-db.c.  I'd suggest to rename the print_image_info function in
your db dumper to dump_image_info and put it into rebase-db.c as well.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat



More information about the Cygwin-apps mailing list