
16 Dec
2013
16 Dec
'13
7:12 p.m.
On Sat, 2013-12-14 at 10:39 +0530, Jagan Teki wrote:
On Sat, Dec 14, 2013 at 9:17 AM, fenghua@phytium.com.cn wrote:
From: Scott Wood scottwood@freescale.com
[snip]
if (argc != 5) {
fprintf(stderr, "Statically apply ELF rela relocations\n");
fprintf(stderr, "Usage: %s <bin file> <text base> " \
"<rela start> <rela end>\n", argv[0]);
Try to avoid unnecessary line continuations at fprintf
FWIW, it wasn't like that it in my original patch. I'm guessing he changed it due to checkpatch output; checkpatch recognizes printf() but not fprintf(). It should probably be changed to recognize *printf(), just as it recognizes *printk().
-Scott