From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <zsh-workers-return-40341-mason-zsh=primenet.com.au@zsh.org>
Received: (qmail 19592 invoked by alias); 12 Jan 2017 16:35:14 -0000
Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm
Precedence: bulk
X-No-Archive: yes
List-Id: Zsh Workers List <zsh-workers.zsh.org>
List-Post: <mailto:zsh-workers@zsh.org>
List-Help: <mailto:zsh-workers-help@zsh.org>
X-Seq: 40341
Received: (qmail 15165 invoked from network); 12 Jan 2017 16:35:14 -0000
X-Qmail-Scanner-Diagnostics: from mailout4.w1.samsung.com by f.primenet.com.au (envelope-from <p.stephenson@samsung.com>, uid 7791) with qmail-scanner-2.11 
 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1.  
 Clear:RC:0(210.118.77.14):SA:0(-8.2/5.0):. 
 Processed in 2.752814 secs); 12 Jan 2017 16:35:14 -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=-8.2 required=5.0 tests=RCVD_IN_DNSWL_HI,
	RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,RP_MATCHES_RCVD,SPF_HELO_PASS
	autolearn=unavailable autolearn_force=no version=3.4.1
X-Envelope-From: p.stephenson@samsung.com
X-Qmail-Scanner-Mime-Attachments: |
X-Qmail-Scanner-Zip-Files: |
Received-SPF: none (ns1.primenet.com.au: domain at samsung.com does not designate permitted sender hosts)
X-AuditID: cbfec7f2-f790f6d000002555-89-5877b03553c4
Date: Thu, 12 Jan 2017 16:34:57 +0000
From: Peter Stephenson <p.stephenson@samsung.com>
To: Zsh Hackers' List <zsh-workers@zsh.org>
Subject: Re: PATCH: autoload with explicit path
Message-id: <20170112163457.152b52a1@pwslap01u.europe.root.pri>
In-reply-to: <20170112162338.GA8691@fujitsu.shahaf.local2>
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+NgFnrJIsWRmVeSWpSXmKPExsWy7djPc7qmG8ojDDY94bM42PyQyYHRY9XB
	D0wBjFFcNimpOZllqUX6dglcGTtWyhW8Zqtonj6PuYFxFWsXIyeHhICJxMsTC1ggbDGJC/fW
	s3UxcnEICSxllFi/uosRJCEk0Msk0fNTG6Zh5stLTBBFyxglVr9ezA5RNI1JYsM9Xgj7DKPE
	vmuREEVnGSXu/nkNVsQioCrx99xCsNVsAoYSUzfNBtrAwSEioC3R/lEMJCwsoC8x/XMvWDmv
	gL3Ehl3dYDangJXE+/sg13Fy8APVXP37iQniIHuJmVfOMELUC0r8mHwP7BtmAR2Jbdses0PY
	8hKb17xlBrlHQuA3m8TjZ+/ZQfZKCMhKbDrADDHHReLoimdQM4UlXh3fwg5hy0hcntwNDaF+
	Rokn3b4Qc2YwSpw+s4MNImEt0Xf7IiPEMj6JSdumM0PM55XoaBOCKPGQmPF4FzTUHSXOLnzD
	MoFRcRaSs2chOXsWkrMXMDKvYhRJLS3OTU8tNtYrTswtLs1L10vOz93ECEwAp/8d/7SD8esJ
	q0OMAhyMSjzAUCuLEGJNLCuuzD3EKMHBrCTCu29deYQQb0piZVVqUX58UWlOavEhRmkOFiVx
	3j0LroQLCaQnlqRmp6YWpBbBZJk4OKUaGG3PMHtUnN0790LLuilTmz73fbp44vIahu7KWl2n
	LJWuiJl90/7WHOVL2KjeFmoppNPxN/PRnOxPC1b1uT5Otlf7pj7LyvGx8MqS5v27gwTS00oX
	BOefv3bxUcKPSVe7rC76r+jLX6/BxNzw7OWBNsHfTAdsnQSdrW40HooNPyW71myWzq7euUos
	xRmJhlrMRcWJAMfwJMz8AgAA
