On Sat, Jan 24, 2004 at 08:48:59PM -0800, Bryan Whitehead wrote:
> This patch keeps the same functionality but removes the warning the compiler generates.
I sent you a patch exactly like this a few days ago, but I don't know
if you got it. This way is a lot more simple than the approach you went
for in your last patch, but it really shouldn't matter at all. All it
does is to clear a warning. One tip though, in SubmittingPatches you
can read that the best way to create patches is by making them apply
with the -p1 flag. This is done by including the actual kernel source
directory when making the diff, such as this:
diff -up linux/fs/xfs/xfs_log_recover.c.orig linux/fs/xfs/xfs_log_recover.c
It doesn't matter what you named your kernel directory, since the -p1 flag
ignores that name. Using this command will improve your chances of getting
your patches included.
Tim Cambrant
Attachment:
pgp355225.pgp
Description: PGP signature
Follow-Ups:
- Re: [PATCH 2.6.2-rc1-mm3] fs/xfs/xfs_log_recover.cBryan Whitehead
- [PATCH 2.6.2-rc1-mm3] fs/xfs/xfs_log_recover.cBryan Whitehead
- Prev by Date: Re: 2.6.1: process start times by procps
- Next by Date: 2.6.2-rc in BK: Oops loading parport_pc.
- Previous by thread: [PATCH 2.6.2-rc1-mm3] fs/xfs/xfs_log_recover.c
- Next by thread: Re: [PATCH 2.6.2-rc1-mm3] fs/xfs/xfs_log_recover.c
- Indexes:[Main][Thread]