https://sourceware.org/bugzilla/show_bug.cgi?id=29457

Jan Beulich <jbeulich at suse dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeulich at suse dot com

--- Comment #6 from Jan Beulich <jbeulich at suse dot com> ---
(In reply to Martin Liska from comment #2)
> Yes, thanks for it. I've just tested the patch and it looks nice!

I'm afraid I disagree. It not only doesn't look nice, especially register names
are close to unreadable on a terminal with a black background. Nick - I'd like
to ask that this be reconsidered. When Andrew first introduced the feature, I
specifically asked that coloring not be made the default. And now it suddenly
is (the patch wasn't even sent to the list afaics, which is likely part of the
reason why I didn't notice until now). I would have been kind of okay if the
envvar override had been accepted, but now I need to always remember to add a
pretty long option to objdump, just for its output to be at least readable?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to