X-Brightmail-Tracker:
 H4sIAAAAAAAAA+NgFlrKIsWRmVeSWpSXmKPExsVy+t/xa7q6G8ojDC5Ol7c42PyQyYHRY9XB
	D0wBjFFuNhmpiSmpRQqpecn5KZl56bZKoSFuuhZKCnmJuam2ShG6viFBSgpliTmlQJ6RARpw
	cA5wD1bSt0twy9ixUq7gNVtF8/R5zA2Mq1i7GDk5JARMJGa+vMQEYYtJXLi3nq2LkYtDSGAJ
	o8Sig79YIZwZTBKvp7yHcs4xShy8tIMZwjnLKHHi51EWkH4WAVWJv+cWgs1lEzCUmLppNmMX
	IweHiIC2RPtHMZCwsIC+xPTPvewgNq+AvcSGXd1gNqeAlcT7+zCrJzJL3Ok4BpbgB2q4+vcT
	1H32EjOvnGGEaBaU+DH5HtheZgEtic3bmlghbHmJzWveMoPYQgLqEjfu7mafwCg8C0nLLCQt
	s5C0LGBkXsUoklpanJueW2ykV5yYW1yal66XnJ+7iREYRduO/dyyg7HrXfAhRgEORiUe3h02
	ZRFCrIllxZW5hxglOJiVRHj3rSuPEOJNSaysSi3Kjy8qzUktPsRoCgyYicxSosn5wAjPK4k3
	NDE0tzQ0MrawMDcyUhLnnfrhSriQQHpiSWp2ampBahFMHxMHp1QD47Gbp1/8/ehXH/fuHvPH
	mpXzVC8EiP1o+/KzJLClOL3kqtfLE4cWur+LSGt/IOs2QbdLbfemOXtOX34t3cO8ppBtPded
	uJUO9kZsr4S+yuW8jFQNCjt32SunoTnOwevm11PvJgppBS6zeyb4qzOk6++/fwJ5DOxiE883
	aLgZP0g6m8a8x/LIJCWW4oxEQy3mouJEAI1PZ424AgAA
X-MTR: 20000000000000000@CPGS
X-CMS-MailID: 20170112163500eucas1p15a62fe15646f14389dfd7100d5c17656
X-Msg-Generator: CA
X-Sender-IP: 182.198.249.180
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-HopCount: 7
X-CMS-RootMailID: 20170112125605eucas1p1b2539afbacec2d28d44c6fd73b0d50af
X-RootMTR: 20170112125605eucas1p1b2539afbacec2d28d44c6fd73b0d50af
References:
 <CGME20170112125605eucas1p1b2539afbacec2d28d44c6fd73b0d50af@eucas1p1.samsung.com>
 <20170112125602.4fa64dc6@pwslap01u.europe.root.pri>
 <20170112154057.GA4899@fujitsu.shahaf.local2>
 <20170112155920.360ad5d7@pwslap01u.europe.root.pri>
 <20170112160921.GA8048@fujitsu.shahaf.local2>
 <20170112161651.46865f31@pwslap01u.europe.root.pri>
 <20170112162338.GA8691@fujitsu.shahaf.local2>

On Thu, 12 Jan 2017 16:23:38 +0000
Daniel Shahaf <d.s@daniel.shahaf.name> wrote:
> The point is that the next time writes:
> .
>     foo(shf->filename);
> .
> that won't compile, so he will be forced to take into account the two
> distinct overloaded meanings.

OK, I can see that might be useful.

> This might have prevented the bug you fixed in 40335 from being
> written.

No, it would still have been possible to resolve all uses into one form
or the other.  The problem was it was done implicitly by location in the
code and needed discriminating by something else such as the new bit
(or the implied NULL / non-NULL test of a separate field).

> I'll see if adding it makes things clearer, will post if it does.

Fine, you might want to see what happens with the change for caching the
autoload directory first as it hits the same lines.

pws