Zsh Mailing List Archive Messages sorted by: Reverse Date, Date, Thread, Author

Re: [Bug] comparisson operator `[[` not working as expected

X-seq: zsh-workers 50381
From: Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>
To: Mikael Magnusson <mikachu@xxxxxxxxx>
Cc: Paulo Diovani <paulo@xxxxxxxxxxx>, Zsh hackers list <zsh-workers@xxxxxxx>
Subject: Re: [Bug] comparisson operator `[[` not working as expected
Date: Mon, 20 Jun 2022 20:42:34 -0700
Archived-at: <https://zsh.org/workers/50381>
In-reply-to: <CAHYJk3RA3KJ3+AQP0KrYLkq-z0nXL-fuROwz7X+MgsUhUd8VcQ@mail.gmail.com>
List-id: <zsh-workers.zsh.org>
References: <CABdM12-XmzJNFECJto8hR0YROQOuq71p20Hb9jVAx6YNyF2RDg@mail.gmail.com> <CAH+w=7YnmfM3g6j9TB6NXHcFE7yTSY3-UG8wKg1OicwSgj3VZw@mail.gmail.com> <CAHYJk3RA3KJ3+AQP0KrYLkq-z0nXL-fuROwz7X+MgsUhUd8VcQ@mail.gmail.com>

On Mon, Jun 20, 2022 at 5:47 PM Mikael Magnusson <mikachu@xxxxxxxxx> wrote:
>
> > -               curmaxjob = oldmaxjob ? oldmaxjob - 1 : 0;
> > +               curmaxjob = oldmaxjob ? oldmaxjob : 0;
>
> Isn't this now just a very funny way to spell
>   curmaxjob = oldmaxjob;
> ?

Now that you mention it, yes.  Removing the -1 was just the closest
equivalent to the patch that created the problem.

References:
- [Bug] comparisson operator `[[` not working as expected
  - From: Paulo Diovani
- Re: [Bug] comparisson operator `[[` not working as expected
  - From: Bart Schaefer
- Re: [Bug] comparisson operator `[[` not working as expected
  - From: Mikael Magnusson

Messages sorted by: Reverse Date, Date, Thread, Author