public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Phil Muldoon <pmuldoon@redhat.com>
Cc: frysk@sourceware.org
Subject: Re: [SCM]  master: Merge branch 'master' of 	ssh://sources.redhat.com/git/frysk
Date: Wed, 02 Apr 2008 07:59:00 -0000	[thread overview]
Message-ID: <1207123115.4953.32.camel@cc1341701-a.deven1.ov.home.nl> (raw)
In-Reply-To: <47F242FD.4030001@redhat.com>

Hi Phil,

On Tue, 2008-04-01 at 15:13 +0100, Phil Muldoon wrote:
> This commit build a dead simple watchpoint scenario. The assembly level 
> program writes to a global and exits. For the time being it is a sanity 
> tests for the upcoming Watchpoint observer code, and tests that 
> watchpoints at the lowest and most simple level work.

Thanks for commenting on what you work on and why you introduced what.
It really helps keeping an overview of what is happening with the code
base and the various changes made in general. And also thanks for adding
comments to this file in a later commit (my assembly is very rusty, so
reading what it is actually supposed to do is very appreciated).

Cheers,

Mark

P.S. The diffs on frysk-cvs look really strange at times. I always just
post the actual patches myself since it seems the git diff on sourceware
just scrambles them trying to be clever and associating random files
with each other and generating a diff of them instead of a real diff of
the original file :{

  reply	other threads:[~2008-04-02  7:59 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080401122840.732.qmail@sourceware.org>
2008-04-01 14:13 ` Phil Muldoon
2008-04-02  7:59   ` Mark Wielaard [this message]
2008-04-02  9:06     ` Phil Muldoon
2008-04-02 10:43       ` Mark Wielaard
2008-04-03 13:51         ` Mark Wielaard
2008-04-03 13:52           ` Mark Wielaard

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=1207123115.4953.32.camel@cc1341701-a.deven1.ov.home.nl \
    --to=mark@klomp.org \
    --cc=frysk@sourceware.org \
    --cc=pmuldoon@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).