This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: Redboot 'gunzip' command
- From: Peter Korsgaard <jacmet at sunsite dot dk>
- To: Andrew Lunn <andrew at lunn dot ch>
- Cc: ecos-patches at ecos dot sourceware dot org
- Date: Wed, 23 Nov 2005 21:41:08 +0100
- Subject: Re: Redboot 'gunzip' command
- References: <87acoe70b5.fsf@p4.48ers.dk> <87d5t6siz2.fsf@p4.48ers.dk> <20050413180658.GK21136@lunn.ch>
>>>>> "Andrew" == Andrew Lunn <andrew@lunn.ch> writes:
Andrew> Thanks for the patch. Before we can accept it we need a copyright
Andrew> assignment.
It's now finally on it's way to eCosCentric - sorry for the delay.
Please apply the following patch which fixes 2 diag_printf format
string warnings once gunzip.c gets added to CVS.
diff -urpN /tmp/packages/redboot/current/ChangeLog packages/redboot/current/ChangeLog
--- /tmp/packages/redboot/current/ChangeLog 2005-11-23 14:04:42.000000000 +0100
+++ packages/redboot/current/ChangeLog 2005-11-23 14:14:26.000000000 +0100
@@ -1,3 +1,7 @@
+2005-11-23 Peter Korsgaard <peter.korsgaard@barco.com>
+
+ * src/gunzip.c (do_gunzip): Fixed diag_printf format string warnings.
+
2005-10-17 Gary Thomas <gary@mlbassoc.com>
* src/iomem.c (do_iopeek): Correct number of options.
diff -urpN /tmp/packages/redboot/current/src/gunzip.c packages/redboot/current/src/gunzip.c
--- /tmp/packages/redboot/current/src/gunzip.c 2005-11-23 14:05:32.000000000 +0100
+++ packages/redboot/current/src/gunzip.c 2005-11-23 14:13:26.000000000 +0100
@@ -85,7 +85,8 @@ do_gunzip(int argc, char *argv[])
&& load_address >= (CYG_ADDRESS)ram_start
&& load_address < load_address_end) {
src = load_address;
- diag_printf("Decompressing from %p to %p\n", src, dst);
+ diag_printf("Decompressing from %p to %p\n",
+ (void*)src, (void*)dst);
}
else
{
@@ -113,7 +114,7 @@ do_gunzip(int argc, char *argv[])
} else {
load_address = entry_address = (CYG_ADDRESS)dst;
load_address_end = (CYG_ADDRESS)p->out_buf;
- diag_printf("Decompressed %d bytes\n",
+ diag_printf("Decompressed %lu bytes\n",
load_address_end - load_address);
}
}
--
Bye, Peter Korsgaard