Messages in this thread | | | Date | Thu, 31 May 2007 04:11:36 +0530 | From | "Satyam Sharma" <> | Subject | Re: [PATCH 1/3] hexdump: more output formatting |
| |
On 5/31/07, Christoph Lameter <clameter@sgi.com> wrote: > On Thu, 31 May 2007, Satyam Sharma wrote: > > > Yes, you're right, but I was just wondering whether any users really > > cared enough about the rowsize and groupsize, also seeing that > > accommodating these two args leads to a lot of increase in code. > > Have two functions > > 1. A function that prints full lines with address and ascii > This defaults to 16 bytes by default + ascii. > > hex_dump? > > 2. A function to convert a few bytes for use in other output. > > sprint_hex?
Yes, or we could go the whole hog and have a total of 4 functions:
(1) For users that want to use a short version, and are fine with the default values for the grouping / other parameters -- also have both the print_to_buffer/sprint_hex and print_to_console/hex_dump variants.
(2) For users that care about these parameters, both the variants with the expanded arglist (the ones listed above in (1) would then just be inlines wrapping over these ones)
Just that it would be nice if there is some simple (with short arglist) function that works well-enough for most people that everyone would be able to adopt/use quickly. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |