public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Hans-Peter Nilsson <hp@bitrange.com>
To: Jan Beulich <JBeulich@novell.com>
Cc: binutils@sources.redhat.com
Subject: Re: QUOTES_IN_INSN
Date: Wed, 11 May 2005 19:05:00 -0000	[thread overview]
Message-ID: <Pine.BSF.4.58.0505111358580.57290@dair.pair.com> (raw)
In-Reply-To: <s282446a.086@emea1-mh.id2.novell.com>

On Wed, 11 May 2005, Jan Beulich wrote:

> >>> Hans-Peter Nilsson <hp@bitrange.com> 11.05.05 18:37:00 >>>
> >On Wed, 11 May 2005, Jan Beulich wrote:
> >> gas/testsuite/
> >> 2005-05-11  Jan Beulich  <jbeulich@novell.com>
> >>
> >> 	* gas/mmix/err-byte1.s: Don't expect specific error text on line 10.
> >
> >Can you explain this, please?  Your message said nothing about it.
> >Briefly, it looks like you introduce a regression in the error
> >message.  What is the exact change in the error output?
>
> There are now two messages on the same subject. I simply
> didn't want to check for both indivually; if you'd rather have
> me do it that way, I'll do.

Yes, please.  At least one of the checks should be explicitly
for something like "unterminated string".  Your patch changed
the test to match *any* error message on that line, which is
what I oppose.

Can you please quote the whole error output from the changed
test?

> This appears to be the only
> testcase
> that ever cared to check that unterminated strings are caught,
> which I found kind of surprising...

Sorry. O:-)

(And the specific reason I do it in the mmix-specific parts of
the testsuite is the munging I do in mmix_handle_mmixal called
for some sanity checking, IIRC.)

brgds, H-P

       reply	other threads:[~2005-05-11 18:05 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <s282446a.086@emea1-mh.id2.novell.com>
2005-05-11 19:05 ` Hans-Peter Nilsson [this message]
2005-05-13 13:35 QUOTES_IN_INSN Jan Beulich
2005-05-13 16:20 ` QUOTES_IN_INSN Hans-Peter Nilsson
2005-05-17 13:14 ` QUOTES_IN_INSN Nick Clifton
  -- strict thread matches above, loose matches on Subject: below --
2005-05-11 17:32 QUOTES_IN_INSN Jan Beulich
2005-05-11  9:13 QUOTES_IN_INSN Jan Beulich
2005-05-11 16:44 ` QUOTES_IN_INSN Hans-Peter Nilsson
2005-05-02  8:25 QUOTES_IN_INSN Jan Beulich
2005-05-02 18:50 ` QUOTES_IN_INSN Ian Lance Taylor

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=Pine.BSF.4.58.0505111358580.57290@dair.pair.com \
    --to=hp@bitrange.com \
    --cc=JBeulich@novell.com \
    --cc=binutils@sources.redhat.com \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).