On Saturday 24 September 2005 11:26 pm, Frank Mayhar wrote:
> I've been using libufs as the I/O mechanism for my (heavy) modification
> of sysutils/ffsrecov. It's working to my needs and now I'm poking at
> other bits and pieces to maybe get it suitable for release into the
> wild. I just looked at cgread() to see what it does and noticed that
> there seems to be a redundant line:
>
> .
> .
> if (c >= fs->fs_ncg) {
> return (0);
> }
> ccg = fsbtodb(fs, cgtod(fs, c)) * disk->d_bsize;
> if (bread(disk, fsbtodb(fs, cgtod(fs, c)),
> disk->d_cgunion.d_buf,
> .
> .
>
> That assignment up there looks redundant, as ccg is never used. I
> suspect that it's a relic of an old lseek()/read() pair that's long
> gone.
The person you probably want to ask is [EMAIL PROTECTED] I can't find anything
in
cvs annotate or in the p4 depot to give history on how that got there. On
the face of it, it looks like it shouldn't cause any harm to just remove ccg.
--
John Baldwin <[EMAIL PROTECTED]> <>< http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve" = http://www.FreeBSD.org
_______________________________________________
freebsd-hackers@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[EMAIL PROTECTED]"
