From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from duke.felloff.net ([216.126.196.34]) by ewsd; Thu Nov 15 13:00:12 EST 2018
Message-ID: <BF76DDEC3ABAB8EB078F0BDE14A1B4FD@felloff.net>
Date: Thu, 15 Nov 2018 19:00:05 +0100
From: cinap_lenrek@felloff.net
To: 9front@9front.org
Subject: Re: [9front] man rc patch
In-Reply-To: 20181111171607.GA9751@master
MIME-Version: 1.0
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit
List-ID: <9front.9front.org>
List-Help: <http://lists.9front.org>
X-Glyph: ➈
X-Bullshit: storage out-scaling markup DOM-scale optimizer 

but thats not what the patch is doing :-)

 +               if(star && 1<=n && n<=count(star)){
 +                       while(--n) star = star->next;
 +                       a = newword(star->word, a);
 +               }

this means for $"n where n is out of range, it would still
yield a null list.

also, i do not like how Xqdol() duplicates the logic.
it really should be two separate instructions. Xdol
doing the dereference of the variable, handling the special
star variables... and another instruction Xqote or something
that flattens it out to a single string where () becomes
the empty string.

--
cinap