On 2023-08-07  13:53, Ihor Radchenko wrote:

Thanks for reviewing.

>>   - If the comparison value is a plain number, a numerical comparison is
>>     done, and the allowed operators are =<=, ===, =>=, =<==, =>==, and
>> -  =<>=.
>> +  =<>=.  As synonym for the equality operator there is also ====, as
>> +  synonyms for the inequality operator there are =!== and =/==.
> 
> I'd mention which operator is "equality" and "inequality" to avoid
> ambiguity:
> 
> As a synonym for the equality operator ===, there is also ====; =!== and
> =/== are synonyms of inequality operator =<>=.

Done.

>> -		      (let* ((gv (pcase (upcase (match-string 5 term))
>> +		      (let* (;; Convert property name to an Elisp
>> +			     ;; accessor for that property (aka. as
>> +			     ;; getter value?).
 >
 > Is there any specific reason why you put question mark in the comment
 > here?

I was trying to decipher and document the acronyms used for the
let-bindings in that `let*' sexp.  I think I got them right but was
unsure about the "gv".  Removed the question mark, let's just assume
that "gv" means "getter value" and be done with it.

Updated patch attached.