Messages in this thread | | | Subject | Re: [PATCH v3 0/7] Hexdump Enhancements | From | "Alastair D'Silva" <> | Date | Thu, 20 Jun 2019 09:15:58 +1000 |
| |
On Wed, 2019-06-19 at 09:31 -0700, Joe Perches wrote: > On Mon, 2019-06-17 at 12:04 +1000, Alastair D'Silva wrote: > > From: Alastair D'Silva <alastair@d-silva.org> > > > > Apologies for the large CC list, it's a heads up for those > > responsible > > for subsystems where a prototype change in generic code causes a > > change > > in those subsystems. > > > > This series enhances hexdump. > > Still not a fan of these patches.
I'm afraid there's not too much action I can take on that, I'm happy to address specific issues though.
> > > These improve the readability of the dumped data in certain > > situations > > (eg. wide terminals are available, many lines of empty bytes exist, > > etc). > > Changing hexdump's last argument from bool to int is odd. >
Think of it as replacing a single boolean with many booleans.
> Perhaps a new function should be added instead of changing > the existing hexdump. >
There's only a handful of consumers, I don't think there is a value-add in creating more wrappers vs updating the existing callers.
-- Alastair D'Silva mob: 0423 762 819 skype: alastair_dsilva Twitter: @EvilDeece blog: http://alastair.d-silva.org
| |