[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.
From: |
Jim Ursetto |
Subject: |
Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path. |
Date: |
Fri, 15 Mar 2013 16:17:07 -0500 |
On Mar 15, 2013, at 3:35 PM, Peter Bex wrote:
>
> Finally, a flag would also need to go everywhere a path is accessed
> (including file-open, directory, directory?, call-with-input-file, etc...)
> unless you make it a parameter (which would be slower, if I'm not
> mistaken).
Oops, I see you already mentioned the parameter possibility.
Sure, it would be slower in theory, but looking at the code
for e.g. open-input-file I doubt you would even notice. And
looking at something like with-input-from-file, which calls
apply, fluid-let, call-with-values, values, and apply again,
you will certainly not notice the difference in that case.
I think it would unusual to specify this flag on a per-call
basis; more typically it would be once at toplevel, so you're
not really incurring a per-call penalty to set the parameter,
just a small penalty to read the parameter in the procedure.
In a case where you normally don't want expansion but
occasionally do, you can call expand-path directly, so I
would argue for an expand-path procedure + expand-pathnames
parameter.
That said, even if you go the parameter way, you probably
need a backwards-compatibility module anyway.
Jim
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., (continued)
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Alaric Snell-Pym, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Florian Zumbiehl, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Jörg F . Wittenberger, 2013/03/20
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/18
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path.,
Jim Ursetto <=
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Felix, 2013/03/16
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Peter Bex, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., John Cowan, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., John Cowan, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Mario Domenech Goulart, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., John Cowan, 2013/03/19
- Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Jim Ursetto, 2013/03/19
Re: [Chicken-hackers] [PATCH 3/4] Remove ##sys#expand-home-path., Jim Ursetto, 2013/03/15
Re: [Chicken-hackers] [PATCH 1/4] csi dirseparator?: don't treat backslash as dir separator on non-windows, Christian Kellermann, 2013/03/15