From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15182 invoked by alias); 9 Oct 2017 08:45:01 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 41834 Received: (qmail 13912 invoked by uid 1010); 9 Oct 2017 08:45:01 -0000 X-Qmail-Scanner-Diagnostics: from mailout2.w1.samsung.com by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1. Clear:RC:0(210.118.77.12):SA:0(-6.9/5.0):. Processed in 3.674626 secs); 09 Oct 2017 08:45:01 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.1 X-Envelope-From: p.stephenson@samsung.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | X-AuditID: cbfec7f1-f793a6d00000326b-00-59db3703aafa Date: Mon, 09 Oct 2017 09:44:51 +0100 From: Peter Stephenson To: Zsh hackers list Subject: Re: [bug] sh: tilde expansion after field splitting Message-id: <20171009094451.1334c863@pwslap01u.europe.root.pri> In-reply-to: <171008134514.ZM12272@torch.brasslantern.com> Organization: Samsung Cambridge Solution Centre X-Mailer: Claws Mail 3.7.9 (GTK+ 2.22.0; i386-redhat-linux-gnu) MIME-version: 1.0 Content-type: text/plain; charset="US-ASCII" Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrLIsWRmVeSWpSXmKPExsWy7djP87rM5rcjDdpXC1ocbH7I5MDoserg B6YAxigum5TUnMyy1CJ9uwSujO9bdQpecVRMXDSFpYHxE1sXIyeHhICJxKkTe1ggbDGJC/fW g8WFBJYySmy7VNTFyAVk9zJJrF/7jwWmYfWfHUwQiWWMElMXTGCGcKYxScw9s4AVwjnNKDF3 zlF2COcMo8T0pTfABrMIqErc2T4PbBabgKHE1E2zGUFsEQEtiR0nTzKB2MICNhK/Jp4AmsTB wStgL/HsTipImFPASqK3cwVYCb+AvsTVv5+YIE6yl5h55QzYGF4BQYkfk++BjWcW0JHYtu0x O4QtL7F5zVuwSyUEFrBJ/J7dBNXsIrG97w87hC0s8er4FihbRqKz4yBUTT+jxJNuX4jmGYwS p8/sgIaetUTf7YuMEBv4JCZtm84McrSEAK9ER5sQRImHxJzeRlYI21Hi873z0ECZziSx8O5l tgmMCrOQHD4LyeGzkBy+gJF5FaNIamlxbnpqsZFecWJucWleul5yfu4mRmAiOP3v+McdjO9P WB1iFOBgVOLhvTHjVqQQa2JZcWXuIUYJDmYlEd6PRrcjhXhTEiurUovy44tKc1KLDzFKc7Ao ifPaRrVFCgmkJ5akZqemFqQWwWSZODilGhgPmqx6siq+4N3C2o79t/V/NyUqv/yz7OXjtU/0 jtzIqXYWmZ8vvdvl+yV1ZlcRmflzDqRZLxDRLPvmaXsl88P2HSa9787o/UoLbDrVtHjpE4Xb mfafahR4G+QKftdvyDWc3jkr/ttHlymLFzrc5k+2vT3x8mUxvVbhy4w7dvKxZ5Rvmv25fM9y JZbijERDLeai4kQAFr8whwADAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrPLMWRmVeSWpSXmKPExsVy+t/xK7pM5rcjDZbt1rU42PyQyYHRY9XB D0wBjFFcNimpOZllqUX6dglcGd+36hS84qiYuGgKSwPjJ7YuRk4OCQETidV/djBB2GISF+6t B4pzcQgJLGGUWPv5MQuEM4NJ4sG5FawQzmlGiTsbTjBCOGcYJSbcXcYO0s8ioCpxZ/s8FhCb TcBQYuqm2YwgtoiAlsSOkyfBdggL2Ej8mngCaBIHB6+AvcSzO6kgYU4BK4nezhVMEDOnM0nM +/YXrJdfQF/i6t9PUPfZS8y8cgYszisgKPFj8j2wXcxA8zdva2KFsOUlNq95ywxiCwmoS9y4 u5t9AqPwLCQts5C0zELSsoCReRWjSGppcW56brGhXnFibnFpXrpecn7uJkZgMG879nPzDsZL G4MPMQpwMCrx8DZMuxUpxJpYVlyZe4hRgoNZSYT3o9HtSCHelMTKqtSi/Pii0pzU4kOM0hws SuK8vXtWRwoJpCeWpGanphakFsFkmTg4pRoYQy4d3e+xQclb8OPe07s3rlO8rVCvUbfxzzv7 X4KvV8vkZCoULJDi8uSYr5/KUKe2477rSUUegWNvrPb2f7BujL3RXr7nu5xjZRD3c06Fryyb Tyl8jlWXYLGPY6/7+bjx5zK243sFHorf/sgbHRMSFvp8dVTkfXu1DL4VPpl/Mic/u837x/eM EktxRqKhFnNRcSIAB8Agk2ICAAA= X-CMS-MailID: 20171009084450eucas1p2df0b8f1baa3174da15fc7a47c2e32c50 X-Msg-Generator: CA X-Sender-IP: 182.198.249.179 X-Local-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUb?= =?UTF-8?B?7IK87ISx7KCE7J6QG1ByaW5jaXBhbCBFbmdpbmVlciwgU29mdHdhcmU=?= X-Global-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUbU2Ft?= =?UTF-8?B?c3VuZyBFbGVjdHJvbmljcxtQcmluY2lwYWwgRW5naW5lZXIsIFNvZnR3YXJl?= X-Sender-Code: =?UTF-8?B?QzEwG0VIURtDMTBDRDA1Q0QwNTAwNTg=?= CMS-TYPE: 201P X-CMS-RootMailID: 20171008204546epcas3p3cee5a462865b44c8a76f39b9d16a9d1f X-RootMTR: 20171008204546epcas3p3cee5a462865b44c8a76f39b9d16a9d1f References: <06fb6a91-6766-bbd7-9543-cbafe704ee59@inlv.org> <171008005347.ZM1177@torch.brasslantern.com> <20171008202032.11ea7bf9@ntlworld.com> <171008134514.ZM12272@torch.brasslantern.com> On Sun, 8 Oct 2017 13:45:14 -0700 Bart Schaefer wrote: > } > - There either isn't a Test/ for the keyvalpairelement() case in the > } > first hunk below, or it isn't rigorous enough, because I initially > } > forgot the incnode(node) in that hunk, yet the shell did *not* go > } > into an infinite loop during "make check", nor did any test > } > fail > } > } That doesn't make sense. > > I don't know what to tell you; I just now commented it back out and tried > again, and it still works. I suspect what's happening is that after the > first call to keyvalpairelement() node = insnode is done, and then we > continue around the loop and call keyvalpairelement() a second time on > the same node, at which point it fails and we fall through the rest of > the loop and hit the incnode() at the end, which fixes things. Yes, that's it --- the key / value pair is turned into a Marker, key, value list triad, and furthermore the key and value are untokenised as no more substitution is wanted. So it'll just go striaght through. This doesn't indicate a problem. pws