From: Frank Terbeck <ft@bewatermyfriend.org>
To: zsh-workers@zsh.org
Cc: richih.mailinglist@gmail.com
Subject: Re: PATCH: PATCH: Add `sourcetrace' option
Date: Sat, 23 Jan 2010 23:14:43 +0100 [thread overview]
Message-ID: <877hr8xycs.fsf@ft.bewatermyfriend.org> (raw)
In-Reply-To: <1264283601-27005-1-git-send-email-ft@bewatermyfriend.org> (Frank Terbeck's message of "Sat, 23 Jan 2010 22:53:21 +0100")
Frank Terbeck writes:
[...]
> diff --git a/Src/init.c b/Src/init.c
> index 123e20e..c59d632 100644
> --- a/Src/init.c
> +++ b/Src/init.c
> @@ -1102,6 +1102,11 @@ source(char *s)
> return SOURCE_NOT_FOUND;
> }
>
> + if (isset(SOURCETRACE)) {
> + fprintf(stderr, "sourcetrace> \"%s\"\n", s);
> + fflush(stderr);
> + }
> +
> /* save the current shell state */
> fd = SHIN; /* store the shell input fd */
> obshin = bshin; /* store file handle for buffered shell input */
[...]
Okay, this breaks menu selection for me...
I've got no idea what's going on. Needless to say, this patch is worth
nothing if it breaks other parts of the shell. The test suite seems to
pass just fine. But menu selection, as I said doesn't work anymore.
I'm really interested in why the code above blows up menu selection.
Anyone who knows what's going on, feel free to explain what the deal
is here.
Regards, Frank - puzzled.
next prev parent reply other threads:[~2010-01-23 22:17 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4B582F3C.2010709@mycircuit.org>
[not found] ` <100121080540.ZM21444@torch.brasslantern.com>
[not found] ` <4B58AB3A.7050300@mycircuit.org>
2010-01-23 10:18 ` Frank Terbeck
2010-01-23 15:46 ` Baptiste Daroussin
2010-01-23 20:16 ` Richard Hartmann
2010-01-23 21:53 ` PATCH: " Frank Terbeck
2010-01-23 22:07 ` Benjamin R. Haskell
2010-01-23 22:23 ` Frank Terbeck
2010-01-23 23:46 ` Bart Schaefer
2010-01-24 0:10 ` Frank Terbeck
2010-01-24 0:18 ` Frank Terbeck
2010-01-24 21:12 ` Peter Stephenson
2010-01-24 22:27 ` Frank Terbeck
2010-01-23 22:14 ` Frank Terbeck [this message]
2010-01-23 23:44 ` PATCH: " Bart Schaefer
2010-01-24 0:05 ` Frank Terbeck
2010-01-24 9:21 ` Peter
[not found] ` <m3fx5y2le2.fsf@klanderman.net>
[not found] ` <87hbqdu9xs.fsf@ft.bewatermyfriend.org>
[not found] ` <4B5A26BA.5080307@mycircuit.org>
[not found] ` <87fx5x6918.fsf@ft.bewatermyfriend.org>
[not found] ` <4B5ABB0D.9010504@mycircuit.org>
[not found] ` <14989d6e1001230228k22e32200s10db146bd8596dc2@mail.gmail.com>
2010-01-23 12:03 ` what is the effect of a percentage sign (%) as last character on the command line Peter
[not found] ` <2d3b879b1001211336m5b78982au96f267ea472af43d@mail.gmail.com>
[not found] ` <4B58D35D.1050208@mycircuit.org>
[not found] ` <100121182713.ZM21821@torch.brasslantern.com>
[not found] ` <4B597061.10701@mycircuit.org>
[not found] ` <100123155239.ZM27228@torch.brasslantern.com>
2010-01-24 9:51 ` Peter
2010-01-24 10:58 ` Frank Terbeck
2010-01-24 14:50 ` Peter
2010-01-26 7:52 ` Peter
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=877hr8xycs.fsf@ft.bewatermyfriend.org \
--to=ft@bewatermyfriend.org \
--cc=richih.mailinglist@gmail.com \
--cc=zsh-workers@zsh.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.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
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 NNTP newsgroup(s).