Re: [PATCH] Ghost specifiers
- First Post
- Replies
- Stats
-
Go to
- ----- 2024 -----
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2007 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2006 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2005 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2004 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2003 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2002 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2001 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2000 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1999 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1998 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
Olivier Galibert <galibert(a)pobox.com> writes:
On Fri, May 01, 1998 at 12:29:42AM -0700, Ben Wing wrote:
> In the specifier code and in a lot of other code that I wrote, I had to
> copy internal lists whenever they were returned just to make sure that
> malicious code couldn't mess things up by modifying them. This is
> inefficient, and could be avoided if we had read-only lists. The same
> thing goes for strings, although they aren't generally copied.
What do you want exactly, read-only or copy-on-write ?
Read-only.
(prog1 'bar
(aset (symbol-name 'bar) 0 ?x))
=> xar
Or:
(setq bar 10)
=> 10
(aset (symbol-name 'bar) 0 ?x)
=> ?x
bar
--> Symbol's value as variable is void: bar
In both cases, `aset' should have signaled an error.
--
Hrvoje Niksic <hniksic(a)srce.hr> | Student at FER Zagreb, Croatia
--------------------------------+--------------------------------
Reporter: Mr Gandhi, what do you think of Western Civilization?
Gandhi: I think it would be a good idea.
Olivier Galibert
Friday, 1 May
Fri, 1 May
8:15 p.m.
New subject: [PATCH] Ghost specifiers
On Fri, May 01, 1998 at 08:33:01PM +0200, Hrvoje Niksic wrote:
> What do you want exactly, read-only or copy-on-write ?
Read-only.
(prog1 'bar
(aset (symbol-name 'bar) 0 ?x))
=> xar
Or:
(setq bar 10)
=> 10
(aset (symbol-name 'bar) 0 ?x)
=> ?x
bar
--> Symbol's value as variable is void: bar
In both cases, `aset' should have signaled an error.
With the indexed lrecord implementation, it is quite easy to do. But
I'm not convinced this is what you really want to have:
(setq bar 10)
10
(setq nbar (symbol-name 'bar))
"bar"
(aset nbar 0 ?x)
?x
nbar
"xar"
bar
--> Symbol's value as variable is void: bar
How am I supposed to work on the string which happens to be initially
a symbol name ? I see no "copy-string" or equivalent function.
Maybe the answer is "it is broken now, it will only detect bugs" :-)
OG.
Kirill M. Katsnelson
Friday, 1 May
Fri, 1 May
8:59 p.m.
New subject: [PATCH] Ghost specifiers
Some time ago, Olivier Galibert wrote...
|+
|
| With the indexed lrecord implementation, it is quite easy to do. But
| I'm not convinced this is what you really want to have:
|
| (setq bar 10)
| 10
| (setq nbar (symbol-name 'bar))
| "bar"
| (aset nbar 0 ?x)
| ?x
| nbar
| "xar"
| bar
| --> Symbol's value as variable is void: bar
|
| How am I supposed to work on the string which happens to be initially
| a symbol name ? I see no "copy-string" or equivalent function.
(copy-sequence) is the function you are looking for.
So, IMO the correct behavior in your sample would be that aset signals;
although if you write (setq nbar (copy-sequence (symbol-name 'bar))) it
is ok, as it mangles with a copy of the string.
Kirill
9501
days inactive
9501
days old
2 comments
3 participants
tags (0)
participants (3)
-
Hrvoje Niksic -
Kirill M. Katsnelson -
Olivier Galibert