zsh-workers
 help / color / mirror / code / Atom feed
* old patches that might have been missed
@ 2018-05-09 15:24 ` Oliver Kiddle
  2018-05-09 15:30   ` dana
  2018-05-11  8:37   ` Peter Stephenson
  0 siblings, 2 replies; 3+ messages in thread
From: Oliver Kiddle @ 2018-05-09 15:24 UTC (permalink / raw)
  To: Zsh workers; +Cc: lists, dana

Checking over mailing list messages that arrived at times when I was
busy with other things, I see there are some patches that didn't get
applied. Without having gone into details, some of the following
seem reasonable enough:

42101, dana: Fix value of $? when used in array assignment
42113, dana: Millisecond and microsecond specifiers for TIMEFMT
42180, dana: Completion segmentation fault
42234/42226, Stephane: kill %jobspec tries to kill dead processes
42297, dana: Fix (e) subscript parameter used in scalar context
42363, Eitan Adler: REPORTMEMORY units

Any thoughts on any of those?

There's also the Inf/NaN changes which I was leaving until after 5.5 was
settled, which I guess it is now.

Finally, there's 42533 that was rejected because it was suggesting
disabling typeset family reserved words in ksh emulation which I
wouldn't think improves compatibility with real ksh. But some subset of
that patch may be reasonable, for sh emulation only.

Oliver


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: old patches that might have been missed
  2018-05-09 15:24 ` old patches that might have been missed Oliver Kiddle
@ 2018-05-09 15:30   ` dana
  2018-05-11  8:37   ` Peter Stephenson
  1 sibling, 0 replies; 3+ messages in thread
From: dana @ 2018-05-09 15:30 UTC (permalink / raw)
  To: Oliver Kiddle; +Cc: Zsh workers, lists

On 9 May 2018, at 10:24, Oliver Kiddle <okiddle@yahoo.co.uk> wrote:
>42113, dana: Millisecond and microsecond specifiers for TIMEFMT

That one was merged, cb04ae40af

dana


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: old patches that might have been missed
  2018-05-09 15:24 ` old patches that might have been missed Oliver Kiddle
  2018-05-09 15:30   ` dana
@ 2018-05-11  8:37   ` Peter Stephenson
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Stephenson @ 2018-05-11  8:37 UTC (permalink / raw)
  To: Zsh workers

On Wed, 09 May 2018 17:24:25 +0200
Oliver Kiddle <okiddle@yahoo.co.uk> wrote:
> Checking over mailing list messages that arrived at times when I was
> busy with other things, I see there are some patches that didn't get
> applied. Without having gone into details, some of the following
> seem reasonable enough:

Thanks, this certainly needed doing.  I've started looking.

> 42101, dana: Fix value of $? when used in array assignment

I slightly tweaked this to avoid a compiler warning.  Manipulating the
status during code execution is never elegant (as you may have noiced
with the patch a couple of days ago).

> 42180, dana: Completion segmentation fault

There may be more to this problem, but without a research project making
the code a little safer as here seems a definite improvement.

I've submitted these two.

I'll look at the others.

> There's also the Inf/NaN changes which I was leaving until after 5.5
> was settled, which I guess it is now.

Sounds OK to me, though I don't think anything I do is going to be
particularly affected.

> Finally, there's 42533 that was rejected because it was suggesting
> disabling typeset family reserved words in ksh emulation which I
> wouldn't think improves compatibility with real ksh. But some subset
> of that patch may be reasonable, for sh emulation only.

Yes, I think so.

pws


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-05-11  8:45 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20180509152510epcas2p26241dc4717cf7d1ccc317aa1eedb05e1@epcas2p2.samsung.com>
2018-05-09 15:24 ` old patches that might have been missed Oliver Kiddle
2018-05-09 15:30   ` dana
2018-05-11  8:37   ` Peter Stephenson

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).