all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: nickrob@snap.net.nz (Nick Roberts)
To: YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp>
Cc: emacs-devel@gnu.org
Subject: Re: GDB on Mac is (NOT) Broken
Date: Sun, 21 Mar 2010 19:50:41 +1300	[thread overview]
Message-ID: <19365.49601.379093.424303@totara.tehura.co.nz> (raw)
In-Reply-To: <wl39zvk4ze.wl%mituharu@math.s.chiba-u.ac.jp>

 > >> ***************
 > >> *** 1769,1774 ****
 > >> --- 1781,1792 ----
 > >> (progn
 > >> (setq output (gdb-concat-output output gud-marker-acc))
 > >> (setq gud-marker-acc "")))
 > >> +       (if (not (string-match "\n" gdb-first-output-line))
 > 
 > >            (if (gdb-first-post-prompt)
 > 
 > > would probably also work
 > 
 > I guess you meant "(if gdb-first-prompt", but whichever the variables,
 > I think its value might be changed in the while-loop in
 > gud-gdba-marker-filter via annotation handler calls.

Actually both probably come too late.

One problem with trying to match for "Apple" in the first output line is that
the user might use -q:

  nickrob@totara:~$ gdb -q myprog
  (gdb) 

To be safer you need to add something like:

  (gdb-enqueue-input (list "server show version\n" 'gdb-apple-test)))

maybe at the start of gdb-init-2 and where gdb-apple-test inspects the
output of "show version" (the prefix server ensures that the user
doesn't see this command in the gdb history):

  ;; Workaround for some Apple versions of GDB that add ^M at EOL
  ;; after the command "server interpreter mi -stack-info-frame".
  (defun gdb-apple-test ()
  (goto-char (point-min))
  (if (re-search-forward "(Apple version " nil t)
      (setq gdb-version "pre-6.4")
      (let* ((process (get-buffer-process gud-comint-buffer))
	     (coding-systems (process-coding-system process)))
	(set-process-coding-system process
				   (coding-system-change-eol-conversion
				    (car coding-systems) 'dos)
				   (cdr coding-systems))))))

To see how this works instrument gdb-apple-test with Edebug and look in
the " *partial-output-yourprog" buffer (note leading space means hidden
buffer).

 >                                                         If we want to
 > avoid string-match for most cases, then we can save the original value
 > at the beginning of the while-loop to some variable (say,
 > orig-gdb-first-prompt) and use it like this:

The above patch would avoid continually matching in gud-gdba-marker-filter
too.

WDYT?

-- 
Nick                                           http://users.snap.net.nz/~nickrob




  reply	other threads:[~2010-03-21  6:50 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-15  6:23 GDB on Mac is (NOT) Broken Nick Roberts
2010-03-15 16:39 ` Chad Brown
2010-03-15 20:17   ` Nick Roberts
2010-03-16  0:18 ` YAMAMOTO Mitsuharu
2010-03-16  2:59   ` Steve Revilak
2010-03-16  5:27   ` Nick Roberts
2010-03-16  8:55     ` Chad Brown
2010-03-16 21:00       ` Nick Roberts
2010-03-20  4:10     ` YAMAMOTO Mitsuharu
2010-03-21  6:50       ` Nick Roberts [this message]
2010-03-22  1:29         ` YAMAMOTO Mitsuharu
2010-03-22  2:32           ` Nick Roberts
2010-03-22  2:46         ` Stefan Monnier
2010-03-22  3:04           ` Nick Roberts
2010-03-22 13:43             ` Stefan Monnier
2010-03-22 20:55               ` Nick Roberts
2010-03-23  1:13                 ` Stefan Monnier
2010-03-23  1:55                   ` YAMAMOTO Mitsuharu
2010-03-23  2:38                     ` Nick Roberts
2010-03-23  3:12                       ` Stefan Monnier
2010-03-26  9:02                     ` Nick Roberts
2010-03-26 20:20                       ` Stefan Monnier
2010-03-27  1:00                       ` YAMAMOTO Mitsuharu
2010-03-17 13:30   ` Leo
2010-03-20 19:45   ` Stefan Monnier
2010-03-21  4:43     ` YAMAMOTO Mitsuharu
     [not found] <20100314185409.00EDD9B718@mxperim5.sea5.speakeasy.net>
2010-03-15  2:43 ` Steve Revilak
  -- strict thread matches above, loose matches on Subject: below --
2010-03-14  0:32 Nick Roberts
2010-03-14 19:45 ` Harald Hanche-Olsen

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=19365.49601.379093.424303@totara.tehura.co.nz \
    --to=nickrob@snap.net.nz \
    --cc=emacs-devel@gnu.org \
    --cc=mituharu@math.s.chiba-u.ac.jp \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.