Re: [patch] Verify byte-by-byte if both files are the same on "remote:" [Re: [rfc] False separate debuginfo warning with "remot

Ulrich Weigand
2011-10-10T21:23:48Z

Jan Kratochvil wrote:
> On Mon, 10 Oct 2011 15:47:57 +0200, Ulrich Weigand wrote:
> > So this no longer handles the st_ino == 0 case. I think we still need to do
> > that, to cope with filesystems (e.g. on Windows?) where stat works, but does
> > not provide inode numbers ... Two files with zero st_ino should not be
> > considered equal.
>
> I believe the code is right. st_ino == 0 will verified_as_different = 0,
> therefore this code does not make any assumption about such files.
>
> FYI I reordered the condition for some negligible btter performance.

Yes, you're right. Sorry, I misread the original patch ...

> gdb/
> 2011-10-10 Jan Kratochvil <jan.kratochvil@redhat.com>
>
> Fix separate debuginfo warning with "remote:" access.
> * objfiles.h (struct objfile): New fields crc32 and crc32_p.
> * symfile.c (get_file_crc): New function with the code moved from ...
> (separate_debug_file_exists): ... this function, specifically variables
> buffer and count. New variable verified_as_different, set it. Remove
> file_crc initialization. Verify also if both files are not the same
> manually, if needed.

Looks good to me.

Thanks,
Ulrich

--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com