Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Andrew Burgess <andrew.burgess@embecosm.com>
To: Tom Tromey <tom@tromey.com>
Cc: "Maciej W. Rozycki" <macro@wdc.com>,
	Tom Tromey <tromey@sourceware.org>,
	gdb-patches@sourceware.org
Subject: Re: [binutils-gdb] Style the gdb welcome message
Date: Tue, 15 Oct 2019 08:32:00 -0000	[thread overview]
Message-ID: <20191015083234.GT4962@embecosm.com> (raw)
In-Reply-To: <875zkr9c9p.fsf@tromey.com>

* Tom Tromey <tom@tromey.com> [2019-10-14 16:45:54 -0600]:

> Maciej>  I missed that in the various distractions that kept me from using an 
> Maciej> up-to-date version of GDB, but now that I came across it the hard way the 
> Maciej> bright pink version line hurts my eyes and disturbs me.  Setting:
> Maciej> set style enabled off
> Maciej> in ~/.gdbinit does not make any change to it, so I find it a functional 
> Maciej> regression.  I'm sure there were important reasons to make this change, 
> Maciej> but can we please get it under user control somehow?
> 
> I added this to make it clear to users that gdb is style-enabled now.
> I guess I don't care much about this particular instance though.  Maybe
> removing it is the answer.
> 
> I'm curious to hear what others think.  In the absence of comments I'll
> remove it sometime before 9.1, I think.  Or I'll approve a patch to
> do so.

FWIW I hadn't even noticed the use of colour there, so it certainly
doesn't bother me.  On my machine with both a light background and a
dark background I don't see a bright pink, but a darker pink.

I guess there's no simple way that we could postpone printing the
welcome message until some later point, say, just before GDB tries to
print anything else?

What this might mean is that if ~/.gdbinit contains only (or starts
with) commands that produce no output, for example 'set style enabled
off' then these would be processed before GDB printed the welcome
message.

I've not looked at the code at all, so have no idea how hard this
would be to actually do...

Thanks,
Andrew


  parent reply	other threads:[~2019-10-15  8:32 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20181228210008.91892.qmail@sourceware.org>
2019-10-04 22:15 ` Maciej W. Rozycki
2019-10-14 22:45   ` Tom Tromey
2019-10-15  6:32     ` Eli Zaretskii
2019-10-15  8:32     ` Andrew Burgess [this message]
2019-10-19 11:21       ` Maciej W. Rozycki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191015083234.GT4962@embecosm.com \
    --to=andrew.burgess@embecosm.com \
    --cc=gdb-patches@sourceware.org \
    --cc=macro@wdc.com \
    --cc=tom@tromey.com \
    --cc=tromey@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox