[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#34641: rx: (or ...) order unpredictable
From: |
Mattias Engdegård |
Subject: |
bug#34641: rx: (or ...) order unpredictable |
Date: |
Sun, 3 Mar 2019 09:54:13 +0100 |
3 mars 2019 kl. 00.48 skrev Phil Sainty <psainty@orcon.net.nz>:
>
>>> +The optional argument NOREORDER, if nil, allows the returned
>
> Could we change that name to have a positive sense?
>
> Boolean arguments with a negative sense/meaning are invariably
> more awkward to read than ones with a positive meaning, IMO.
>
> NOREORDER set to nil means "No NOREORDER" (aka "Reorder"). Those
> double-negatives should be avoided whenever it's simple do do so,
> as they make things harder for anyone reading the documentation.
>
> We could use KEEP-ORDER or RETAIN-ORDER or SAME-ORDER or anything
> along those lines, and then a 'true' value matches the positive
> sense of the name, which is much nicer.
You are right, and I wasn't happy with the negative name either.
Would KEEP-ORDER do?